Bug 23051: (follow-up) Add renewal feedback and move code to subroutines and test

[koha-ffzg.git] / members / printinvoice.pl

diff --git a/members/printinvoice.pl b/members/printinvoice.pl
index 552e0cd..38950e3 100755 (executable)
--- a/members/printinvoice.pl
+++ b/members/printinvoice.pl
@@ -29,6 +29,7 @@ use CGI qw ( -utf8 );
 use C4::Members;
 use C4::Accounts;
 
+use Koha::Account::Lines;
 use Koha::Patrons;
 use Koha::Patron::Categories;
 
@@ -48,78 +49,46 @@ my $borrowernumber  = $input->param('borrowernumber');
 my $action          = $input->param('action') || '';
 my $accountlines_id = $input->param('accountlines_id');
 
-my $logged_in_user = Koha::Patrons->find( $loggedinuser ) or die "Not logged in";
+my $logged_in_user = Koha::Patrons->find( $loggedinuser );
 my $patron         = Koha::Patrons->find( $borrowernumber );
 output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
 
-my $category = $patron->category;
-
-if ( $category->category_type eq 'C' ) {
-    my $patron_categories = Koha::Patron::Categories->search_limited({ category_type => 'A' }, {order_by => ['categorycode']});
-    $template->param( 'CATCODE_MULTI' => 1) if $patron_categories->count > 1;
-    $template->param( 'catcode' => $patron_categories->next->categorycode )  if $patron_categories->count == 1;
-}
-
 #get account details
-my ( $total, $accts, $numaccts ) = GetMemberAccountRecords($borrowernumber);
+my $total = $patron->account->balance;
+my $accountline_object = Koha::Account::Lines->find($accountlines_id);
+my $accountline = $accountline_object->unblessed;
+
 my $totalcredit;
 if ( $total <= 0 ) {
     $totalcredit = 1;
 }
 
-my @accountrows;    # this is for the tmpl-loop
-
-my $toggle;
-for ( my $i = 0 ; $i < $numaccts ; $i++ ) {
-    next if ( $accts->[$i]{'accountlines_id'} ne $accountlines_id );
-
-    if ( $i % 2 ) {
-        $toggle = 0;
-    } else {
-        $toggle = 1;
-    }
-
-    $accts->[$i]{'toggle'} = $toggle;
-    $accts->[$i]{'amount'} += 0.00;
 
-    if ( $accts->[$i]{'amount'} <= 0 ) {
-        $accts->[$i]{'amountcredit'} = 1;
-    }
-
-    $accts->[$i]{'amountoutstanding'} += 0.00;
-    if ( $accts->[$i]{'amountoutstanding'} <= 0 ) {
-        $accts->[$i]{'amountoutstandingcredit'} = 1;
-    }
+$accountline->{'amount'} += 0.00;
+if ( $accountline->{'amount'} <= 0 ) {
+    $accountline->{'amountcredit'} = 1;
+    $accountline->{'amount'} *= -1.00;
+}
+$accountline->{'amountoutstanding'} += 0.00;
+if ( $accountline->{'amountoutstanding'} <= 0 ) {
+    $accountline->{'amountoutstandingcredit'} = 1;
+}
 
-    my %row = (
-        'date'                    => output_pref({ dt => dt_from_string( $accts->[$i]{'date'} ), dateonly => 1 }),
-        'amountcredit'            => $accts->[$i]{'amountcredit'},
-        'amountoutstandingcredit' => $accts->[$i]{'amountoutstandingcredit'},
-        'toggle'                  => $accts->[$i]{'toggle'},
-        'description'             => $accts->[$i]{'description'},
-        'itemnumber'              => $accts->[$i]{'itemnumber'},
-        'biblionumber'            => $accts->[$i]{'biblionumber'},
-        'amount'                  => sprintf( "%.2f", $accts->[$i]{'amount'} ),
-        'amountoutstanding'       => sprintf( "%.2f", $accts->[$i]{'amountoutstanding'} ),
-        'accountno'               => $accts->[$i]{'accountno'},
-        accounttype               => $accts->[$i]{accounttype},
-        'note'                    => $accts->[$i]{'note'},
-    );
-
-    if ( $accts->[$i]{'accounttype'} ne 'F' && $accts->[$i]{'accounttype'} ne 'FU' ) {
-        $row{'printtitle'} = 1;
-        $row{'title'}      = $accts->[$i]{'title'};
-    }
+my @account_offsets = Koha::Account::Offsets->search( { debit_id => $accountline_object->id } );
 
-    push( @accountrows, \%row );
-}
+my $letter = C4::Letters::getletter( 'circulation', 'ACCOUNT_DEBIT', C4::Context::mybranch, 'print', $patron->lang );
 
 $template->param(
-    patron         => $patron,
-    finesview      => 1,
-    total          => sprintf( "%.2f", $total ),
-    totalcredit    => $totalcredit,
-    accounts       => \@accountrows
+    letter  => $letter,
+    patron  => $patron,
+    library => C4::Context::mybranch,
+    offsets => \@account_offsets,
+    debit   => $accountline_object,
+
+    finesview   => 1,
+    total       => sprintf( "%.2f", $total ),
+    totalcredit => $totalcredit,
+    accounts    => [$accountline],           # FIXME There is always only 1 row!
 );
 
 output_html_with_http_headers $input, $cookie, $template->output;