From: Bill Erickson <berickxx@gmail.com>
Date: Wed, 22 Jan 2020 15:10:20 +0000 (-0500)
Subject: LP1860468 Org unit admin interface sorting
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=f80965ad811f723af8439b7ecd98eb98db9cb3e7;p=contrib%2FConifer.git

LP1860468 Org unit admin interface sorting

Sort org units by name alphabetically in the Angular Org Unit admin
interface navigation tree.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Lynn Floyd <lfloyd1@library.in.gov>
---

diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/org-unit.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/server/org-unit.component.ts
index 20321b9596..ff5a1846f4 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/org-unit.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/org-unit.component.ts
@@ -96,7 +96,12 @@ export class OrgUnitComponent implements OnInit {
                 'admin.server.org_unit.treenode', {org: orgNode}
             ).then(label => treeNode.label = label);
 
-            orgNode.children().forEach(childNode =>
+            // Tree node labels are "name -- shortname".  Sorting
+            // by name suffices and bypasses the need the wait
+            // for all of the labels to interpolate.
+            orgNode.children()
+            .sort((a, b) => a.name() < b.name() ? -1 : 1)
+            .forEach(childNode =>
                 treeNode.children.push(handleNode(childNode))
             );