From 301d01decfed626f82357cba46ecf8a9d9e01b6e Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Thu, 5 Jan 2017 02:01:57 +0000 Subject: [PATCH] [SIGNED-OFF] Bug 15738: Show rental fees on OPAC summary page This patch adds a few lines that check for a rental fee on an item. If yes, it will show in brackets as a rental fee on the OPAC summary page. To test: 1) Have a borrower with an overdue item accruing fines, a lost item and an item with a rental fee. Confirm the Fines column on the OPAC summary page now shows you what you may expect to see for each item. Sponsored-by: Catalyst IT Signed-off-by: Josef Moravec --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 4 +++- opac/opac-user.pl | 4 ++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt index e724b18..e8b6d96 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt @@ -281,7 +281,9 @@ Using this account is not recommended because some parts of Koha will not functi Fines: [% IF ( ISSUE.charges ) %] - Yes + Yes (Item overdue or lost) + [% ELSIF ( ISSUE.rentalfines ) %] + Yes (Rental fees) [% ELSE %] No [% END %] diff --git a/opac/opac-user.pl b/opac/opac-user.pl index 3dc39bf..4035d02 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -188,6 +188,7 @@ if ($issues){ my ( $total , $accts, $numaccts) = GetMemberAccountRecords( $borrowernumber ); my $charges = 0; + my $rentalfines = 0; foreach my $ac (@$accts) { if ( $ac->{'itemnumber'} == $issue->{'itemnumber'} ) { $charges += $ac->{'amountoutstanding'} @@ -196,9 +197,12 @@ if ($issues){ if $ac->{'accounttype'} eq 'FU'; $charges += $ac->{'amountoutstanding'} if $ac->{'accounttype'} eq 'L'; + $rentalfines += $ac->{'amountoutstanding'} + if $ac->{'accounttype'} eq 'Rent'; } } $issue->{'charges'} = $charges; + $issue->{'rentalfines'} = $rentalfines; my $marcrecord = GetMarcBiblio( $issue->{'biblionumber'} ); $issue->{'subtitle'} = GetRecordValue('subtitle', $marcrecord, GetFrameworkCode($issue->{'biblionumber'})); # check if item is renewable -- 2.1.4