Move groups post-processor to store onProxyLoad and initial sort to c…

…onfig
SlateFoundation · Sep 14, 2015 · af9474c · af9474c
1 parent 944113a
commit af9474c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 17 deletions.
diff --git a/sencha-workspace/SlateAdmin/app/controller/People.js b/sencha-workspace/SlateAdmin/app/controller/People.js
@@ -132,12 +132,9 @@ Ext.define('SlateAdmin.controller.People', {
 //        selector: 'people-navpanel #searchOptionsForm'
     }],
 
-
     // controller template methods
     init: function() {
-        var me = this,
-            groupStore = Ext.getStore('people.Groups'),
-            path, parentGroup;
+        var me = this;
 
         me.control({
             'people-navpanel': {
@@ -185,19 +182,6 @@ Ext.define('SlateAdmin.controller.People', {
 //            }
         });
 
-        groupStore.on('load', function (group) {
-            groupStore.sort('LEFT', 'ASC');
-            groupStore.each(function(group) {
-                path = '';
-                if(group.get('ParentID')) {
-                    parentGroup = groupStore.getById(group.get('ParentID'));
-                    path = parentGroup.get('namesPath');
-                }
-
-                group.set('namesPath', path + '/' + group.get('Name'));
-            });
-        });
-
         me.listen({
             store: {
                 '#People': {

diff --git a/sencha-workspace/SlateAdmin/app/store/people/Groups.js b/sencha-workspace/SlateAdmin/app/store/people/Groups.js
@@ -3,6 +3,7 @@ Ext.define('SlateAdmin.store.people.Groups', {
     extend: 'Ext.data.Store',
 
     model: 'SlateAdmin.model.person.Group',
+    sorters: 'Left',
     proxy: {
         type: 'slaterecords',
         url: '/groups',
@@ -12,5 +13,20 @@ Ext.define('SlateAdmin.store.people.Groups', {
         extraParams: {
             parentGroup: 'any'
         }
+    },
+
+    onProxyLoad: function(operation) {
+        var me = this,
+            i = 0, count, group, parentGroup;
+
+        me.callParent(arguments);
+
+        if (operation.wasSuccessful()) {
+            for (count = me.getCount(); i < count; i++) {
+                group = me.getAt(i);
+                parentGroup = me.getById(group.get('ParentID'));
+                group.set('namesPath', (parentGroup ? parentGroup.get('namesPath') : '') + '/' + group.get('Name'));
+            }
+        }
     }
 });