[open-ils-commits] [GIT] Evergreen ILS branch rel_2_5 updated. 6ea75506c952a66d0f3aa98b5a662c3da91a8617

[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_2_5 has been updated
       via  6ea75506c952a66d0f3aa98b5a662c3da91a8617 (commit)
      from  eaf9c8ec442c7da292ac377c6ccd83695920caf0 (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 6ea75506c952a66d0f3aa98b5a662c3da91a8617
Author: Steven Callender <stevecallender at esilibrary.com>
Date:   Thu Nov 14 15:42:12 2013 -0500

    LP1251424 - Fix for submit button when placing holds in staff client
    
    I've moved the initializing of the cur_hold_barcode variable
    to within the function rather than floating outside of it.
    This seemed to fix an intermittent issue where the submit
    button would not become activatable when first going to the
    place hold screen via the staff client. To create the hold,
    ths user had to cycle through the "Place hold by patron barcode"
    option and then back to "Place hold for me" option.
    
    It looked like behind the scenes for certain org units that
    variable would not be set and the button will not open until
    it at least has something. This causes it to be set on the initial
    loading of the screen.
    
    Signed-off-by: Steven Callender <stevecallender at esilibrary.com>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/web/js/ui/default/opac/staff.js b/Open-ILS/web/js/ui/default/opac/staff.js
index e5db008..2be839f 100644
--- a/Open-ILS/web/js/ui/default/opac/staff.js
+++ b/Open-ILS/web/js/ui/default/opac/staff.js
@@ -26,9 +26,9 @@ function no_hold_submit(event) {
     }
     return true;
 }
-var cur_hold_barcode = undefined;
 function staff_hold_usr_barcode_changed(isload) {
     if(typeof xulG != 'undefined' && xulG.get_barcode_and_settings) {
+        var cur_hold_barcode = undefined;
         var barcode = isload;
         if(!barcode || barcode === true) barcode = document.getElementById('staff_barcode').value;
         var only_settings = true;

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

Summary of changes:
 Open-ILS/web/js/ui/default/opac/staff.js |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS