Merge pull request #6971 from Countly/fix/dashboard-app-sorting

[SER-2590] [SER-2591] fix: dashboards list and app management list sorting

Author: tryshank

CHANGELOG.md

@@ -5,7 +5,8 @@ Fixes:
 Enterprise Fixes:
 - [concurrent_users] Fix email check for alert
 - [data-manager] Correctly show last triggered for events if data masking is enabled
-
+- [dashboards] Keep dashboard sidebar sorted alphabetically after additions
+- [applications] Ensure application management list reorders after create/update
 
 ## Version 25.03.27
 Fixes:

frontend/express/public/core/app-management/javascripts/countly.views.js

@@ -59,13 +59,12 @@
                     value: id
                 };
             });
-            if (countlyGlobal.member.appSortList) {
-                appList = this.sortBy(appList, countlyGlobal.member.appSortList);
+            var sortList = (countlyGlobal.member && countlyGlobal.member.appSortList) || [];
+            if (sortList.length) {
+                appList = this.sortBy(appList, sortList);
             }
             else {
-                appList.sort(function(a, b) {
-                    return a.label > b.label && 1 || -1;
-                });
+                appList.sort(this.sortAppOptionsAlphabetically);
             }
             var app_id = this.$route.params.app_id || countlyCommon.ACTIVE_APP_ID;
             return {
@@ -340,6 +339,56 @@
                     }
                 }
             },
+            getAppListItemId: function(option) {
+                if (!option) {
+                    return "";
+                }
+
+                return (option._id || option.value || option.id || option.app_id || "") + "";
+            },
+            sortAppOptionsAlphabetically: function(optionA, optionB) {
+                var labelA = (optionA.label || "").toLowerCase();
+                var labelB = (optionB.label || "").toLowerCase();
+
+                if (labelA < labelB) {
+                    return -1;
+                }
+
+                if (labelA > labelB) {
+                    return 1;
+                }
+
+                var valueA = (optionA.value || "").toLowerCase();
+                var valueB = (optionB.value || "").toLowerCase();
+
+                if (valueA < valueB) {
+                    return -1;
+                }
+
+                if (valueA > valueB) {
+                    return 1;
+                }
+
+                return 0;
+            },
+            applyAppListOrdering: function() {
+                if (!Array.isArray(this.appList)) {
+                    return;
+                }
+
+                var sortList = (countlyGlobal.member && countlyGlobal.member.appSortList) || [];
+                var currentList = this.appList.slice();
+                var orderedList;
+
+                if (sortList.length) {
+                    orderedList = this.sortBy(currentList, sortList);
+                }
+                else {
+                    orderedList = currentList.sort(this.sortAppOptionsAlphabetically);
+                }
+
+                this.appList = orderedList;
+            },
             sortBy: function(arrayToSort, sortList) {
                 if (!sortList.length) {
                     return arrayToSort;
@@ -349,9 +398,10 @@
                     retArr = [];
                 var i;
                 for (i = 0; i < arrayToSort.length; i++) {
-                    var objId = arrayToSort[i]._id + "";
-                    if (sortList.indexOf(objId) !== -1) {
-                        tmpArr[sortList.indexOf(objId)] = arrayToSort[i];
+                    var objId = this.getAppListItemId(arrayToSort[i]);
+                    var desiredIndex = sortList.indexOf(objId);
+                    if (desiredIndex !== -1) {
+                        tmpArr[desiredIndex] = arrayToSort[i];
                     }
                 }
 
@@ -392,6 +442,7 @@
                                     value: data._id + "",
                                     label: data.name
                                 });
+                                self.applyAppListOrdering();
                                 self.$store.dispatch("countlyCommon/addToAllApps", data);
                                 self.$store.dispatch("countlyCommon/updateActiveApp", data._id + "");
                                 countlyCommon.setActiveApp(data._id);
@@ -459,6 +510,7 @@
                                 break;
                             }
                         }
+                        self.applyAppListOrdering();
                         self.discardForm();
                         CountlyHelpers.notify({
                             title: jQuery.i18n.map["configs.changed"],

plugins/dashboards/frontend/public/javascripts/countly.views.js

@@ -1722,7 +1722,22 @@
             allDashboards: function() {
                 var query = this.searchQuery;
 
-                var dashboards = this.$store.getters["countlyDashboards/all"];
+                var dashboards = (this.$store.getters["countlyDashboards/all"] || []).slice();
+
+                dashboards.sort(function(a, b) {
+                    var nameA = (a.name || "").toLowerCase();
+                    var nameB = (b.name || "").toLowerCase();
+
+                    if (nameA < nameB) {
+                        return -1;
+                    }
+
+                    if (nameA > nameB) {
+                        return 1;
+                    }
+
+                    return 0;
+                });
 
                 if (!query) {
                     return dashboards;