[open-ils-commits] [GIT] Evergreen ILS branch rel_3_3 updated. bb42d7e586a4c502aad280c187cf5d7dd9d672bb

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_3_3 has been updated
       via  bb42d7e586a4c502aad280c187cf5d7dd9d672bb (commit)
      from  338021686f109acd3152547804b0a2b848f2ba99 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bb42d7e586a4c502aad280c187cf5d7dd9d672bb
Author: Mike Rylander <mrylander at gmail.com>
Date:   Fri Sep 14 09:52:03 2018 -0400

    LP#1792188: Record -> View Holds fails to refresh on Next/Prev record
    
    Here we clean up and add back a call to refresh the holds list when navigating
    between records in a result set.
    
    To test
    -------
    [1] In the web staff client, perform a catalog search, bring up a record,
        and change to the View Holds tab.
    [2] Click the "Next" button to navigate to the next hold. Note that
        the list of holds doesn't change.
    [3] Apply the patch, refresh, and repeat steps #1 and #2. This time,
        the list of holds should get updated as you navigate between
        records in the results list.
    
    Signed-off-by: Mike Rylander <mrylander at gmail.com>
    Signed-off-by: John Yorio <jyorio at equinoxinitiative.org>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>

diff --git a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
index 7d525c5f43..8d3ed24a78 100644
--- a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
@@ -591,6 +591,7 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e
                 $scope.conjoinedGridDataProvider.refresh();
             });
             init_parts_url();
+            $scope.grid_actions.refresh();
             $location.update_path('/cat/catalog/record/' + $scope.record_id);
             // update_path() bypasses the controller for path 
             // /cat/catalog/record/:record_id. Manually set title here too.
@@ -1734,13 +1735,14 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e
     // ------------------------------------------------------------------
     // Holds 
     var provider = egGridDataProvider.instance({});
+    var holds = []; // current list of holds
+    var hold_count = 0;
+
     $scope.hold_grid_data_provider = provider;
     $scope.grid_actions = egHoldGridActions;
     $scope.grid_actions.refresh = function () { holds = []; hold_count = 0; provider.refresh() };
     $scope.hold_grid_controls = {};
 
-    var holds = []; // current list of holds
-    var hold_count = 0;
     provider.get = function(offset, count) {
         if ($scope.record_tab != 'holds') return $q.when();
 

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/web/js/ui/default/staff/cat/catalog/app.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS