From aae90985e56a86f4aa7d801a173580009cc6ccf5 Mon Sep 17 00:00:00 2001 From: Steven Callender Date: Thu, 14 Nov 2013 15:42:12 -0500 Subject: [PATCH] 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 Signed-off-by: Ben Shum --- Open-ILS/web/js/ui/default/opac/staff.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/opac/staff.js b/Open-ILS/web/js/ui/default/opac/staff.js index d50653be1c..afc7b503e8 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; -- 2.11.0