From 56bcda57b3d8a364abc883afe9e27563bef8f6cd Mon Sep 17 00:00:00 2001 From: Kyle Huckins Date: Thu, 11 Jan 2018 18:10:51 +0000 Subject: [PATCH] lp1742194 - Print Current Bills only printing circ bills - Check xact for Circulation before assigning circulation data to xact for printing. To test ------- [1] Set up a patron with at least one "grocery" bill. [2] In the web staff client, go to Patron Account > Bills > Actions > Print Bills [3] Note that the current bills summary is not printed and that there an error in the browser console. [4] Apply the patch and repeat step #2. This time, the bills summary should be printed. Signed-off-by: Kyle Huckins Signed-off-by: Terra McCanna Signed-off-by: Galen Charlton --- .../web/js/ui/default/staff/circ/patron/bills.js | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) 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 ce240f919b..46fc0b9ccb 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 @@ -498,8 +498,10 @@ function($scope , $q , $routeParams , egCore , egConfirmDialog , $location, unrecovered : xact.unrecovered(), xact_finish : xact.xact_finish(), xact_start : xact.xact_start(), - copy_barcode : xact.circulation().target_copy().barcode(), - title : xact.circulation().target_copy().call_number().record().simple_record().title() + } + if (xact.circulation()) { + newXact.copy_barcode = xact.circulation().target_copy().barcode(), + newXact.title = xact.circulation().target_copy().call_number().record().simple_record().title() } xacts.push(newXact); } @@ -963,8 +965,10 @@ function($scope, $q , egCore , patronSvc , billSvc , egPromptDialog , $location unrecovered : xact.unrecovered(), xact_finish : xact.xact_finish(), xact_start : xact.xact_start(), - copy_barcode : xact.circulation().target_copy().barcode(), - title : xact.circulation().target_copy().call_number().record().simple_record().title() + } + if (xact.circulation()) { + newXact.copy_barcode = xact.circulation().target_copy().barcode(), + newXact.title = xact.circulation().target_copy().call_number().record().simple_record().title() } xacts.push(newXact); } -- 2.43.2