fix: Update sort by source functionality to sort also alphabetically

src/app/core/record-affiliations-sort/record-affiliations-sort.service.ts

@@ -1,10 +1,6 @@
 import { Injectable } from '@angular/core'
 import { MonthDayYearDate } from 'src/app/types'
-import {
-  AffiliationUIGroup,
-  AffiliationGroup,
-  AffiliationType,
-} from 'src/app/types/record-affiliation.endpoint'
+import { AffiliationGroup, AffiliationType, AffiliationUIGroup } from 'src/app/types/record-affiliation.endpoint'
 import { UserRecordOptions } from 'src/app/types/record.local'
 
 @Injectable({
@@ -88,15 +84,9 @@ export class AffiliationsSortService {
           }
 
           if (by === 'source') {
-            affiliationGroup.sort((a, b) => {
-              return (
-                Number(AffiliationsSortService.isSelfAsserted(a, orcid)) -
-                Number(AffiliationsSortService.isSelfAsserted(b, orcid))
-              )
-            })
-            if (ascending) {
-              affiliationGroup.reverse()
-            }
+            const selfAsserted = this.getSelfAssertedOrValidatedAffiliations(affiliationGroup, ascending, orcid, 'self-asserted')
+            const validated = this.getSelfAssertedOrValidatedAffiliations(affiliationGroup, ascending, orcid, 'validated')
+            x.affiliationGroup = ascending ? [...selfAsserted, ...validated] : [...validated, ...selfAsserted]
           }
         }
       })
@@ -206,7 +196,21 @@ export class AffiliationsSortService {
       })
   }
 
-  private static isSelfAsserted(
+  private getSelfAssertedOrValidatedAffiliations(affiliationGroup: AffiliationGroup[], ascending: boolean, orcid: string, type: 'self-asserted' | 'validated'): AffiliationGroup[] {
+    return affiliationGroup
+      .filter(
+        (affiliationGroup) => {
+          const selfAsserted = this.isSelfAsserted(affiliationGroup, orcid)
+          return type === 'self-asserted' ? selfAsserted : !selfAsserted
+        },
+      ).sort((a, b) => {
+        return (
+          '' + a.defaultAffiliation.affiliationName.value
+        ).localeCompare('' + b.defaultAffiliation.affiliationName.value)
+      })
+  }
+
+  private isSelfAsserted(
     affiliationGroup: AffiliationGroup,
     orcid: string
   ): boolean {