don't show or allow payments to negative-balance transactions

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 (file)
--- 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;
+       });
+    */
+}