From c13e8e527536e3882fdab5d2c68c5db33b8591fd Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 12 Feb 2013 10:54:23 -0500 Subject: [PATCH] Bug 8079 - opac not showing fines - says 'no' Make charges of type FU be counted for opac-user.pl Test Plan: 1) Checkout an item to a patron, back date the date due enough to create fines 2) Run fines.pl 3) Log into the patron's account via the OPAC 4) Note the fines line for that issue says 'No' 5) Apply patch 6) Reload opac-user.pl, not the fines column now says 'Yes' Signed-off-by: David Cook Works great, Kyle. Seeing as my test patron had $4500 in fines for that item, I was glad to see that column switch to "Yes" ;). I'm not sure what the difference is between FU and F is, or what L means, but the patch works, so I'm signing off. Signed-off-by: Katrin Fischer This works as described, all tests and QA script are ok. Note: why show yes and not the fine amount? Signed-off-by: Jared Camins-Esakov --- opac/opac-user.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/opac/opac-user.pl b/opac/opac-user.pl index c5a7f5afe6..f4273cbece 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -169,6 +169,8 @@ if ($issues){ if ( $ac->{'itemnumber'} == $issue->{'itemnumber'} ) { $charges += $ac->{'amountoutstanding'} if $ac->{'accounttype'} eq 'F'; + $charges += $ac->{'amountoutstanding'} + if $ac->{'accounttype'} eq 'FU'; $charges += $ac->{'amountoutstanding'} if $ac->{'accounttype'} eq 'L'; } -- 2.39.5