From: Jason Stephenson Date: Wed, 14 Feb 2018 23:32:46 +0000 (-0500) Subject: Lp 1749593: Fix TypeError: Cannot set property 'grid' of undefined X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b81e944f8f49fb75b0e15f332238919ca919def6;p=Evergreen.git Lp 1749593: Fix TypeError: Cannot set property 'grid' of undefined Commit b35f0c0 added a reference to the grid in the grid.dataProvider in services/grid.js in the web staff client. This is great, but the dataProvider could be undefined where it was being set. This leads to busted grid views and a TypeError in the console. This commit moves the code down until after a fallback dataProvider is used when the dataProvider is undefined and thereby avoids both problems of adding the grid to an undefined dataProvider. Signed-off-by: Jason Stephenson Signed-off-by: Kathy Lussier --- diff --git a/Open-ILS/web/js/ui/default/staff/services/grid.js b/Open-ILS/web/js/ui/default/staff/services/grid.js index b023df8eca..33fd892423 100644 --- a/Open-ILS/web/js/ui/default/staff/services/grid.js +++ b/Open-ILS/web/js/ui/default/staff/services/grid.js @@ -178,9 +178,6 @@ angular.module('egGridMod', delete $scope.idField; grid.dataProvider = $scope.itemsProvider; - // make grid ref available in get() to set totalCount, if known. - // this allows us disable the 'next' paging button correctly - grid.dataProvider.grid = grid; if (!grid.indexField && grid.idlClass) grid.indexField = egCore.idl.classes[grid.idlClass].pkey; @@ -222,6 +219,10 @@ angular.module('egGridMod', }); } + // make grid ref available in get() to set totalCount, if known. + // this allows us disable the 'next' paging button correctly + grid.dataProvider.grid = grid; + grid.dataProvider.columnsProvider = grid.columnsProvider; $scope.itemFieldValue = grid.dataProvider.itemFieldValue;