From 2de8747fc4fb0cb619667db6723338611e5dfd9b Mon Sep 17 00:00:00 2001 From: Steven Callender Date: Thu, 14 Nov 2013 15:42:12 -0500 Subject: [PATCH] Small change to the code for placing holds. 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 somethig. This causes it to be set on the inital loading of the screen. Signed-off-by: Steven Callender --- 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.11.0