From b7bc865f93274fef38a1b21d130f417d25edfb28 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 e5db0087b4..2be839f36b 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.43.2