LP#1805895 Bucket grid configuration updates do not save
authorDan Briem <dbriem@wlsmail.org>
Fri, 5 Jul 2019 23:34:13 +0000 (19:34 -0400)
committerJason Boyer <jboyer@library.in.gov>
Thu, 1 Aug 2019 16:08:00 +0000 (12:08 -0400)
Some saved columns stick, but others are ignored. Issue affects eg-grid[s]
that use the auto-fields attr to load all columns on idl-class attr. This
is because saved columns are whitelisted against the initial column config
before the auto fields are loaded.

Patch adds $scope.handleAutoFields() to egCore.hatch.getItem. This happens
after the eg-grid-fields transclude (no collisions) but before the saved
columns are whitelisted. The original call remains to handle cases where
there is no persist-key or saved config.

Test:
1. Go to Circulation->User Buckets
2. Add the Is Deleted column to the grid
3. Save columns and refresh (note Is Deleted is not visible)
4. Apply patch and repeat steps 1-3 (note Is Deleted is now visible)

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Open-ILS/web/js/ui/default/staff/services/grid.js

index 732ec59..8421cc1 100644 (file)
@@ -498,6 +498,9 @@ angular.module('egGridMod',
                 .then(function(conf) {
                     if (!conf) return;
 
+                    // load all column options before validating saved columns
+                    $scope.handleAutoFields();
+
                     var columns = grid.columnsProvider.columns;
                     var new_cols = [];