[open-ils-commits] [GIT] Evergreen ILS branch rel_3_5 updated. f95cf009f6e131b1cad0bd02f8a31629dee658f0

[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_5 has been updated
       via  f95cf009f6e131b1cad0bd02f8a31629dee658f0 (commit)
      from  d38082cd9df5080fc41c291e3025fade18fb0316 (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 f95cf009f6e131b1cad0bd02f8a31629dee658f0
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date:   Tue Jul 7 16:24:49 2020 -0400

    LP#1886713: fix issue where AngularJS hotkeys fail to initialize
    
    Fixes a race condition where the navbar service can look for
    eg-accesskey attributes before all of the elements that have
    them meet the conditions to show up in the DOM.
    
    To test
    -------
    [1] From an AngularJS part of the staff interface, note that
        hitting F8 sometimes does not perform the retrieve
        last patron action, nor does F8 show up on the list of
        hot keys when you hit "?" outside of an input.
    [2] Apply the patch and repeat step 1. This time, the F8
        hotkey should consistently work.
    
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    
    Signed-off-by: Rogan Hamby <rogan.hamby at gmail.com>
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/web/js/ui/default/staff/services/navbar.js b/Open-ILS/web/js/ui/default/staff/services/navbar.js
index b6ff1d31fe..a00e26a55c 100644
--- a/Open-ILS/web/js/ui/default/staff/services/navbar.js
+++ b/Open-ILS/web/js/ui/default/staff/services/navbar.js
@@ -120,12 +120,15 @@ angular.module('egCoreMod')
 
                                 $scope.showAngularCatalog = 
                                     s['ui.staff.angular_catalog.enabled'];
+                            }).then(function() {
+                                // need to defer initialization of hotkeys to this point
+                                // as it depends on various settings.
+                                $timeout(function(){find_accesskeys($element)});
                             });
+                        } else {
+                            // fallback initialization of hotkeys
+                            $timeout(function(){find_accesskeys($element)});
                         }
-                        // need to defer initialization of hotkeys to this point
-                        // as some of them are conditional on whether one is logged in
-                        // or is working in offline circulation mode
-                        $timeout(function(){find_accesskeys($element)});
                     }
                 );
             }

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

Summary of changes:
 Open-ILS/web/js/ui/default/staff/services/navbar.js | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
Evergreen ILS