Bug 25261: (QA follow-up) Prevent server error on bad barcode
[koha.git] / circ / returns.pl
1 #!/usr/bin/perl
2
3 # Copyright 2000-2002 Katipo Communications
4 #           2006 SAN-OP
5 #           2007-2010 BibLibre, Paul POULAIN
6 #           2010 Catalyst IT
7 #           2011 PTFS-Europe Ltd.
8 #
9 # This file is part of Koha.
10 #
11 # Koha is free software; you can redistribute it and/or modify it
12 # under the terms of the GNU General Public License as published by
13 # the Free Software Foundation; either version 3 of the License, or
14 # (at your option) any later version.
15 #
16 # Koha is distributed in the hope that it will be useful, but
17 # WITHOUT ANY WARRANTY; without even the implied warranty of
18 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
19 # GNU General Public License for more details.
20 #
21 # You should have received a copy of the GNU General Public License
22 # along with Koha; if not, see <http://www.gnu.org/licenses>.
23
24 =head1 returns.pl
25
26 script to execute returns of books
27
28 =cut
29
30 use Modern::Perl;
31
32 # FIXME There are weird things going on with $patron and $borrowernumber in this script
33
34 use CGI qw ( -utf8 );
35 use DateTime;
36
37 use C4::Auth qw/:DEFAULT get_session/;
38 use C4::Output;
39 use C4::Circulation;
40 use C4::Reserves;
41 use C4::Biblio;
42 use C4::Circulation;
43 use C4::Context;
44 use C4::Items;
45 use C4::Koha;   # FIXME : is it still useful ?
46 use C4::Members::Messaging;
47 use C4::Members;
48 use C4::Output;
49 use C4::Reserves;
50 use C4::RotatingCollections;
51 use Koha::AuthorisedValues;
52 use Koha::BiblioFrameworks;
53 use Koha::Calendar;
54 use Koha::Checkouts;
55 use Koha::DateUtils;
56 use Koha::Holds;
57 use Koha::Items;
58 use Koha::Patrons;
59
60 my $query = new CGI;
61
62 #getting the template
63 my ( $template, $librarian, $cookie, $flags ) = get_template_and_user(
64     {
65         template_name   => "circ/returns.tt",
66         query           => $query,
67         type            => "intranet",
68         flagsrequired   => { circulate => "circulate_remaining_permissions" },
69     }
70 );
71
72 my $sessionID = $query->cookie("CGISESSID");
73 my $session = get_session($sessionID);
74
75 # Print a reserve slip on this page
76 if ( $query->param('print_slip') ) {
77     $template->param(
78         print_slip     => 1,
79         reserve_id => scalar $query->param('reserve_id'),
80     );
81 }
82
83 #####################
84 #Global vars
85 my $userenv = C4::Context->userenv;
86 my $userenv_branch = $userenv->{'branch'} // '';
87 my $forgivemanualholdsexpire = $query->param('forgivemanualholdsexpire');
88
89 my $overduecharges = (C4::Context->preference('finesMode') && C4::Context->preference('finesMode') ne 'off');
90
91 # Set up the item stack ....
92 my %returneditems;
93 my %riduedate;
94 my %riborrowernumber;
95 my @inputloop;
96 foreach ( $query->param ) {
97     my $counter;
98     if (/ri-(\d*)/) {
99         $counter = $1;
100         if ($counter > 20) {
101             next;
102         }
103     }
104     else {
105         next;
106     }
107
108     my %input;
109     my $barcode        = $query->param("ri-$counter");
110     my $duedate        = $query->param("dd-$counter");
111     my $borrowernumber = $query->param("bn-$counter");
112     $counter++;
113
114     # decode barcode    ## Didn't we already decode them before passing them back last time??
115     $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace
116     $barcode = barcodedecode($barcode) if(C4::Context->preference('itemBarcodeInputFilter'));
117
118     ######################
119     #Are these lines still useful ?
120     $returneditems{$counter}    = $barcode;
121     $riduedate{$counter}        = $duedate;
122     $riborrowernumber{$counter} = $borrowernumber;
123
124     #######################
125     $input{counter}        = $counter;
126     $input{barcode}        = $barcode;
127     $input{duedate}        = $duedate;
128     $input{borrowernumber} = $borrowernumber;
129     push( @inputloop, \%input );
130 }
131
132 ############
133 # Deal with the requests....
134
135 if ($query->param('WT-itemNumber')){
136         updateWrongTransfer ($query->param('WT-itemNumber'),$query->param('WT-waitingAt'),$query->param('WT-From'));
137 }
138
139 if ( $query->param('reserve_id') ) {
140     my $itemnumber     = $query->param('itemnumber');
141     my $borrowernumber = $query->param('borrowernumber');
142     my $reserve_id     = $query->param('reserve_id');
143     my $diffBranchReturned = $query->param('diffBranch');
144     my $cancel_reserve = $query->param('cancel_reserve');
145     # fix up item type for display
146     my $item = Koha::Items->find( $itemnumber );
147     my $biblio = $item->biblio;
148
149     if ( $cancel_reserve ) {
150         my $hold = Koha::Holds->find( $reserve_id );
151         if ( $hold ) {
152             $hold->cancel( { charge_cancel_fee => !$forgivemanualholdsexpire } );
153         } # FIXME else?
154     } else {
155         my $diffBranchSend = ($userenv_branch ne $diffBranchReturned) ? $diffBranchReturned : undef;
156         # diffBranchSend tells ModReserveAffect whether document is expected in this library or not,
157         # i.e., whether to apply waiting status
158         ModReserveAffect( $itemnumber, $borrowernumber, $diffBranchSend, $reserve_id );
159     }
160 #   check if we have other reserves for this document, if we have a return send the message of transfer
161     my ( $messages, $nextreservinfo ) = GetOtherReserves($itemnumber);
162
163     my $patron = Koha::Patrons->find( $nextreservinfo );
164     my $name   = $patron ? $patron->surname . ", " . $patron->title . " " . $patron->firstname : '';
165     if ( $messages->{'transfert'} ) {
166         $template->param(
167             itemtitle      => $biblio->title,
168             itemnumber     => $item->itemnumber,
169             itembiblionumber => $biblio->biblionumber,
170             iteminfo       => $biblio->author,
171             name           => $name,
172             patron         => $patron,
173             diffbranch     => 1,
174         );
175     }
176 }
177
178 my $borrower;
179 my $returned = 0;
180 my $messages;
181 my $issue;
182 my $itemnumber;
183 my $barcode     = $query->param('barcode');
184 my $exemptfine  = $query->param('exemptfine');
185 if (
186   $exemptfine &&
187   !C4::Auth::haspermission(C4::Context->userenv->{'id'}, {'updatecharges' => 'writeoff'})
188 ) {
189     # silently prevent unauthorized operator from forgiving overdue
190     # fines by manually tweaking form parameters
191     undef $exemptfine;
192 }
193 my $dropboxmode = $query->param('dropboxmode');
194 my $dotransfer  = $query->param('dotransfer');
195 my $canceltransfer = $query->param('canceltransfer');
196 my $dest = $query->param('dest');
197 #dropbox: get last open day (today - 1)
198 my $dropboxdate = Koha::Checkouts::calculate_dropbox_date();
199
200 my $return_date_override = $query->param('return_date_override');
201 my $return_date_override_dt;
202 my $return_date_override_remember =
203   $query->param('return_date_override_remember');
204 if ($return_date_override) {
205     if ( C4::Context->preference('SpecifyReturnDate') ) {
206         $return_date_override_dt = eval {dt_from_string( $return_date_override ) };
207         if ( $return_date_override_dt ) {
208             # note that we've overriden the return date
209             $template->param( return_date_was_overriden => 1);
210             # Save the original format if we are remembering for this series
211             $template->param(
212                 return_date_override          => $return_date_override,
213                 return_date_override_remember => 1
214             ) if ($return_date_override_remember);
215
216             $return_date_override =
217               DateTime::Format::MySQL->format_datetime( $return_date_override_dt );
218         }
219     }
220     else {
221         $return_date_override = q{};
222     }
223 }
224
225 if ($dotransfer){
226 # An item has been returned to a branch other than the homebranch, and the librarian has chosen to initiate a transfer
227     my $transferitem = $query->param('transferitem');
228     my $tobranch     = $query->param('tobranch');
229     my $trigger      = $query->param('trigger');
230     ModItemTransfer($transferitem, $userenv_branch, $tobranch, $trigger);
231 }
232
233 if ($canceltransfer){
234     $itemnumber=$query->param('itemnumber');
235     DeleteTransfer($itemnumber);
236     if($dest eq "ttr"){
237         print $query->redirect("/cgi-bin/koha/circ/transferstoreceive.pl");
238         exit;
239     } else {
240         $template->param( transfercancelled => 1);
241     }
242 }
243
244 # actually return book and prepare item table.....
245 my $returnbranch;
246 if ($barcode) {
247     $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace
248     $barcode = barcodedecode($barcode) if C4::Context->preference('itemBarcodeInputFilter');
249     my $item = Koha::Items->find({ barcode => $barcode });
250
251     if ( $item ) {
252         $itemnumber = $item->itemnumber;
253         # Check if we should display a checkin message, based on the the item
254         # type of the checked in item
255         my $itemtype = Koha::ItemTypes->find( $item->effective_itemtype );
256         if ( $itemtype && $itemtype->checkinmsg ) {
257             $template->param(
258                 checkinmsg     => $itemtype->checkinmsg,
259                 checkinmsgtype => $itemtype->checkinmsgtype,
260             );
261         }
262
263         # make sure return branch respects home branch circulation rules, default to homebranch
264         my $hbr = GetBranchItemRule($item->homebranch, $itemtype ? $itemtype->itemtype : undef )->{'returnbranch'} || "homebranch";
265         $returnbranch = $hbr ne 'noreturn' ? $item->$hbr : $userenv_branch; # can be noreturn, homebranch or holdingbranch
266
267         my $materials = $item->materials;
268         my $descriptions = Koha::AuthorisedValues->get_description_by_koha_field({frameworkcode => '', kohafield =>'items.materials', authorised_value => $materials });
269         $materials = $descriptions->{lib} // $materials;
270
271         my $checkout = $item->checkout;
272         my $biblio   = $item->biblio;
273         $template->param(
274             title                => $biblio->title,
275             homebranch           => $item->homebranch,
276             holdingbranch        => $item->holdingbranch,
277             returnbranch         => $returnbranch,
278             author               => $biblio->author,
279             itembarcode          => $item->barcode,
280             itemtype             => $item->effective_itemtype,
281             ccode                => $item->ccode,
282             itembiblionumber     => $biblio->biblionumber,
283             biblionumber         => $biblio->biblionumber,
284             additional_materials => $materials,
285             issue                => $checkout,
286         );
287     } # FIXME else we should not call AddReturn but set BadBarcode directly instead
288
289     my %input = (
290         counter => 0,
291         first   => 1,
292         barcode => $barcode,
293     );
294
295     my $return_date = $dropboxmode ? $dropboxdate : $return_date_override_dt;
296
297     # Block return if multi-part and confirm has not been received
298     my $needs_confirm =
299          C4::Context->preference("CircConfirmItemParts")
300       && $item
301       && $item->materials
302       && !$query->param('multiple_confirm');
303     $template->param( 'multiple_confirmed' => 1 )
304       if $query->param('multiple_confirm');
305
306     # do the return
307     ( $returned, $messages, $issue, $borrower ) =
308       AddReturn( $barcode, $userenv_branch, $exemptfine, $return_date )
309           unless $needs_confirm;
310
311     if ($returned) {
312         my $time_now = dt_from_string()->truncate( to => 'minute');
313         my $date_due_dt = dt_from_string( $issue->date_due, 'sql' );
314         my $duedate = $date_due_dt->strftime('%Y-%m-%d %H:%M');
315         $returneditems{0}      = $barcode;
316         $riborrowernumber{0}   = $borrower->{'borrowernumber'};
317         $riduedate{0}          = $duedate;
318         $input{borrowernumber} = $borrower->{'borrowernumber'};
319         $input{duedate}        = $duedate;
320         unless ( $dropboxmode ) {
321             $input{return_overdue} = 1 if (DateTime->compare($date_due_dt, dt_from_string()) == -1);
322         } else {
323             $input{return_overdue} = 1 if (DateTime->compare($date_due_dt, $dropboxdate) == -1);
324         }
325         push( @inputloop, \%input );
326
327         if ( C4::Context->preference("FineNotifyAtCheckin") ) {
328             my $patron = Koha::Patrons->find( $borrower->{borrowernumber} );
329             my $balance = $patron->account->balance;
330
331             if ($balance > 0) {
332                 $template->param( fines => sprintf("%.2f", $balance) );
333                 $template->param( fineborrowernumber => $borrower->{'borrowernumber'} );
334             }
335         }
336
337         if (C4::Context->preference("WaitingNotifyAtCheckin") ) {
338             #Check for waiting holds
339             my $patron = Koha::Patrons->find( $borrower->{borrowernumber} );
340             my $waiting_holds = $patron->holds->search({ found => 'W', branchcode => $userenv_branch })->count;
341             if ($waiting_holds > 0) {
342                 $template->param(
343                     waiting_holds       => $waiting_holds,
344                     holdsborrowernumber => $borrower->{'borrowernumber'},
345                     holdsfirstname => $borrower->{'firstname'},
346                     holdssurname => $borrower->{'surname'},
347                 );
348             }
349         }
350     } elsif ( C4::Context->preference('ShowAllCheckins') and !$messages->{'BadBarcode'} and !$needs_confirm ) {
351         $input{duedate}   = 0;
352         $returneditems{0} = $barcode;
353         $riduedate{0}     = 0;
354         push( @inputloop, \%input );
355     }
356     $template->param( privacy => $borrower->{privacy} );
357
358     if ( $needs_confirm ) {
359         $template->param( needs_confirm => $needs_confirm );
360     }
361 }
362 $template->param( inputloop => \@inputloop );
363
364
365 my $found    = 0;
366 my $waiting  = 0;
367 my $reserved = 0;
368
369 # new op dev : we check if the document must be returned to his homebranch directly,
370 #  if the document is transferred, we have warning message .
371
372 if ( $messages->{'WasTransfered'} ) {
373     $template->param(
374         found          => 1,
375         transfer       => 1,
376         itemnumber     => $itemnumber,
377     );
378 }
379
380 if ( $messages->{'NeedsTransfer'} ){
381     $template->param(
382         found          => 1,
383         needstransfer  => $messages->{'NeedsTransfer'},
384         trigger        => $messages->{'TransferTrigger'},
385         itemnumber     => $itemnumber,
386     );
387 }
388
389 if ( $messages->{'Wrongbranch'} ){
390     $template->param(
391         wrongbranch => 1,
392         rightbranch => $messages->{'Wrongbranch'}->{'Rightbranch'},
393     );
394 }
395
396 # case of wrong transfert, if the document wasn't transferred to the right library (according to branchtransfer (tobranch) BDD)
397
398 if ( $messages->{'WrongTransfer'} and not $messages->{'WasTransfered'}) {
399     $template->param(
400         WrongTransfer  => 1,
401         TransferWaitingAt => $messages->{'WrongTransfer'},
402         WrongTransferItem => $messages->{'WrongTransferItem'},
403         itemnumber => $itemnumber,
404     );
405
406     my $reserve    = $messages->{'ResFound'};
407     if ( $reserve ) {
408         my $patron = Koha::Patrons->find( $reserve->{'borrowernumber'} );
409         my $name = $patron->surname . ", " . $patron->title . " " . $patron->firstname;
410         $template->param(
411             patron => $patron,
412         );
413     }
414     $template->param(
415         wtransfertFrom  => $userenv_branch,
416     );
417 }
418
419 #
420 # reserve found and item arrived at the expected branch
421 #
422 if ( $messages->{'ResFound'}) {
423     my $reserve    = $messages->{'ResFound'};
424     my $patron = Koha::Patrons->find( $reserve->{borrowernumber} );
425     my $holdmsgpreferences =  C4::Members::Messaging::GetMessagingPreferences( { borrowernumber => $reserve->{'borrowernumber'}, message_name   => 'Hold_Filled' } );
426     my $branchCheck = ( $userenv_branch eq $reserve->{branchcode} );
427     if ( $reserve->{'ResFound'} eq "Reserved" && C4::Context->preference('HoldsAutoFill') ) {
428         my $item = Koha::Items->find( $itemnumber );
429         my $biblio = $item->biblio;
430
431         my $diffBranchSend = !$branchCheck ? $reserve->{branchcode} : undef;
432         ModReserveAffect( $reserve->{itemnumber}, $reserve->{borrowernumber}, $diffBranchSend, $reserve->{reserve_id} );
433         my ( $messages, $nextreservinfo ) = GetOtherReserves($reserve->{itemnumber});
434
435         $template->param(
436             hold_auto_filled => 1,
437             print_slip       => C4::Context->preference('HoldsAutoFillPrintSlip'),
438             reserve_id       => $nextreservinfo->{reserve_id},
439             itemnumber       => $itemnumber,
440         );
441
442         if ( $messages->{'transfert'} ) {
443             $template->param(
444                 itemtitle        => $biblio->title,
445                 itembiblionumber => $biblio->biblionumber,
446                 iteminfo         => $biblio->author,
447                 diffbranch       => 1,
448             );
449         }
450     }
451     elsif ( $reserve->{'ResFound'} eq "Waiting" or $reserve->{'ResFound'} eq "Reserved" ) {
452         if ( $reserve->{'ResFound'} eq "Waiting" ) {
453             $template->param(
454                 waiting      => $branchCheck ? 1 : undef,
455             );
456         } elsif ( $reserve->{'ResFound'} eq "Reserved" ) {
457             $template->param(
458                 intransit    => $branchCheck ? undef : 1,
459                 transfertodo => $branchCheck ? undef : 1,
460                 reserve_id   => $reserve->{reserve_id},
461                 reserved     => 1,
462                 itemnumber   => $itemnumber,
463             );
464         }
465
466     } # else { ; }  # error?
467
468     # same params for Waiting or Reserved
469     $template->param(
470         found          => 1,
471         patron         => $patron,
472         barcode        => $barcode,
473         destbranch     => $reserve->{'branchcode'},
474         reservenotes   => $reserve->{'reservenotes'},
475         reserve_id     => $reserve->{reserve_id},
476         bormessagepref => $holdmsgpreferences->{'transports'},
477     );
478 }
479
480 # Error Messages
481 my @errmsgloop;
482 foreach my $code ( keys %$messages ) {
483     my %err;
484     my $exit_required_p = 0;
485     if ( $code eq 'BadBarcode' ) {
486         $err{badbarcode} = 1;
487         $err{msg}        = $messages->{'BadBarcode'};
488     }
489     elsif ( $code eq 'NotIssued' ) {
490         $err{notissued} = 1;
491         $err{msg} = '';
492     }
493     elsif ( $code eq 'LocalUse' ) {
494         $err{localuse} = 1;
495     }
496     elsif ( $code eq 'WasLost' ) {
497         $err{waslost} = 1;
498         $exit_required_p = 1 if C4::Context->preference("BlockReturnOfLostItems");
499     }
500     elsif ( $code eq 'LostItemFeeRefunded' ) {
501         $template->param( LostItemFeeRefunded => 1 );
502     }
503     elsif ( $code eq 'ResFound' ) {
504         ;    # FIXME... anything to do here?
505     }
506     elsif ( $code eq 'WasReturned' ) {
507         ;    # FIXME... anything to do here?
508     }
509     elsif ( $code eq 'WasTransfered' ) {
510         ;    # FIXME... anything to do here?
511     }
512     elsif ( $code eq 'withdrawn' ) {
513         $err{withdrawn} = 1;
514         $exit_required_p = 1 if C4::Context->preference("BlockReturnOfWithdrawnItems");
515     }
516     elsif ( $code eq 'WrongTransfer' ) {
517         ;    # FIXME... anything to do here?
518     }
519     elsif ( $code eq 'WrongTransferItem' ) {
520         ;    # FIXME... anything to do here?
521     }
522     elsif ( $code eq 'NeedsTransfer' ) {
523     }
524     elsif ( $code eq 'TransferTrigger' ) {
525         ;    # Handled alongside NeedsTransfer
526     }
527     elsif ( $code eq 'Wrongbranch' ) {
528     }
529     elsif ( $code eq 'Debarred' ) {
530         $err{debarred}            = $messages->{'Debarred'};
531         $err{debarcardnumber}     = $borrower->{cardnumber};
532         $err{debarborrowernumber} = $borrower->{borrowernumber};
533         $err{debarname}           = "$borrower->{firstname} $borrower->{surname}";
534     }
535     elsif ( $code eq 'PrevDebarred' ) {
536         $err{prevdebarred}        = $messages->{'PrevDebarred'};
537     }
538     elsif ( $code eq 'ForeverDebarred' ) {
539         $err{foreverdebarred}        = $messages->{'ForeverDebarred'};
540     }
541     elsif ( $code eq 'ItemLocationUpdated' ) {
542         $err{ItemLocationUpdated} = $messages->{ItemLocationUpdated};
543     }
544     elsif ( $code eq 'NotForLoanStatusUpdated' ) {
545         $err{NotForLoanStatusUpdated} = $messages->{NotForLoanStatusUpdated};
546     }
547     elsif ( $code eq 'DataCorrupted' ) {
548         $err{data_corrupted} = 1;
549     }
550     elsif ( $code eq 'ReturnClaims' ) {
551         $template->param( ReturnClaims => $messages->{ReturnClaims} );
552     } else {
553         die "Unknown error code $code";    # note we need all the (empty) elsif's above, or we die.
554         # This forces the issue of staying in sync w/ Circulation.pm
555     }
556     if (%err) {
557         push( @errmsgloop, \%err );
558     }
559     last if $exit_required_p;
560 }
561 $template->param( errmsgloop => \@errmsgloop );
562
563 #set up so only the last 8 returned items display (make for faster loading pages)
564 my $returned_counter = ( C4::Context->preference('numReturnedItemsToShow') ) ? C4::Context->preference('numReturnedItemsToShow') : 8;
565 my $count = 0;
566 my @riloop;
567 my $shelflocations =
568   { map { $_->{authorised_value} => $_->{lib} } Koha::AuthorisedValues->get_descriptions_by_koha_field( { frameworkcode => '', kohafield => 'items.location' } ) };
569 foreach ( sort { $a <=> $b } keys %returneditems ) {
570     my %ri;
571     if ( $count++ < $returned_counter ) {
572         my $bar_code = $returneditems{$_};
573         if ($riduedate{$_}) {
574             my $duedate = dt_from_string( $riduedate{$_}, 'sql');
575             $ri{year}  = $duedate->year();
576             $ri{month} = $duedate->month();
577             $ri{day}   = $duedate->day();
578             $ri{hour}   = $duedate->hour();
579             $ri{minute}   = $duedate->minute();
580             $ri{duedate} = output_pref($duedate);
581             my $patron = Koha::Patrons->find( $riborrowernumber{$_} );
582             unless ( $dropboxmode ) {
583                 $ri{return_overdue} = 1 if (DateTime->compare($duedate, dt_from_string()) == -1);
584             } else {
585                 $ri{return_overdue} = 1 if (DateTime->compare($duedate, $dropboxdate) == -1);
586             }
587             $ri{patron} = $patron,
588             $ri{borissuescount} = $patron->checkouts->count;
589         }
590         else {
591             $ri{borrowernumber} = $riborrowernumber{$_};
592         }
593
594         my $item = Koha::Items->find({ barcode => $bar_code });
595         next unless $item; # FIXME The item has been deleted in the meantime,
596                            # we could handle that better displaying a message in the template
597
598         my $biblio = $item->biblio;
599         # FIXME pass $item to the template and we are done here...
600         $ri{itembiblionumber}    = $biblio->biblionumber;
601         $ri{itemtitle}           = $biblio->title;
602         $ri{subtitle}            = $biblio->subtitle;
603         $ri{part_name}           = $biblio->part_name;
604         $ri{part_number}         = $biblio->part_number;
605         $ri{itemauthor}          = $biblio->author;
606         $ri{itemcallnumber}      = $item->itemcallnumber;
607         $ri{dateaccessioned}     = $item->dateaccessioned;
608         $ri{recordtype}          = $biblio->itemtype;
609         $ri{itemtype}            = $item->itype;
610         $ri{itemnote}            = $item->itemnotes;
611         $ri{itemnotes_nonpublic} = $item->itemnotes_nonpublic;
612         $ri{ccode}               = $item->ccode;
613         $ri{enumchron}           = $item->enumchron;
614         $ri{itemnumber}          = $item->itemnumber;
615         $ri{barcode}             = $bar_code;
616         $ri{homebranch}          = $item->homebranch;
617         $ri{holdingbranch}       = $item->holdingbranch;
618
619         $ri{location} = $item->location;
620         my $shelfcode = $ri{'location'};
621         $ri{'location'} = $shelflocations->{$shelfcode} if ( defined( $shelfcode ) && defined($shelflocations) && exists( $shelflocations->{$shelfcode} ) );
622
623     }
624     else {
625         last;
626     }
627     push @riloop, \%ri;
628 }
629
630 $template->param(
631     riloop         => \@riloop,
632     errmsgloop     => \@errmsgloop,
633     exemptfine     => $exemptfine,
634     dropboxmode    => $dropboxmode,
635     dropboxdate    => $dropboxdate,
636     forgivemanualholdsexpire => $forgivemanualholdsexpire,
637     overduecharges => $overduecharges,
638     AudioAlerts        => C4::Context->preference("AudioAlerts"),
639 );
640
641 if ( $barcode ) {
642     my $item_from_barcode = Koha::Items->find({barcode => $barcode }); # How many times do we fetch this item?!?
643     if ( $item_from_barcode ) {
644         $itemnumber = $item_from_barcode->itemnumber;
645         my ( $holdingBranch, $collectionBranch ) = GetCollectionItemBranches( $itemnumber );
646         if ( $holdingBranch and $collectionBranch ) {
647             $holdingBranch //= '';
648             $collectionBranch //= $returnbranch;
649             if ( ! ( $holdingBranch eq $collectionBranch ) ) {
650                 $template->param(
651                   collectionItemNeedsTransferred => 1,
652                   collectionBranch => $collectionBranch,
653                   itemnumber => $itemnumber,
654                 );
655             }
656         }
657     }
658 }
659
660 # Checking if there is a Fast Cataloging Framework
661 $template->param( fast_cataloging => 1 ) if Koha::BiblioFrameworks->find( 'FA' );
662
663 # actually print the page!
664 output_html_with_http_headers $query, $cookie, $template->output;