From 6794478e574375013b626c73fe794b3bf7bc805f Mon Sep 17 00:00:00 2001 From: berick Date: Mon, 7 Mar 2011 13:55:11 -0500 Subject: [PATCH] don't show or allow payments to negative-balance transactions --- Open-ILS/web/opac/skin/kcls/js/myopac.js | 25 +++++++++++++++++++++++++ 1 files changed, 25 insertions(+), 0 deletions(-) diff --git a/Open-ILS/web/opac/skin/kcls/js/myopac.js b/Open-ILS/web/opac/skin/kcls/js/myopac.js index d753877..59a7e29 100644 --- a/Open-ILS/web/opac/skin/kcls/js/myopac.js +++ b/Open-ILS/web/opac/skin/kcls/js/myopac.js @@ -1395,6 +1395,7 @@ function myOPACShowTransactions(r) { var trans = transactions[idx].transaction; var record = transactions[idx].record; var circ = transactions[idx].circ; + if(trans.balance_owed() <= 0) continue; // XXX don't show negative-balance transactions for now transCache[trans.id()] = transactions[idx]; if(trans.xact_type() == 'circulation') myOPACShowCircTransaction(trans, record, circ); @@ -2328,3 +2329,27 @@ function buildOrgSelAlt(selector, org, offset, namecol) { buildOrgSelAlt( selector, kids[c], offset, namecol); } } + +// alternative to checkAll that does not select <= balances... needs testing +function checkAllXact(parent, id, name, balanceName) {//Object, string + var obj = typeof(id)=="object"?id:$(id); + if(!parent || !obj) return; + if(!name) name = id.toString(); + dojo.forEach(parent.childNodes, + function(row) { + if(row.nodeName.match(/tr/i)) { + var input = dojo.query('input[name='+name+']', row)[0]; + var balance = dojo.query('input[name='+balanceName+']', row)[0]; + if(Number(balance) > 0) + balance.checked = obj.checked; + } + } + ); + /* + var nodes = dojo.query('input[name='+name+']', parent); + if(!nodes.length) nodes = dojo.query('.'+name, parent); + nodes.forEach(function(node, index){ + node.checked = obj.checked; + }); + */ +} -- 1.7.2.5