[open-ils-commits] r14614 - in branches/rel_1_6/Open-ILS: web/opac/locale/en-US xul/staff_client/server/admin xul/staff_client/server/circ (phasefx)

svn at svn.open-ils.org svn at svn.open-ils.org
Mon Oct 26 22:57:48 EDT 2009


Author: phasefx
Date: 2009-10-26 22:57:45 -0400 (Mon, 26 Oct 2009)
New Revision: 14614

Modified:
   branches/rel_1_6/Open-ILS/web/opac/locale/en-US/lang.dtd
   branches/rel_1_6/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
   branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.js
   branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.xul
Log:
backporting changeset 14613: Tweak the Record In-House Use interface so that it has sound, a larger textbox for the '# of uses' field, and org unit settings that control the warning threshold and max allowed value for '# of uses'. 
Exceeding the cap no longer sets the value to the cap.



Modified: branches/rel_1_6/Open-ILS/web/opac/locale/en-US/lang.dtd
===================================================================
--- branches/rel_1_6/Open-ILS/web/opac/locale/en-US/lang.dtd	2009-10-27 02:39:59 UTC (rev 14613)
+++ branches/rel_1_6/Open-ILS/web/opac/locale/en-US/lang.dtd	2009-10-27 02:57:45 UTC (rev 14614)
@@ -2752,6 +2752,10 @@
 <!ENTITY staff.portal.default.welcome "This is the Evergreen staff client portal page. You can customize this with the information or functionality your organization needs to support its operations. For example, you could add links to process and procedure documents.">
 <!ENTITY ui.circ.patron_summary.horizontal "GUI: Alternative Horizontal Patron Summary Panel">
 <!ENTITY ui.circ.patron_summary.horizontal.desc "This replaces the vertical Patron Summary Sidebar.">
+<!ENTITY ui.circ.in_house_use.entry_cap "GUI: Record In-House Use: Maximum # of uses allowed per entry.">
+<!ENTITY ui.circ.in_house_use.entry_cap.desc "The # of uses entry in the Record In-House Use interface may not exceed the value of this setting.">
+<!ENTITY ui.circ.in_house_use.entry_warn "GUI: Record In-House Use: # of uses threshold for Are You Sure? dialog.">
+<!ENTITY ui.circ.in_house_use.entry_warn.desc "In the Record In-House Use interface, a submission attempt will warn if the # of uses field exceeds the value of this setting.">
 <!ENTITY ui.general.button_bar "GUI: Above-Tab Button Bar">
 <!ENTITY ui.general.button_bar.desc "This enables a row of buttons and/or icons between the top-level menu system and the tabbed interfaces.">
 <!ENTITY ui.network.progress_meter "GUI: Network Activity Meter">

Modified: branches/rel_1_6/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
===================================================================
--- branches/rel_1_6/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2009-10-27 02:39:59 UTC (rev 14613)
+++ branches/rel_1_6/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2009-10-27 02:57:45 UTC (rev 14614)
@@ -142,6 +142,16 @@
                     desc : '&ui.circ.patron_summary.horizontal.desc;',
                     type : 'bool'
                 },
+                'ui.circ.in_house_use.entry_cap' : {
+                    label : '&ui.circ.in_house_use.entry_cap;',
+                    desc : '&ui.circ.in_house_use.entry_cap.desc;',
+                    type : 'integer'
+                },
+                'ui.circ.in_house_use.entry_warn' : {
+                    label : '&ui.circ.in_house_use.entry_warn;',
+                    desc : '&ui.circ.in_house_use.entry_warn.desc;',
+                    type : 'integer'
+                },
                 'ui.general.button_bar' : {
                     label : '&ui.general.button_bar;',
                     desc : '&ui.general.button_bar.desc;',

Modified: branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.js
===================================================================
--- branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.js	2009-10-27 02:39:59 UTC (rev 14613)
+++ branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.js	2009-10-27 02:57:45 UTC (rev 14614)
@@ -9,6 +9,7 @@
 	JSAN.use('util.barcode');
 	JSAN.use('util.date');
 	JSAN.use('OpenILS.data'); this.data = new OpenILS.data(); this.data.init({'via':'stash'});
+	JSAN.use('util.sound'); this.sound = new util.sound();
 }
 
 circ.in_house_use.prototype = {
@@ -17,6 +18,9 @@
 
 		var obj = this;
 
+		obj.entry_cap = Number( obj.data.hash.aous['ui.circ.in_house_use.entry_cap'] ) || 99; 
+		obj.entry_warn = Number( obj.data.hash.aous['ui.circ.in_house_use.entry_warn'] ) || 20; 
+
 		JSAN.use('circ.util');
 		var columns = circ.util.columns( 
 			{ 
@@ -135,13 +139,19 @@
 								try {
 									var value = Number(ev.target.value);
 									if (value > 0) {
-										if (value > 99) ev.target.value = 99;
+										if (value > obj.entry_cap) { throw(value); }
 									} else {
-										ev.target.value = 1;
+										throw(value);
 									}
 								} catch(E) {
 									dump('in_house_use:multiplier: ' + E + '\n');
-									ev.target.value = 1;
+									obj.sound.circ_bad();
+									setTimeout(
+										function() {
+											obj.controller.view.in_house_use_multiplier_textbox.focus();
+											obj.controller.view.in_house_use_multiplier_textbox.select();
+										}, 0
+									);
 								}
 							}
 						}
@@ -230,13 +240,13 @@
 				return; 
 			}
 
-			if (multiplier == 0 || multiplier > 99) {
+			if (multiplier == 0 || multiplier > obj.entry_cap) {
 				obj.controller.view.in_house_use_multiplier_textbox.focus();
 				obj.controller.view.in_house_use_multiplier_textbox.select();
 				return;
 			}
 
-			if (multiplier > 20) {
+			if (multiplier > obj.entry_warn) {
 				var r = obj.error.yns_alert(
 					document.getElementById('circStrings').getFormattedString('staff.circ.in_house_use.confirm_multiple', [barcode, multiplier]),
 					document.getElementById('circStrings').getString('staff.circ.in_house_use.confirm_multiple.title'),
@@ -308,6 +318,7 @@
 				//I could override map_row_to_column here
 				}
 			);
+			obj.sound.circ_good();
 
 			if (typeof obj.on_in_house_use == 'function') {
 				obj.on_in_house_use(result);

Modified: branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.xul
===================================================================
--- branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.xul	2009-10-27 02:39:59 UTC (rev 14613)
+++ branches/rel_1_6/Open-ILS/xul/staff_client/server/circ/in_house_use.xul	2009-10-27 02:57:45 UTC (rev 14614)
@@ -92,7 +92,7 @@
 
 	<hbox id="in_house_use_top_ui">
 		<label id="in_house_use_multiplier_label" value="&staff.circ.in_house_use.multiplier.label;" control="in_house_use_multiplier_textbox" accesskey="&staff.circ.in_house_use.multiplier.accesskey;" hidden="false"/>
-		<textbox id="in_house_use_multiplier_textbox" value="1" hidden="false" size="2" cols="2" context="clipboard"/>
+		<textbox id="in_house_use_multiplier_textbox" value="1" hidden="false" size="6" cols="6" context="clipboard"/>
 		<hbox id="in_house_use_menu_placeholder" flex="0"/>
 		<textbox id="in_house_use_barcode_entry_textbox" context="clipboard"/>
 		<button id="in_house_use_submit_barcode_button" 



More information about the open-ils-commits mailing list