#
# This file is part of Koha.
#
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
#
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
=head1 pay.pl
=cut
-use strict;
-use warnings;
+use Modern::Perl;
+use URI::Escape;
use C4::Context;
use C4::Auth;
use C4::Output;
-use CGI;
+use CGI qw ( -utf8 );
use C4::Members;
use C4::Accounts;
use C4::Stats;
use C4::Koha;
use C4::Overdues;
-use C4::Branch;
-use C4::Members::Attributes qw(GetBorrowerAttributes);
+use Koha::Patrons;
+use Koha::Items;
+
+use Koha::Patron::Categories;
+use URI::Escape;
our $input = CGI->new;
-our $writeoff_sth;
-our $add_writeoff_sth;
+my $updatecharges_permissions = $input->param('woall') ? 'writeoff' : 'remaining_permissions';
our ( $template, $loggedinuser, $cookie ) = get_template_and_user(
- { template_name => 'members/pay.tmpl',
+ { template_name => 'members/pay.tt',
query => $input,
type => 'intranet',
authnotrequired => 0,
- flagsrequired => { borrowers => 1, updatecharges => 1 },
+ flagsrequired => { borrowers => 'edit_borrowers', updatecharges => $updatecharges_permissions },
debug => 1,
}
);
$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
-our $borrower = GetMember( borrowernumber => $borrowernumber );
+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 $user = $input->remote_user;
$user ||= q{};
-my $branches = GetBranches();
-our $branch = GetBranch( $input, $branches );
+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 $itemno = $input->param('itemnumber');
- my $account_type = $input->param('accounttype');
- my $amount = $input->param('amountoutstanding');
- WriteOffFee( $borrowernumber, $accountlines_id, $itemno, $account_type, $amount );
+ 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
+ . "&debit_type_code=" . $accountline->debit_type_code
+ . "&accountlines_id=" . $accountlines_id
+ . "&change_given=" . $change_given
+ . "&writeoff_individual=1"
+ . "&error_over=1" );
+
+ } else {
+ $payment_id = Koha::Account->new( { patron_id => $borrowernumber } )->pay(
+ {
+ amount => $amount,
+ lines => [ Koha::Account::Lines->find($accountlines_id) ],
+ type => 'WRITEOFF',
+ note => $payment_note,
+ interface => C4::Context->interface,
+ library_id => $branch,
+ }
+ );
+ }
}
for (@names) {
}
}
-$template->param( activeBorrowerRelationship => (C4::Context->preference('borrowerRelationship') ne '') );
+# 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,
+ payment_id => $payment_id,
+ change_given => $change_given,
+ renew_results => $renew_results_display
+);
add_accounts_to_template();
output_html_with_http_headers $input, $cookie, $template->output;
-sub writeoff {
- my ( $accountlines_id, $itemnum, $accounttype, $amount ) = @_;
- my $manager_id = 0;
- $manager_id = C4::Context->userenv->{'number'} if C4::Context->userenv;
-
- # if no item is attached to fine, make sure to store it as a NULL
- $itemnum ||= undef;
- get_writeoff_sth();
- $writeoff_sth->execute( $accountlines_id );
-
- my $acct = getnextacctno($borrowernumber);
- $add_writeoff_sth->execute( $borrowernumber, $acct, $itemnum, $amount, $manager_id );
-
- UpdateStats( $branch, 'writeoff', $amount, q{}, q{}, q{}, $borrowernumber );
-
- return;
-}
-
sub add_accounts_to_template {
- my ( $total, undef, undef ) = GetMemberAccountRecords($borrowernumber);
- my $accounts = [];
- my @notify = NumberNotifyId($borrowernumber);
-
- my $notify_groups = [];
- for my $notify_id (@notify) {
- my ( $acct_total, $accountlines, undef ) =
- GetBorNotifyAcctRecord( $borrowernumber, $notify_id );
- if ( @{$accountlines} ) {
- my $totalnotify = AmountNotify( $notify_id, $borrowernumber );
- push @{$accounts},
- { accountlines => $accountlines,
- notify => $notify_id,
- total => $totalnotify,
- };
- }
+ my $patron = Koha::Patrons->find( $borrowernumber );
+ 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 ) {
+ push @accounts, $account_line;
}
- borrower_add_additional_fields($borrower);
+
$template->param(
- accounts => $accounts,
- borrower => $borrower,
+ patron => $patron,
+ accounts => \@accounts,
total => $total,
+ outstanding_credits => $outstanding_credits
);
+
return;
}
sub get_for_redirect {
my ( $name, $name_in, $money ) = @_;
my $s = q{&} . $name . q{=};
- my $value = $input->param($name_in);
+ my $value;
+ if (defined $input->param($name_in)) {
+ $value = uri_escape_utf8( scalar $input->param($name_in) );
+ }
if ( !defined $value ) {
$value = ( $money == 1 ) ? 0 : q{};
}
"/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( 'accountno', "accountno$line_no", 0 );
- $redirect .= get_for_redirect( 'description', "description$line_no", 0 );
- $redirect .= get_for_redirect( 'title', "title$line_no", 0 );
+ $redirect .= get_for_redirect( 'description', "description$line_no", 0 );
+ $redirect .= get_for_redirect( 'title', "title$line_no", 0 );
$redirect .= get_for_redirect( 'itemnumber', "itemnumber$line_no", 0 );
- $redirect .= get_for_redirect( 'notify_id', "notify_id$line_no", 0 );
- $redirect .= get_for_redirect( 'notify_level', "notify_level$line_no", 0 );
$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);
}
sub writeoff_all {
my @params = @_;
- my @wo_lines = grep { /^accountno\d+$/ } @params;
+ my @wo_lines = grep { /^accountlines_id\d+$/ } @params;
+
+ my $borrowernumber = $input->param('borrowernumber');
+
for (@wo_lines) {
if (/(\d+)/) {
- my $value = $1;
- my $accounttype = $input->param("accounttype$value");
-
- # my $borrowernum = $input->param("borrowernumber$value");
- my $itemno = $input->param("itemnumber$value");
- my $amount = $input->param("amountoutstanding$value");
- my $accountno = $input->param("accountno$value");
+ my $value = $1;
+ my $amount = $input->param("amountoutstanding$value");
my $accountlines_id = $input->param("accountlines_id$value");
- WriteOffFee( $borrowernumber, $accountlines_id, $itemno, $accounttype, $amount );
+ my $payment_note = $input->param("payment_note_$value");
+ Koha::Account->new( { patron_id => $borrowernumber } )->pay(
+ {
+ amount => $amount,
+ lines => [ Koha::Account::Lines->find($accountlines_id) ],
+ type => 'WRITEOFF',
+ note => $payment_note,
+ interface => C4::Context->interface,
+ library_id => $branch,
+ }
+ );
}
}
- $borrowernumber = $input->param('borrowernumber');
- print $input->redirect(
- "/cgi-bin/koha/members/boraccount.pl?borrowernumber=$borrowernumber");
+ print $input->redirect("/cgi-bin/koha/members/boraccount.pl?borrowernumber=$borrowernumber");
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 ( $catcodes, $labels ) =
- GetborCatFromCatType( 'A', 'WHERE category_type = ?' );
- if ( @{$catcodes} ) {
- if ( @{$catcodes} > 1 ) {
- $b_ref->{CATCODE_MULTI} = 1;
- } elsif ( @{$catcodes} == 1 ) {
- $b_ref->{catcode} = $catcodes->[0];
- }
- }
- } elsif ( $b_ref->{category_type} eq 'A' ) {
- $b_ref->{adultborrower} = 1;
- }
- my ( $picture, $dberror ) = GetPatronImage( $b_ref->{cardnumber} );
- if ($picture) {
- $b_ref->{has_picture} = 1;
- }
-
- if (C4::Context->preference('ExtendedPatronAttributes')) {
- $b_ref->{extendedattributes} = GetBorrowerAttributes($borrowernumber);
- $template->param(
- ExtendedPatronAttributes => 1,
- );
- }
+sub payselected {
+ my $parameters = shift;
- $b_ref->{branchname} = GetBranchName( $b_ref->{branchcode} );
- return;
-}
+ my @params = @{ $parameters->{params} };
+ my $type = $parameters->{type} || 'PAYMENT';
-sub payselected {
- my @params = @_;
my $amt = 0;
my @lines_to_pay;
foreach (@params) {
if (/^incl_par_(\d+)$/) {
my $index = $1;
- push @lines_to_pay, $input->param("accountno$index");
+ push @lines_to_pay, scalar $input->param("accountlines_id$index");
$amt += $input->param("amountoutstanding$index");
}
}
$amt = '&amt=' . $amt;
my $sel = '&selected=' . join ',', @lines_to_pay;
+ 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;
+ . $sel
+ . $notes;
print $input->redirect($redirect);
return;
}
-sub get_writeoff_sth {
+sub apply_credits {
+ my ($args) = @_;
- # lets prepare these statement handles only once
- if ($writeoff_sth) {
- return;
- } else {
- my $dbh = C4::Context->dbh;
-
- # Do we need to validate accounttype
- my $sql = 'Update accountlines set amountoutstanding=0 '
- . 'WHERE accountlines_id=?';
- $writeoff_sth = $dbh->prepare($sql);
- my $insert =
-q{insert into accountlines (borrowernumber,accountno,itemnumber,date,amount,description,accounttype,manager_id)}
- . q{values (?,?,?,now(),?,'Writeoff','W',?)};
- $add_writeoff_sth = $dbh->prepare($insert);
- }
+ 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;
}