use Modern::Perl;
use CGI qw ( -utf8 );
+
+use Try::Tiny qw( catch try );
+
use C4::Context;
-use C4::Output;
-use C4::Auth;
-use C4::Members;
+use C4::Output qw( output_and_exit_if_error output_and_exit output_html_with_http_headers );
+use C4::Auth qw( get_template_and_user );
use Koha::Patrons;
use Koha::Token;
use Koha::Patron::Categories;
+use Koha::Suggestions;
-my $input = new CGI;
+my $input = CGI->new;
-my ($template, $loggedinuser, $cookie)
- = get_template_and_user({template_name => "members/deletemem.tt",
- query => $input,
- type => "intranet",
- authnotrequired => 0,
- flagsrequired => {borrowers => 'edit_borrowers'},
- debug => 1,
- });
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+ { template_name => "members/deletemem.tt",
+ query => $input,
+ type => "intranet",
+ flagsrequired => { borrowers => 'delete_borrowers' },
+ }
+);
#print $input->header;
my $member = $input->param('member');
exit 0; # Exit without error
}
-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( $member );
output_and_exit_if_error( $input, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
-my $charges = $patron->account->non_issues_charges;
+my $debits = $patron->account->outstanding_debits->total_outstanding;
+my $credits = abs $patron->account->outstanding_credits->total_outstanding;
my $countissues = $patron->checkouts->count;
my $userenv = C4::Context->userenv;
exit 0; # Exit without error
}
} else {
- unless(C4::Auth::haspermission($userenv->{'id'},{'borrowers'=>'edit_borrowers'})) {
+ unless(C4::Auth::haspermission($userenv->{'id'},{'borrowers'=>'delete_borrowers'})) {
print $input->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$member&error=CANT_DELETE");
exit 0; # Exit without error
}
}
}
+if ( my $anonymous_patron = C4::Context->preference("AnonymousPatron") ) {
+ if ( $patron->id eq $anonymous_patron ) {
+ print $input->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$member&error=CANT_DELETE_ANONYMOUS_PATRON");
+ exit 0; # Exit without error
+ }
+}
+
my $op = $input->param('op') || 'delete_confirm';
my $dbh = C4::Context->dbh;
my $is_guarantor = $patron->guarantee_relationships->count;
my $countholds = $dbh->selectrow_array("SELECT COUNT(*) FROM reserves WHERE borrowernumber=?", undef, $member);
-if ( $op eq 'delete_confirm' or $countissues > 0 or $charges or $is_guarantor ) {
+# Add warning if patron has pending suggestions
+$template->param(
+ pending_suggestions => Koha::Suggestions->search({ suggestedby => $member, STATUS => 'ASKED' })->count,
+);
+
+$template->param(
+ patron => $patron,
+ ItemsOnIssues => $countissues,
+ debits => $debits,
+ credits => $credits,
+ is_guarantor => $is_guarantor,
+ ItemsOnHold => $countholds,
+);
+
+if ( $op eq 'delete_confirm' or $countissues > 0 or $debits or $is_guarantor ) {
$template->param(
- patron => $patron,
+ op => 'delete_confirm',
+ csrf_token => Koha::Token->new->generate_csrf({ session_id => scalar $input->cookie('CGISESSID') }),
);
- if ($countissues >0) {
- $template->param(ItemsOnIssues => $countissues);
- }
- if ( $charges > 0 ) {
- $template->param(charges => $charges);
- }
- if ( $is_guarantor ) {
- $template->param( guarantees => 1 );
- }
- if($countholds > 0){
- $template->param(ItemsOnHold => $countholds);
- }
- # This is silly written but reflect the same conditions as above
- if ( not $countissues > 0 and not $charges and not $is_guarantor ) {
- $template->param(
- op => 'delete_confirm',
- csrf_token => Koha::Token->new->generate_csrf({ session_id => scalar $input->cookie('CGISESSID') }),
- );
- }
-} elsif ( $op eq 'delete_confirmed' ) {
+} elsif ( $op eq 'delete_confirmed' ) {
output_and_exit( $input, $cookie, $template, 'wrong_csrf_token' )
unless Koha::Token->new->check_csrf( {
session_id => $input->cookie('CGISESSID'),
token => scalar $input->param('csrf_token'),
});
+
my $patron = Koha::Patrons->find( $member );
- $patron->move_to_deleted;
- $patron->delete;
+
+ try {
+ my $schema = Koha::Database->new->schema;
+ $schema->txn_do(
+ sub {
+ $patron->move_to_deleted;
+ $patron->delete;
+ print $input->redirect( "/cgi-bin/koha/members/members-home.pl" );
+ }
+ );
+ }
+ catch {
+ if ( $_->isa('Koha::Exceptions::Patron::FailedDeleteAnonymousPatron') ) {
+ print $input->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$member&error=CANT_DELETE_ANONYMOUS_PATRON");
+ }
+ else {
+ $_->rethrow;
+ }
+ };
# TODO Tell the user everything went ok
- print $input->redirect("/cgi-bin/koha/members/members-home.pl");
exit 0; # Exit without error
}