Sort cluster by name is the settings

Signed-off-by: azerr <[email protected]>
jlandersen · Jan 18, 2021 · b0dd97d · b0dd97d
1 parent a0257d0
commit b0dd97d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/src/explorer/models/kafka.ts b/src/explorer/models/kafka.ts
@@ -1,5 +1,5 @@
 import { Disposable, TreeItemCollapsibleState } from "vscode";
-import { ClientAccessor, Cluster } from "../../client";
+import { ClientAccessor } from "../../client";
 import { ClusterSettings } from "../../settings";
 import { ClusterItem, NoClusterItem } from "./cluster";
 import { NodeBase } from "./nodeBase";
@@ -16,8 +16,7 @@ export class KafkaModel extends NodeBase implements Disposable {
     }
 
     public async computeChildren(): Promise<NodeBase[]> {
-        const clusters = this.clusterSettings.getAll()
-            .sort(this.sortByNameAscending);
+        const clusters = this.clusterSettings.getAll();
         if (clusters.length === 0) {
             return [new NoClusterItem(this)];
         }
@@ -26,12 +25,6 @@ export class KafkaModel extends NodeBase implements Disposable {
         });
     }
 
-    private sortByNameAscending(a: Cluster, b: Cluster): -1 | 0 | 1 {
-        if (a.name.toLowerCase() < b.name.toLowerCase()) { return -1; }
-        if (a.name.toLowerCase() > b.name.toLowerCase()) { return 1; }
-        return 0;
-    }
-
     public dispose(): void {
         this.children?.forEach(child => (<ClusterItem>child).dispose());
     }

diff --git a/src/settings/clusters.ts b/src/settings/clusters.ts
@@ -21,7 +21,7 @@ export interface ClusterSettings {
     onDidChangeSelected: vscode.Event<SelectedClusterChangedEvent>;
 
     /**
-     * Gets the full cluster collection..
+     * Gets the full cluster collection sorted by cluster name.
      */
     getAll(): Cluster[];
 
@@ -78,7 +78,14 @@ class MementoClusterSettings implements ClusterSettings {
 
     getAll(): Cluster[] {
         const state = this.storage.get<ClusterStoreType>(this.clusterCollectionStorageKey, {});
-        return Object.values(state);
+        return Object.values(state)
+            .sort(this.sortByNameAscending);
+    }
+
+    private sortByNameAscending(a: Cluster, b: Cluster): -1 | 0 | 1 {
+        if (a.name.toLowerCase() < b.name.toLowerCase()) { return -1; }
+        if (a.name.toLowerCase() > b.name.toLowerCase()) { return 1; }
+        return 0;
     }
 
     get(id: string): Cluster | undefined {
@@ -112,7 +119,7 @@ class MementoClusterSettings implements ClusterSettings {
     }
 
     private setSelectedClusterIfNeeded(): void {
-        if (this.selected !== undefined){
+        if (this.selected !== undefined) {
             return;
         }
         const all = this.getAll();
@@ -122,4 +129,4 @@ class MementoClusterSettings implements ClusterSettings {
     }
 }
 
-export const getClusterSettings = (): ClusterSettings => MementoClusterSettings.getInstance();
+export const getClusterSettings = (): ClusterSettings => MementoClusterSettings.getInstance();