[open-ils-commits] r14616 - in branches/rel_1_6_0/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:58:44 EDT 2009


Author: phasefx
Date: 2009-10-26 22:58:41 -0400 (Mon, 26 Oct 2009)
New Revision: 14616

Modified:
   branches/rel_1_6_0/Open-ILS/web/opac/locale/en-US/lang.dtd
   branches/rel_1_6_0/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
   branches/rel_1_6_0/Open-ILS/xul/staff_client/server/circ/in_house_use.js
   branches/rel_1_6_0/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_0/Open-ILS/web/opac/locale/en-US/lang.dtd
===================================================================
--- branches/rel_1_6_0/Open-ILS/web/opac/locale/en-US/lang.dtd	2009-10-27 02:58:06 UTC (rev 14615)
+++ branches/rel_1_6_0/Open-ILS/web/opac/locale/en-US/lang.dtd	2009-10-27 02:58:41 UTC (rev 14616)
@@ -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_0/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
===================================================================
--- branches/rel_1_6_0/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2009-10-27 02:58:06 UTC (rev 14615)
+++ branches/rel_1_6_0/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2009-10-27 02:58:41 UTC (rev 14616)
@@ -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_0/Open-ILS/xul/staff_client/server/circ/in_house_use.js
===================================================================
--- branches/rel_1_6_0/Open-ILS/xul/staff_client/server/circ/in_house_use.js	2009-10-27 02:58:06 UTC (rev 14615)
+++ branches/rel_1_6_0/Open-ILS/xul/staff_client/server/circ/in_house_use.js	2009-10-27 02:58:41 UTC (rev 14616)
@@ -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_0/Open-ILS/xul/staff_client/server/circ/in_house_use.xul
===================================================================
--- branches/rel_1_6_0/Open-ILS/xul/staff_client/server/circ/in_house_use.xul	2009-10-27 02:58:06 UTC (rev 14615)
+++ branches/rel_1_6_0/Open-ILS/xul/staff_client/server/circ/in_house_use.xul	2009-10-27 02:58:41 UTC (rev 14616)
@@ -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