From: phasefx <phasefx@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Tue, 30 Nov 2010 18:17:47 +0000 (+0000)
Subject: add click event to red-text label in check-in UI for retrieving a patron when the...
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=752554984dfdda0cf2572d0f9a0c11ebb43da664;p=evergreen%2Fmasslnc.git
add click event to red-text label in check-in UI for retrieving a patron when the associated transaction has a balance owed. Code tweak to reduce the number of getElementById lookups
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18877 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
diff --git a/Open-ILS/xul/staff_client/server/circ/checkin.js b/Open-ILS/xul/staff_client/server/circ/checkin.js
index c52d66f9c2..bac58aef90 100644
--- a/Open-ILS/xul/staff_client/server/circ/checkin.js
+++ b/Open-ILS/xul/staff_client/server/circ/checkin.js
@@ -408,16 +408,18 @@ circ.checkin.prototype = {
var row = params.row;
if (typeof params.on_retrieve == 'function') params.on_retrieve(row);
- if (row.my.mbts && ( document.getElementById('no_change_label') || document.getElementById('fine_tally') ) ) {
+ var no_change_label = document.getElementById('no_change_label');
+ if (row.my.mbts && ( no_change_label || document.getElementById('fine_tally') ) ) {
var bill = row.my.mbts;
if (Number(bill.balance_owed()) == 0) { return; }
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
- document.getElementById('no_change_label').setAttribute(
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
+ no_change_label.setAttribute(
'value',
m + document.getElementById('circStrings').getFormattedString('staff.circ.utils.billable.amount', [row.my.acp.barcode(), util.money.sanitize(bill.balance_owed())]) + ' '
);
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','xulG.new_patron_tab({},{"id" : '+bill.usr()+' })');
}
if (document.getElementById('fine_tally')) {
var amount = Number( document.getElementById('fine_tally').getAttribute('amount') ) + Number( bill.balance_owed() );
diff --git a/Open-ILS/xul/staff_client/server/circ/util.js b/Open-ILS/xul/staff_client/server/circ/util.js
index 2405182e31..47fea736e1 100644
--- a/Open-ILS/xul/staff_client/server/circ/util.js
+++ b/Open-ILS/xul/staff_client/server/circ/util.js
@@ -2404,9 +2404,12 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
if (!check.route_to) { check.route_to = ' '; }
- if (document.getElementById('no_change_label')) {
- document.getElementById('no_change_label').setAttribute('value','');
- document.getElementById('no_change_label').setAttribute('hidden','true');
+ var no_change_label = document.getElementById('no_change_label');
+
+ if (no_change_label) {
+ no_change_label.setAttribute('value','');
+ no_change_label.setAttribute('hidden','true');
+ no_change_label.setAttribute('onclick','');
}
var msg = '';
@@ -2471,11 +2474,12 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
if (check.ilsevent == 3 /* NO_CHANGE */) {
//msg = 'This item is already checked in.\n';
check.what_happened = 'no_change';
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
var text = document.getElementById('circStrings').getFormattedString('staff.circ.utils.item_checked_in', [params.barcode]);
- document.getElementById('no_change_label').setAttribute('value', m + text + ' ');
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('value', m + text + ' ');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( text );
}
@@ -2708,12 +2712,13 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
}
}
msg = '';
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
var text = document.getElementById('circStrings').getFormattedString('staff.circ.utils.capture', [params.barcode]);
m += text + ' ';
- document.getElementById('no_change_label').setAttribute('value', m);
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('value', m);
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( text );
}
@@ -2744,11 +2749,12 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
sound.circ_bad();
}
}
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
var needs_cat = document.getElementById('circStrings').getFormattedString('staff.circ.utils.needs_cataloging', [params.barcode]);
- document.getElementById('no_change_label').setAttribute('value', m + needs_cat + ' ');
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('value', m + needs_cat + ' ');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( needs_cat );
}
@@ -2828,12 +2834,13 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
}
}
msg = '';
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
var text = document.getElementById('circStrings').getFormattedString('staff.circ.utils.reservation_capture', [params.barcode]);
m += text + ' ';
- document.getElementById('no_change_label').setAttribute('value', m);
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('value', m);
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( text );
}
@@ -3088,11 +3095,12 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
alert(err_msg);
}
}
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
var trans_msg = document.getElementById('circStrings').getFormattedString('staff.circ.utils.payload.in_transit', [params.barcode]);
- document.getElementById('no_change_label').setAttribute('value', m + trans_msg + ' ');
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ no_change_label.setAttribute('value', m + trans_msg + ' ');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( trans_msg );
}
@@ -3117,10 +3125,11 @@ circ.util.checkin_via_barcode2 = function(session,params,backdate,auto_print,che
// FIXME: add SFX and/or GFX
sound.circ_bad();
}
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
- document.getElementById('no_change_label').setAttribute('value',m + mis_scan_msg + ' ');
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
+ no_change_label.setAttribute('value',m + mis_scan_msg + ' ');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( mis_scan_msg );
}
@@ -3217,10 +3226,11 @@ circ.util.renew_via_barcode = function ( params, async ) {
null,
document.getElementById('circStrings').getString('staff.circ.confirm.msg')
);
- if (document.getElementById('no_change_label')) {
- var m = document.getElementById('no_change_label').getAttribute('value');
- document.getElementById('no_change_label').setAttribute('value',m + mis_scan_msg + ' ');
- document.getElementById('no_change_label').setAttribute('hidden','false');
+ if (no_change_label) {
+ var m = no_change_label.getAttribute('value');
+ no_change_label.setAttribute('value',m + mis_scan_msg + ' ');
+ no_change_label.setAttribute('hidden','false');
+ no_change_label.setAttribute('onclick','');
if (typeof params.info_blurb == 'function') {
params.info_blurb( mis_scan_msg );
}