Attachment 20851 Details for Bug 10027 – [PASSED QA] Bug 10027 - Fine details not showing when printing a fine

[patch] [PASSED QA] Bug 10027 - Fine details not showing when printing a fine

PASSED-QA-Bug-10027---Fine-details-not-showing-whe.patch (text/plain), 1.47 KB, created by Katrin Fischer on 2013-09-07 18:09:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2013-09-07 18:09:04 UTC

Size: 1.47 KB

patch

obsolete

>From c677a6922f7dd2390ca4e38bcc76f6d83b519394 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 10 Apr 2013 13:45:18 -0400
>Subject: [PATCH] [PASSED QA] Bug 10027 - Fine details not showing when
> printing a fine
>
>Previously printing a fine invoice would result in a page with both the
>single item charge and the total outstanding dues. Right now only the
>total outstanding is printed.
>
>Test Plan:
>1) Print a fine, note the fine doesn't show, only the total
>2) Apply patch
>3) Print a fine, not the fine does show
>
>Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Comments on third patch.
>---
> members/printinvoice.pl |    3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/members/printinvoice.pl b/members/printinvoice.pl
>index 3e04fcc..9e69fb7 100755
>--- a/members/printinvoice.pl
>+++ b/members/printinvoice.pl
>@@ -47,7 +47,6 @@ my ($template, $loggedinuser, $cookie)
> 
> my $borrowernumber=$input->param('borrowernumber');
> my $action = $input->param('action') || '';
>-my $accountno = $input->param('accountno');
> my $accountlines_id = $input->param('accountlines_id');
> 
> #get borrower details
>@@ -74,7 +73,7 @@ my @accountrows; # this is for the tmpl-loop
> 
> my $toggle;
> for (my $i=0;$i<$numaccts;$i++){
>-   next if ($accts->[$i]{'accountno'} ne $accountno);
>+   next if ($accts->[$i]{'accountlines_id'} ne $accountlines_id );
>     if($i%2){
>             $toggle = 0;
>     } else {
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 10027: 17348 | 17349 | 17354 | 17355 | 20812 | 20851 | 20852 | 20853