From f80299b11483c8c46fedf9a2a24b419be966482d Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 12 Feb 2013 10:54:23 -0500 Subject: [PATCH] [PASSED QA] 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? --- opac/opac-user.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/opac/opac-user.pl b/opac/opac-user.pl index c5a7f5a..f4273cb 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -170,6 +170,8 @@ if ($issues){ $charges += $ac->{'amountoutstanding'} if $ac->{'accounttype'} eq 'F'; $charges += $ac->{'amountoutstanding'} + if $ac->{'accounttype'} eq 'FU'; + $charges += $ac->{'amountoutstanding'} if $ac->{'accounttype'} eq 'L'; } } -- 1.7.9.5