EZP-26897: Display broken when subitems sorted by not displayed column

ezsystems · Jan 25, 2017 · d8a424d · d8a424d
1 parent 365e4be
commit d8a424d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/Resources/public/js/views/subitem/ez-subitemlistmoreview.js b/Resources/public/js/views/subitem/ez-subitemlistmoreview.js
@@ -123,10 +123,12 @@ YUI.add('ez-subitemlistmoreview', function (Y) {
 
             this._resetSortOrderClass();
 
-            if (sortCondition.sortOrder === 'ASC') {
-                column.addClass(COLUMN_SORT_ASC_CLASS);
-            } else {
-                column.addClass(COLUMN_SORT_DESC_CLASS);
+            if (column) {
+                if (sortCondition.sortOrder === 'ASC') {
+                    column.addClass(COLUMN_SORT_ASC_CLASS);
+                } else {
+                    column.addClass(COLUMN_SORT_DESC_CLASS);
+                }
             }
         },
 

diff --git a/Tests/js/views/subitem/assets/ez-subitemlistmoreview-tests.js b/Tests/js/views/subitem/assets/ez-subitemlistmoreview-tests.js
@@ -376,6 +376,12 @@ YUI.add('ez-subitemlistmoreview-tests', function (Y) {
         tearDown: function () {
             this.view.destroy();
         },
+
+        "Should reset the view state when Location sortField is changed for a non sortable field": function () {
+            this._resetView(Y.bind(function () {
+                this.location.set('sortField', 'PATH');
+            },this));
+        },
     }));
 
     updatePriorityTest = new Y.Test.Case({