Merge pull request #4318 from mosheshp2/move-group-fix

swalters · swalters · commit b75325c60134 · 2015-09-03T15:50:18.000-05:00
fix(grouping): #4318 fixes the moveGroupColumn bug, return original columns instead of null
diff --git a/src/features/grouping/js/grouping.js b/src/features/grouping/js/grouping.js
@@ -608,7 +608,7 @@
        */
       moveGroupColumns: function( grid, columns, rows ){
         if ( grid.options.moveGroupColumns === false){
-          return;
+          return columns;
         }
 
         columns.forEach( function(column, index){

Original file line number	Diff line number	Diff line change
`@@ -608,7 +608,7 @@`
`608`	`608`	`*/`
`609`	`609`	`moveGroupColumns: function( grid, columns, rows ){`
`610`	`610`	`if ( grid.options.moveGroupColumns === false){`
`611`		`- return;`
	`611`	`+ return columns;`
`612`	`612`	`}`
`613`	`613`
`614`	`614`	`columns.forEach( function(column, index){`