Bug 32437: Honor overlay setting in staged batch when adding to a basket
[srvgit] / circ / circulation.pl
index 81c3bcd..61065c7 100755 (executable)
 
 use Modern::Perl;
 use CGI qw ( -utf8 );
+use URI::Escape qw( uri_escape_utf8 );
 use DateTime;
 use DateTime::Duration;
 use Scalar::Util qw( looks_like_number );
-use C4::Output;
-use C4::Auth qw/:DEFAULT get_session haspermission/;
+use C4::Output qw( output_and_exit_if_error output_and_exit output_html_with_http_headers );
+use C4::Auth qw( get_session get_template_and_user );
 use C4::Koha;
-use C4::Circulation;
-use C4::Utils::DataTables::Members;
+use C4::Circulation qw( barcodedecode CanBookBeIssued AddIssue );
 use C4::Members;
-use C4::Biblio;
-use C4::Search;
-use MARC::Record;
+use C4::Biblio qw( TransformMarcToKoha );
+use C4::Search qw( new_record_from_zebra );
 use C4::Reserves;
 use Koha::Holds;
 use C4::Context;
@@ -45,27 +44,23 @@ use CGI::Session;
 use Koha::AuthorisedValues;
 use Koha::CsvProfiles;
 use Koha::Patrons;
-use Koha::Patron::Debarments qw(GetDebarments);
-use Koha::DateUtils;
+use Koha::DateUtils qw( dt_from_string );
+use Koha::Patron::Restriction::Types;
+use Koha::Plugins;
 use Koha::Database;
 use Koha::BiblioFrameworks;
 use Koha::Items;
-use Koha::Patron::Messages;
 use Koha::SearchEngine;
 use Koha::SearchEngine::Search;
 use Koha::Patron::Modifications;
+use Koha::Token;
 
-use Date::Calc qw(
-  Today
-  Add_Delta_Days
-  Date_to_Days
-);
-use List::MoreUtils qw/uniq/;
+use List::MoreUtils qw( uniq );
 
 #
 # PARAMETERS READING
 #
-my $query = new CGI;
+my $query = CGI->new;
 
 my $override_high_holds     = $query->param('override_high_holds');
 my $override_high_holds_tmp = $query->param('override_high_holds_tmp');
