use C4::Stats;
use C4::Koha;
use C4::Overdues;
-use C4::Members::Attributes qw(GetBorrowerAttributes);
use Koha::Patrons;
+use Koha::Items;
use Koha::Patron::Categories;
use URI::Escape;
$borrowernumber = $input->param('borrowernumber0');
}
+my $payment_id = $input->param('payment_id');
+our $change_given = $input->param('change_given');
+our @renew_results = $input->multi_param('renew_result');
+
# get borrower details
-my $logged_in_user = Koha::Patrons->find( $loggedinuser ) or die "Not logged in";
+my $logged_in_user = Koha::Patrons->find( $loggedinuser );
our $patron = Koha::Patrons->find($borrowernumber);
output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
our $branch = C4::Context->userenv->{'branch'};
-my $writeoff_item = $input->param('confirm_writeoff');
-my $paycollect = $input->param('paycollect');
-if ($paycollect) {
+if ( $input->param('paycollect') ) {
print $input->redirect(
- "/cgi-bin/koha/members/paycollect.pl?borrowernumber=$borrowernumber");
+ "/cgi-bin/koha/members/paycollect.pl?borrowernumber=$borrowernumber&change_given=$change_given");
}
-my $payselected = $input->param('payselected');
-if ($payselected) {
- payselected(@names);
+elsif ( $input->param('payselected') ) {
+ payselected({ params => \@names });
}
-
-my $writeoff_all = $input->param('woall'); # writeoff all fines
-if ($writeoff_all) {
+elsif ( $input->param('writeoff_selected') ) {
+ payselected({ params => \@names, type => 'WRITEOFF' });
+}
+elsif ( $input->param('woall') ) {
writeoff_all(@names);
-} elsif ($writeoff_item) {
+}
+elsif ( $input->param('apply_credits') ) {
+ apply_credits({ patron => $patron, cgi => $input });
+}
+elsif ( $input->param('confirm_writeoff') ) {
my $accountlines_id = $input->param('accountlines_id');
- my $amount = $input->param('amountwrittenoff');
- my $payment_note = $input->param("payment_note");
+ my $amount = $input->param('amountwrittenoff');
+ my $payment_note = $input->param("payment_note");
my $accountline = Koha::Account::Lines->find( $accountlines_id );
+ $amount = $accountline->amountoutstanding if (abs($amount - $accountline->amountoutstanding) < 0.01) && C4::Context->preference('RoundFinesAtPayment');
if ( $amount > $accountline->amountoutstanding ) {
print $input->redirect( "/cgi-bin/koha/members/paycollect.pl?"
. "borrowernumber=$borrowernumber"
. "&amount=" . $accountline->amount
. "&amountoutstanding=" . $accountline->amountoutstanding
- . "&accounttype=" . $accountline->accounttype
+ . "&debit_type_code=" . $accountline->debit_type_code
. "&accountlines_id=" . $accountlines_id
+ . "&change_given=" . $change_given
. "&writeoff_individual=1"
. "&error_over=1" );
} else {
- Koha::Account->new( { patron_id => $borrowernumber } )->pay(
+ $payment_id = Koha::Account->new( { patron_id => $borrowernumber } )->pay(
{
amount => $amount,
- lines => [ scalar Koha::Account::Lines->find($accountlines_id) ],
- type => 'writeoff',
+ lines => [ Koha::Account::Lines->find($accountlines_id) ],
+ type => 'WRITEOFF',
note => $payment_note,
+ interface => C4::Context->interface,
library_id => $branch,
}
);
}
}
+# Populate an arrayref with everything we need to display any
+# renew results that occurred based on what we were passed
+my $renew_results_display = [];
+foreach my $renew_result(@renew_results) {
+ my ($itemnumber, $success, $info) = split(/,/, $renew_result);
+ my $item = Koha::Items->find($itemnumber);
+ if ($success) {
+ $info = uri_unescape($info);
+ }
+ push @{$renew_results_display}, {
+ item => $item,
+ success => $success,
+ info => $info
+ };
+}
+
$template->param(
- finesview => 1,
+ finesview => 1,
+ payment_id => $payment_id,
+ change_given => $change_given,
+ renew_results => $renew_results_display
);
add_accounts_to_template();
sub add_accounts_to_template {
my $patron = Koha::Patrons->find( $borrowernumber );
- my $total = $patron->account->balance;
- my $account_lines = Koha::Account::Lines->search({ borrowernumber => $borrowernumber, amountoutstanding => { '!=' => 0 } }, { order_by => ['accounttype'] });
+ my $account = $patron->account;
+ my $outstanding_credits = $account->outstanding_credits;
+ my $account_lines = $account->outstanding_debits;
+ my $total = $account_lines->total_outstanding;
my @accounts;
while ( my $account_line = $account_lines->next ) {
- $account_line = $account_line->unblessed;
- if ( $account_line->{itemnumber} ) {
- my $item = Koha::Items->find( $account_line->{itemnumber} );
- my $biblio = $item->biblio;
- $account_line->{biblionumber} = $biblio->biblionumber;
- $account_line->{title} = $biblio->title;
- }
push @accounts, $account_line;
}
- borrower_add_additional_fields($patron->unblessed);
$template->param(
patron => $patron,
accounts => \@accounts,
total => $total,
+ outstanding_credits => $outstanding_credits
);
+
return;
}
"/cgi-bin/koha/members/paycollect.pl?borrowernumber=$borrowernumber";
$redirect .= q{&};
$redirect .= "$action=1";
- $redirect .= get_for_redirect( 'accounttype', "accounttype$line_no", 0 );
+ $redirect .= get_for_redirect( 'debit_type_code', "debit_type_code$line_no", 0 );
$redirect .= get_for_redirect( 'amount', "amount$line_no", 1 );
$redirect .=
get_for_redirect( 'amountoutstanding', "amountoutstanding$line_no", 1 );
$redirect .= get_for_redirect( 'accountlines_id', "accountlines_id$line_no", 0 );
$redirect .= q{&} . 'payment_note' . q{=} . uri_escape_utf8( scalar $input->param("payment_note_$line_no") );
$redirect .= '&remote_user=';
+ $redirect .= "change_given=$change_given";
$redirect .= $user;
return print $input->redirect($redirect);
}
Koha::Account->new( { patron_id => $borrowernumber } )->pay(
{
amount => $amount,
- lines => [ scalar Koha::Account::Lines->find($accountlines_id) ],
- type => 'writeoff',
+ lines => [ Koha::Account::Lines->find($accountlines_id) ],
+ type => 'WRITEOFF',
note => $payment_note,
+ interface => C4::Context->interface,
library_id => $branch,
}
);
return;
}
-sub borrower_add_additional_fields {
- my $b_ref = shift;
-
-# some borrower info is not returned in the standard call despite being assumed
-# in a number of templates. It should not be the business of this script but in lieu of
-# a revised api here it is ...
- if ( $b_ref->{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;
- }
-
- if (C4::Context->preference('ExtendedPatronAttributes')) {
- $b_ref->{extendedattributes} = GetBorrowerAttributes($borrowernumber);
- $template->param(
- ExtendedPatronAttributes => 1,
- );
- }
+sub payselected {
+ my $parameters = shift;
- return;
-}
+ my @params = @{ $parameters->{params} };
+ my $type = $parameters->{type} || 'PAYMENT';
-sub payselected {
- my @params = @_;
my $amt = 0;
my @lines_to_pay;
foreach (@params) {
my $notes = '¬es=' . join("%0A", map { scalar $input->param("payment_note_$_") } @lines_to_pay );
my $redirect =
"/cgi-bin/koha/members/paycollect.pl?borrowernumber=$borrowernumber"
+ . "&type=$type"
. $amt
. $sel
. $notes;
print $input->redirect($redirect);
return;
}
+
+sub apply_credits {
+ my ($args) = @_;
+
+ my $patron = $args->{patron};
+ my $cgi = $args->{cgi};
+
+ $patron->account->reconcile_balance();
+
+ print $cgi->redirect("/cgi-bin/koha/members/pay.pl?borrowernumber=" . $patron->borrowernumber );
+ return;
+}