RANGER-4611 : Security zone form not populate resources value properl…

…y while creating and editing zone. Signed-off-by: Dineshkumar Yadav <[email protected]>
apache · Jan 4, 2024 · c2b83f5 · c2b83f5
1 parent ed3a0f3
commit c2b83f5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/security-admin/src/main/webapp/scripts/collection_bases/RangerZoneListBase.js b/security-admin/src/main/webapp/scripts/collection_bases/RangerZoneListBase.js
@@ -41,9 +41,6 @@ define(function(require){
                         this.modelName = 'RangerZone';
                         this.modelAttrName = 'securityZones';
                         this.bindErrorEvents();
-                },
-                comparator: function(def) {
-                        return def.get('name');
                 }
         },{
                 // static class members

diff --git a/security-admin/src/main/webapp/scripts/controllers/Controller.js b/security-admin/src/main/webapp/scripts/controllers/Controller.js
@@ -574,6 +574,9 @@ define(function(require) {
                 cache: false,
                 async: false,
             });
+            rangerZoneList.models = rangerZoneList.sortBy(function(model) {
+                return model.get('name').toLowerCase();
+            });
             App.rContent.show(new vSecurityZone({
                 rangerService: rangerServiceList,
                 collection: rangerZoneList,

diff --git a/security-admin/src/main/webapp/scripts/controllers/NController.js b/security-admin/src/main/webapp/scripts/controllers/NController.js
@@ -552,6 +552,9 @@ define(function(require) {
                 cache: false,
                 async: false,
             });
+            rangerZoneList.models = rangerZoneList.sortBy(function(model) {
+                return model.get('name').toLowerCase();
+            });
             if (App.rSideBar.currentView) {
                 this.rSidebarContentHideAndShow('SecurityZone');
                 if (!_.isNaN(parseInt(listId))) {