@@ -80,7 +75,9 @@ my $autoswitched;
 my $borrowernumber = $query->param('borrowernumber');
 
 if (C4::Context->preference("AutoSwitchPatron") && $barcode) {
-    if (Koha::Patrons->search( { cardnumber => $barcode} )->count() > 0) {
+    my $new_barcode = $barcode;
+    Koha::Plugins->call( 'patron_barcode_transform', \$new_barcode );
+    if (Koha::Patrons->search( { cardnumber => $new_barcode} )->count() > 0) {
         $findborrower = $barcode;
         undef $barcode;
         undef $borrowernumber;
@@ -116,7 +113,7 @@ my $batch = $query->param('batch');
 my $batch_allowed = 0;
 if ( $batch && C4::Context->preference('BatchCheckouts') ) {
     $template_name = q|circ/circulation_batch_checkouts.tt|;
-    my @batch_category_codes = split '\|', C4::Context->preference('BatchCheckoutsValidCategories');
+    my @batch_category_codes = split ',', C4::Context->preference('BatchCheckoutsValidCategories');
     my $categorycode = $patron->categorycode;
     if ( $categorycode && grep { $_ eq $categorycode } @batch_category_codes ) {
         $batch_allowed = 1;
@@ -130,7 +127,6 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user (
         template_name   => $template_name,
         query           => $query,
         type            => "intranet",
-        authnotrequired => 0,
         flagsrequired   => { circulate => 'circulate_remaining_permissions' },
     }
 );
@@ -158,26 +154,20 @@ my $searchtype = $query->param('searchtype') || q{contain};
 
 my $branch = C4::Context->userenv->{'branch'};
 
-if (C4::Context->preference("DisplayClearScreenButton")) {
-    $template->param(DisplayClearScreenButton => 1);
-}
-
 for my $barcode ( @$barcodes ) {
-    $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace
-    $barcode = barcodedecode($barcode)
-        if( $barcode && C4::Context->preference('itemBarcodeInputFilter'));
+    $barcode = barcodedecode( $barcode ) if $barcode;
 }
 
 my $stickyduedate  = $query->param('stickyduedate') || $session->param('stickyduedate');
 my $duedatespec    = $query->param('duedatespec')   || $session->param('stickyduedate');
-$duedatespec = eval { output_pref( { dt => dt_from_string( $duedatespec ), dateformat => 'iso' }); }
-    if ( $duedatespec );
 my $restoreduedatespec  = $query->param('restoreduedatespec') || $duedatespec || $session->param('stickyduedate');
 if ( $restoreduedatespec && $restoreduedatespec eq "highholds_empty" ) {
     undef $restoreduedatespec;
 }
 my $issueconfirmed = $query->param('issueconfirmed');
 my $cancelreserve  = $query->param('cancelreserve');
+my $cancel_recall  = $query->param('cancel_recall');
+my $recall_id      = $query->param('recall_id');
 my $debt_confirmed = $query->param('debt_confirmed') || 0; # Don't show the debt error dialog twice
 my $charges        = $query->param('charges') || q{};
 
@@ -201,8 +191,9 @@ my ($datedue,$invalidduedate);
 
 my $duedatespec_allow = C4::Context->preference('SpecifyDueDate');
 if( $onsite_checkout && !$duedatespec_allow ) {
-    $datedue = output_pref({ dt => dt_from_string, dateonly => 1, dateformat => 'iso' });
-    $datedue .= ' 23:59:00';
+    $datedue = dt_from_string()->truncate(to => 'day');
+    $datedue->set_hour(23);
+    $datedue->set_minute(59);
 } elsif( $duedatespec_allow ) {
     if ( $duedatespec ) {
         $datedue = eval { dt_from_string( $duedatespec ) };
@@ -227,29 +218,13 @@ if ( @$barcodes == 0 && $charges eq 'yes' ) {
 #
 my $message;
 if ($findborrower) {
+    Koha::Plugins->call( 'patron_barcode_transform', \$findborrower );
     my $patron = Koha::Patrons->find( { cardnumber => $findborrower } );
     if ( $patron ) {
         $borrowernumber = $patron->borrowernumber;
     } else {
-        my $dt_params = { iDisplayLength => -1 };
-        my $results = C4::Utils::DataTables::Members::search(
-            {
-                searchmember => $findborrower,
-                searchtype   => $searchtype,
-                dt_params    => $dt_params,
-            }
-        );
-        my $borrowers = $results->{patrons};
-        if ( scalar @$borrowers == 1 ) {
-            $borrowernumber = $borrowers->[0]->{borrowernumber};
-            $query->param( 'borrowernumber', $borrowernumber );
-            $query->param( 'barcode',           '' );
-        } elsif ( @$borrowers ) {
-            $template->param( borrowers => $borrowers );
-        } else {
-            $query->param( 'findborrower', '' );
-            $message = "'$findborrower'";
-        }
+        print $query->redirect( "/cgi-bin/koha/members/member.pl?quicksearch=1&circsearch=1&searchmember=" . uri_escape_utf8($findborrower) );
+        exit;
     }
 }
 
@@ -261,7 +236,7 @@ if ($patron) {
     $template->param( borrowernumber => $patron->borrowernumber );
     output_and_exit_if_error( $query, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
 
-    my $overdues = $patron->get_overdues;
+    my $overdues = $patron->overdues;
     my $issues = $patron->checkouts;
     $balance = $patron->account->balance;
 
@@ -294,6 +269,7 @@ if ($patron) {
         $template->param(
             'userdebarred'    => $patron->debarred,
             'debarredcomment' => $patron->debarredcomment,
+            'debarredsince'   => $patron->restrictions->search()->single->created,
         );
 
         if ( $patron->debarred ne "9999-12-31" ) {
@@ -362,33 +338,36 @@ if (@$barcodes) {
 
         # if multiple hits, offer options to librarian
         if ( $total_hits > 0 ) {
-            my @options = ();
+            my @barcodes;
             foreach my $hit ( @{$results} ) {
-                my $chosen =
-                  TransformMarcToKoha( C4::Search::new_record_from_zebra('biblioserver',$hit) );
+                my $chosen = # Maybe easier to retrieve the itemnumber from $hit?
+                  TransformMarcToKoha({ record => C4::Search::new_record_from_zebra('biblioserver',$hit) });
 
                 # offer all barcodes individually
                 if ( $chosen->{barcode} ) {
-                    foreach my $barcode ( sort split(/\s*\|\s*/, $chosen->{barcode}) ) {
-                        my %chosen_single = %{$chosen};
-                        $chosen_single{barcode} = $barcode;
-                        push( @options, \%chosen_single );
-                    }
+                    push @barcodes, sort split(/\s*\|\s*/, $chosen->{barcode});
                 }
             }
-            $template_params->{options} = \@options;
+            my $items = Koha::Items->search({ barcode => {-in => \@barcodes}});
+            $template_params->{options} = $items;
         }
     }
 
-    if ( $error->{UNKNOWN_BARCODE} or not $onsite_checkout or not C4::Context->preference("OnSiteCheckoutsForce") ) {
-        delete $question->{'DEBT'} if ($debt_confirmed);
-        foreach my $impossible ( keys %$error ) {
-            $template_params->{$impossible} = $$error{$impossible};
+    # Only some errors will block when performing forced onsite checkout,
+    # for other cases all errors will block
+    my @blocking_error_codes = ($onsite_checkout and C4::Context->preference("OnSiteCheckoutsForce")) ?
+        qw( UNKNOWN_BARCODE ) : (keys %$error);
+
+    foreach my $code ( @blocking_error_codes ) {
+        if ($error->{$code}) {
+            $template_params->{$code} = $error->{$code};
             $template_params->{IMPOSSIBLE} = 1;
             $blocker = 1;
         }
     }
 
+    delete $question->{'DEBT'} if ($debt_confirmed);
+
     if( $item and ( !$blocker or $force_allow_issue ) ){
         my $confirm_required = 0;
         unless($issueconfirmed){
@@ -396,7 +375,7 @@ if (@$barcodes) {
             my $materials = $item->materials;
             my $descriptions = Koha::AuthorisedValues->get_description_by_koha_field({ frameworkcode => $biblio->frameworkcode, kohafield => 'items.materials', authorised_value => $materials });
             $materials = $descriptions->{lib} // $materials;
-            $template_params->{additional_materials} = $materials;
+            $template_params->{ADDITIONAL_MATERIALS} = $materials;
             $template_params->{itemhomebranch} = $item->homebranch;
 
             # pass needsconfirmation to template if issuing is possible and user hasn't yet confirmed.
@@ -410,16 +389,29 @@ if (@$barcodes) {
         }
         unless($confirm_required) {
             my $switch_onsite_checkout = exists $messages->{ONSITE_CHECKOUT_WILL_BE_SWITCHED};
-            my $issue = AddIssue( $patron->unblessed, $barcode, $datedue, $cancelreserve, undef, undef, { onsite_checkout => $onsite_checkout, auto_renew => $session->param('auto_renew'), switch_onsite_checkout => $switch_onsite_checkout, } );
+            if ( C4::Context->preference('UseRecalls') && !$recall_id ) {
+                my $recall = Koha::Recalls->find(
+                    {
+                        biblio_id => $item->biblionumber,
+                        item_id   => [ undef, $item->itemnumber ],
+                        status    => [ 'requested', 'waiting' ],
+                        completed => 0,
+                        patron_id => $patron->borrowernumber,
+                    }
+                );
+                $recall_id = ( $recall and $recall->id ) ? $recall->id : undef;
+            }
+            my $issue = AddIssue( $patron->unblessed, $barcode, $datedue, $cancelreserve, undef, undef, { onsite_checkout => $onsite_checkout, auto_renew => $session->param('auto_renew'), switch_onsite_checkout => $switch_onsite_checkout, cancel_recall => $cancel_recall, recall_id => $recall_id, } );
             $template_params->{issue} = $issue;
             $session->clear('auto_renew');
             $inprocess = 1;
         }
     }
 
-    if ($question->{RESERVE_WAITING} or $question->{RESERVED}){
+    if ($question->{RESERVE_WAITING} or $question->{RESERVED} or $question->{TRANSFERRED} or $question->{PROCESSING}){
         $template->param(
-            reserveborrowernumber => $question->{'resborrowernumber'}
+            reserveborrowernumber => $question->{'resborrowernumber'},
+            reserve_id => $question->{reserve_id},
         );
     }
 
@@ -457,12 +449,21 @@ if ($patron) {
         holds_count  => $holds->count(),
         WaitingHolds => $waiting_holds,
     );
+
+    if ( C4::Context->preference('UseRecalls') ) {
+        my $waiting_recalls = $patron->recalls->search({ status => 'waiting' });
+        $template->param(
+            recalls => $patron->recalls->filter_by_current->search({},{ order_by => { -asc => 'created_date' } }),
+            specific_patron => 1,
+            waiting_recalls => $waiting_recalls,
+        );
+    }
 }
 
 if ( $patron ) {
     my $noissues;
     if ( $patron->gonenoaddress ) {
-        $template->param( gna => 1 );
+        $template->param( gonenoaddress => 1 );
         $noissues = 1;
     }
     if ( $patron->lost ) {
@@ -470,7 +471,7 @@ if ( $patron ) {
         $noissues = 1;
     }
     if ( $patron->is_debarred ) {
-        $template->param( dbarred=> 1 );
+        $template->param( is_debarred=> 1 );
         $noissues = 1;
     }
     my $account = $patron->account;
@@ -488,6 +489,19 @@ if ( $patron ) {
         );
     }
 
+    # Check the debt of this patrons guarantors *and* the guarantees of those guarantors
+    my $no_issues_charge_guarantors = C4::Context->preference("NoIssuesChargeGuarantorsWithGuarantees");
+    if ( $no_issues_charge_guarantors ) {
+        my $guarantors_non_issues_charges = $patron->relationships_debt({ include_guarantors => 1, only_this_guarantor => 0, include_this_patron => 1 });
+
+        if ( $guarantors_non_issues_charges > $no_issues_charge_guarantors ) {
+            $template->param(
+                charges_guarantors_guarantees => $guarantors_non_issues_charges
+            );
+            $noissues = 1 unless C4::Context->preference("allowfineoverride");
+        }
+    }
+
     my $no_issues_charge_guarantees = C4::Context->preference("NoIssuesChargeGuarantees");
     $no_issues_charge_guarantees = undef unless looks_like_number( $no_issues_charge_guarantees );
     if ( defined $no_issues_charge_guarantees ) {
@@ -524,18 +538,18 @@ if ( $patron ) {
             forceallow => $force_allow_issue,
         );
     }
-}
 
-my $messages = Koha::Patron::Messages->search(
-    {
-        'me.borrowernumber' => $borrowernumber,
-    },
-    {
-       join => 'manager',
-       '+select' => ['manager.surname', 'manager.firstname' ],
-       '+as' => ['manager_surname', 'manager_firstname'],
-    }
-);
+    my $patron_messages = $patron->messages->search(
+        {},
+        {
+           join => 'manager',
+           '+select' => ['manager.surname', 'manager.firstname' ],
+           '+as' => ['manager_surname', 'manager_firstname'],
+        }
+    );
+    $template->param( patron_messages => $patron_messages );
+
+}
 
 my $fast_cataloging = 0;
 if ( Koha::BiblioFrameworks->find('FA') ) {
@@ -548,11 +562,11 @@ my $view = $batch
 
 my @relatives;
 if ( $patron ) {
-    if ( my @guarantors = $patron->guarantor_relationships()->guarantors() ) {
+    if ( my @guarantors = $patron->guarantor_relationships()->guarantors->as_list ) {
         push( @relatives, $_->id ) for @guarantors;
-        push( @relatives, $_->id ) for $patron->siblings();
+        push( @relatives, $_->id ) for $patron->siblings->as_list;
     } else {
-        push( @relatives, $_->id ) for $patron->guarantee_relationships()->guarantees();
+        push( @relatives, $_->id ) for $patron->guarantee_relationships()->guarantees->as_list;
     }
 }
 my $relatives_issues_count =
@@ -582,7 +596,6 @@ if ($restoreduedatespec || $stickyduedate) {
 }
 
 $template->param(
-    messages           => $messages,
     borrowernumber    => $borrowernumber,
     branch            => $branch,
     was_renewed       => scalar $query->param('was_renewed') ? 1 : 0,
@@ -606,16 +619,16 @@ $template->param(
 
 
 if ( C4::Context->preference("ExportCircHistory") ) {
-    $template->param(csv_profiles => [ Koha::CsvProfiles->search({ type => 'marc' }) ]);
+    $template->param(csv_profiles => Koha::CsvProfiles->search({ type => 'marc' }));
 }
 
 my $has_modifications = Koha::Patron::Modifications->search( { borrowernumber => $borrowernumber } )->count;
 $template->param(
     debt_confirmed            => $debt_confirmed,
     SpecifyDueDate            => $duedatespec_allow,
-    PatronAutoComplete      => C4::Context->preference("PatronAutoComplete"),
-    debarments                => scalar GetDebarments({ borrowernumber => $borrowernumber }),
-    todaysdate                => output_pref( { dt => dt_from_string()->set(hour => 23)->set(minute => 59), dateformat => 'sql' } ),
+    PatronAutoComplete        => C4::Context->preference("PatronAutoComplete"),
+    todaysdate                => dt_from_string()->set(hour => 23)->set(minute => 59),
+    restriction_types         => scalar Koha::Patron::Restriction::Types->search(),
     has_modifications         => $has_modifications,
     override_high_holds       => $override_high_holds,
     nopermission              => scalar $query->param('nopermission'),
@@ -623,4 +636,9 @@ $template->param(
     logged_in_user            => $logged_in_user,
 );
 
+# Generate CSRF token for upload and delete image buttons
+$template->param(
+    csrf_token => Koha::Token->new->generate_csrf({ session_id => $query->cookie('CGISESSID'),}),
+);
+
 output_html_with_http_headers $query, $cookie, $template->output;