From 83c8f60a41008e8b1e79d1d6851eb68d46aeb47e Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Tue, 27 May 2014 14:51:06 -0400 Subject: [PATCH] refunds cont. add billing to xact Signed-off-by: Bill Erickson --- Open-ILS/src/templates/staff/circ/patron/index.tt2 | 2 + .../staff/circ/patron/t_bill_patron_dialog.tt2 | 32 ++++++ .../templates/staff/circ/patron/t_bills_list.tt2 | 6 +- .../web/js/ui/default/staff/circ/patron/bills.js | 126 ++++++++++++++------- 4 files changed, 123 insertions(+), 43 deletions(-) diff --git a/Open-ILS/src/templates/staff/circ/patron/index.tt2 b/Open-ILS/src/templates/staff/circ/patron/index.tt2 index 12c07e3339..05b3ef3076 100644 --- a/Open-ILS/src/templates/staff/circ/patron/index.tt2 +++ b/Open-ILS/src/templates/staff/circ/patron/index.tt2 @@ -28,6 +28,8 @@ diff --git a/Open-ILS/src/templates/staff/circ/patron/t_bill_patron_dialog.tt2 b/Open-ILS/src/templates/staff/circ/patron/t_bill_patron_dialog.tt2 index ab1cd05dce..4f849710c5 100644 --- a/Open-ILS/src/templates/staff/circ/patron/t_bill_patron_dialog.tt2 +++ b/Open-ILS/src/templates/staff/circ/patron/t_bill_patron_dialog.tt2 @@ -11,6 +11,38 @@ '{{patron.second_given_name()}}', '{{patron.card().barcode()}}') %] + +
+
+
+
[% l('Bill #') %]
+
{{xact.id}}
+
[% l('Total Billed') %]
+
{{xact.total_owed | currency}}
+
+
+
[% l('Type') %]
+
{{xact.xact_type}}
+
[% l('Total Paid') %]
+
{{xact.total_paid | currency}}
+
+
+
[% l('Start') %]
+
{{xact.xact_start | date:'short'}}
+
[% l('Total Billed') %]
+
{{xact.balance_owed | currency}}
+
+
+
[% l('Finish') %]
+
{{xact.xact_finish | date:'short'}}
+
[% l('Renewal?') %]
+
+ [% l('Desk') %] + [% l('Phone') %] + [% l('OPAC') %] +
+
+
diff --git a/Open-ILS/src/templates/staff/circ/patron/t_bills_list.tt2 b/Open-ILS/src/templates/staff/circ/patron/t_bills_list.tt2 index 4373b01123..5827591046 100644 --- a/Open-ILS/src/templates/staff/circ/patron/t_bills_list.tt2 +++ b/Open-ILS/src/templates/staff/circ/patron/t_bills_list.tt2 @@ -38,13 +38,13 @@ handler="voidAllBillings"> + handler="refundXact"> + handler="addBilling"> + handler="showFillDetails"> diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js b/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js index d0a3fcf96b..4286563235 100644 --- a/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js +++ b/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js @@ -56,7 +56,7 @@ function($q , egCore , patronSvc) { }); } - service.billPatron = function(args) { + service.createGroceryXact = function(args) { var groc = new egCore.idl.mg(); groc.billing_location(egCore.auth.user().ws_ou()); groc.note(args.note); @@ -72,25 +72,42 @@ function($q , egCore , patronSvc) { ).then(function(xact_id) { if (evt = egCore.evt.parse(xact_id)) return alert(evt); + return xact_id; + }); + } - var bill = new egCore.idl.mb(); - bill.xact(xact_id); - bill.amount(args.amount); - bill.btype(args.billingType); - bill.billing_type(egCore.env.cbt.map[args.billingType].name()); - bill.note(args.note); - - return egCore.net.request( - 'open-ils.circ', - 'open-ils.circ.money.billing.create', - egCore.auth.token(), bill - ); + service.createBilling = function(xact_id, args) { + var bill = new egCore.idl.mb(); + bill.xact(xact_id); + bill.amount(args.amount); + bill.btype(args.billingType); + bill.billing_type(egCore.env.cbt.map[args.billingType].name()); + bill.note(args.note); + + return egCore.net.request( + 'open-ils.circ', + 'open-ils.circ.money.billing.create', + egCore.auth.token(), bill // check the billing response - }).then(function(bill_id) { - if (evt = egCore.evt.parse(bill_id)) - return alert(evt); + ).then(function(bill_id) { + if (evt = egCore.evt.parse(bill_id)) { + alert(evt); + } else { + return bill_id; + } + }); + } + + service.billPatron = function(args, xact) { + // apply a billing to an existing transaction + if (xact) return service.createBilling(xact.id, args); + + // create a new grocery xact, then apply a billing + return service.createGroceryXact(args) + .then(function(xact_id) { + return service.createBilling(xact_id, args); }); } @@ -102,8 +119,10 @@ function($q , egCore , patronSvc) { * Manages Bills */ .controller('PatronBillsCtrl', - ['$scope','$q','$routeParams','$locale','egCore','egGridDataProvider','billSvc','patronSvc','egPromptDialog','$modal', -function($scope, $q , $routeParams, $locale , egCore , egGridDataProvider , billSvc , patronSvc , egPromptDialog , $modal) { + ['$scope','$q','$routeParams','$locale','egCore','egConfirmDialog', + 'egGridDataProvider','billSvc','patronSvc','egPromptDialog','$modal', +function($scope, $q , $routeParams, $locale , egCore , egConfirmDialog , + egGridDataProvider , billSvc , patronSvc , egPromptDialog , $modal) { $scope.initTab('bills', $routeParams.id); billSvc.userId = $routeParams.id; @@ -207,41 +226,36 @@ function($scope, $q , $routeParams, $locale , egCore , egGridDataProvider , bi // reset all to zero.. angular.forEach(allItems, function(item) {item.payment_pending = 0}); - // then apply the current payment amounts - var payments = generatePayments(); - angular.forEach(payments, function(payment) { - var item = selectedItems.filter(function(item) { - return (item.id == payment[0]) - })[0]; - item.payment_pending = payment[1]; - }); - } - - // builds payment arrays ([xact_id, ammount]) for all transactions - // which have a pending payment amount. - function generatePayments() { - var payments = []; - var paymentAmount = $scope.pending_payment(); + var payment_amount = $scope.pending_payment(); for (var i = 0; i < selectedItems.length; i++) { // for/break var item = selectedItems[i]; var owed = Number(item.balance_owed); - if (paymentAmount > owed) { + if (payment_amount > owed) { // pending payment exceeds balance of current item. // pay the entire item. - payments.push([item.id, owed]); - paymentAmount -= owed; + item.payment_pending = owed; + payment_amount -= owed; } else { // balance owed on the current item matches or exceeds // the pending payment. Apply the full remainder of // the payment to this item.. and we're done. - payments.push([item.id, paymentAmount]); + item.payment_pending = payment_amount; break; } } + } + // builds payment arrays ([xact_id, ammount]) for all transactions + // which have a pending payment amount. + function generatePayments() { + var payments = []; + angular.forEach(selectedItems, function(item) { + if (item.payment_pending == 0) return; + payments.push([item.id, item.payment_pending]); + }); return payments; } @@ -261,14 +275,25 @@ function($scope, $q , $routeParams, $locale , egCore , egGridDataProvider , bi billSvc.fetchSummary().then(function(s) {$scope.summary = s}); }) } + + // For now, only adds billing to first selected item. + // Could do batches later if needed + $scope.addBilling = function(all) { + if (all[0]) showBillDialog(all[0]); + } + + $scope.showBillDialog = function($event) { + showBillDialog(); + } - $scope.showBillDialog = function() { - $modal.open({ + function showBillDialog(xact) { + return $modal.open({ templateUrl: './circ/patron/t_bill_patron_dialog', controller: ['$scope','$modalInstance','billingTypes', function($scope , $modalInstance , billingTypes) { $scope.focus = true; + $scope.xact = xact; $scope.patron = patronSvc.current; $scope.billingTypes = billingTypes; $scope.location = egCore.org.get(egCore.auth.user().ws_ou()), @@ -288,7 +313,7 @@ function($scope, $q , $routeParams, $locale , egCore , egGridDataProvider , bi function(args) { // send the billing to the server using the arguments // provided in the billing dialog, then refresh - billSvc.billPatron(args).then(refreshDisplay); + billSvc.billPatron(args, xact).then(refreshDisplay); } ); } @@ -351,6 +376,27 @@ function($scope, $q , $routeParams, $locale , egCore , egGridDataProvider , bi }); }); } + + // note this is functionally equivalent to selecting a neg. transaction + // then clicking Apply Payment -- this just adds a speed bump (ditto + // the XUL client). + $scope.refundXact = function(all) { + var items = all.filter(function(item) {return item.balance_owed < 0}); + if (items.length == 0) return; + + var ids = items.map(function(item) {return item.id}); + + egConfirmDialog.open( + egCore.strings.CONFIRM_REFUND_PAYMENT, '', + { xactIds : ''+ids, + ok : function() { + // reset the received payment amount. this ensures + // we're not mingling payments with refunds. + $scope.payment_amount = 0; + } + } + ); + } }]) -- 2.11.0