=cut
-use strict;
-use warnings;
+use Modern::Perl;
# FIXME There are weird things going on with $patron and $borrowernumber in this script
use CGI qw ( -utf8 );
use DateTime;
-use C4::Context;
+
use C4::Auth qw/:DEFAULT get_session/;
-use C4::Output;
-use C4::Circulation;
-use C4::Print;
-use C4::Reserves;
use C4::Biblio;
+use C4::Circulation;
+use C4::Context;
use C4::Items;
-use C4::Members;
-use C4::Members::Messaging;
use C4::Koha; # FIXME : is it still useful ?
+use C4::Members::Messaging;
+use C4::Members;
+use C4::Output;
+use C4::Print;
+use C4::Reserves;
use C4::RotatingCollections;
use Koha::AuthorisedValues;
-use Koha::DateUtils;
-use Koha::Calendar;
use Koha::BiblioFrameworks;
+use Koha::Calendar;
use Koha::Checkouts;
+use Koha::DateUtils;
use Koha::Holds;
use Koha::Items;
use Koha::Patrons;
if ( $query->param('print_slip') ) {
$template->param(
print_slip => 1,
- borrowernumber => scalar $query->param('borrowernumber'),
+ borrowernumber => scalar $query->param('borrowernumber'), # FIXME We should send a Koha::Patron and raise an error if not exist.
biblionumber => scalar $query->param('biblionumber'),
+ itemnumber => scalar $query->param('itemnumber'),
);
}
itembiblionumber => $biblio->biblionumber,
iteminfo => $biblio->author,
name => $name,
- borrowernumber => $borrowernumber,
- borcnum => $patron->cardnumber,
- borfirstname => $patron->firstname,
- borsurname => $patron->surname,
- borcategory => $patron->category->description,
+ patron => $patron,
diffbranch => 1,
);
}
my $dotransfer = $query->param('dotransfer');
my $canceltransfer = $query->param('canceltransfer');
my $dest = $query->param('dest');
-my $calendar = Koha::Calendar->new( branchcode => $userenv_branch );
#dropbox: get last open day (today - 1)
-my $today = DateTime->now( time_zone => C4::Context->tz());
-my $dropboxdate = $calendar->addDate($today, -1);
+my $dropboxdate = Koha::Checkouts::calculate_dropbox_date();
my $return_date_override = $query->param('return_date_override');
+my $return_date_override_dt;
my $return_date_override_remember =
$query->param('return_date_override_remember');
if ($return_date_override) {
if ( C4::Context->preference('SpecifyReturnDate') ) {
- my $return_date_override_dt = eval {dt_from_string( $return_date_override ) };
+ $return_date_override_dt = eval {dt_from_string( $return_date_override ) };
if ( $return_date_override_dt ) {
# note that we've overriden the return date
$template->param( return_date_was_overriden => 1);
ccode => $item->ccode,
itembiblionumber => $biblio->biblionumber,
biblionumber => $biblio->biblionumber,
- borrower => $borrower,
additional_materials => $materials,
issue => $checkout,
);
barcode => $barcode,
);
+ my $return_date = $dropboxmode ? $dropboxdate : $return_date_override_dt;
# do the return
( $returned, $messages, $issue, $borrower ) =
- AddReturn( $barcode, $userenv_branch, $exemptfine, $dropboxmode, $return_date_override, $dropboxdate );
+ AddReturn( $barcode, $userenv_branch, $exemptfine, $return_date );
if ($returned) {
my $time_now = DateTime->now( time_zone => C4::Context->tz )->truncate( to => 'minute');
my $reserved = 0;
# new op dev : we check if the document must be returned to his homebranch directly,
-# if the document is transfered, we have warning message .
+# if the document is transferred, we have warning message .
if ( $messages->{'WasTransfered'} ) {
$template->param(
);
}
-# case of wrong transfert, if the document wasn't transfered to the right library (according to branchtransfer (tobranch) BDD)
+# case of wrong transfert, if the document wasn't transferred to the right library (according to branchtransfer (tobranch) BDD)
if ( $messages->{'WrongTransfer'} and not $messages->{'WasTransfered'}) {
$template->param(
my $patron = Koha::Patrons->find( $reserve->{'borrowernumber'} );
my $name = $patron->surname . ", " . $patron->title . " " . $patron->firstname;
$template->param(
- # FIXME The full patron object should be passed to the template
- wname => $name,
- wborfirstname => $patron->firstname,
- wborsurname => $patron->surname,
- wborcategory => $patron->category->description,
- wbortitle => $patron->title,
- wborphone => $patron->phone,
- wboremail => $patron->email,
- streetnumber => $patron->streetnumber,
- address => $patron->address,
- address2 => $patron->address2,
- city => $patron->city,
- zipcode => $patron->zipcode,
- state => $patron->state,
- country => $patron->country,
- wborrowernumber => $reserve->{'borrowernumber'},
- wborcnum => $patron->cardnumber,
+ patron => $patron,
);
}
$template->param(
my $reserve = $messages->{'ResFound'};
my $patron = Koha::Patrons->find( $reserve->{borrowernumber} );
my $holdmsgpreferences = C4::Members::Messaging::GetMessagingPreferences( { borrowernumber => $reserve->{'borrowernumber'}, message_name => 'Hold_Filled' } );
- if ( $reserve->{'ResFound'} eq "Waiting" or $reserve->{'ResFound'} eq "Reserved" ) {
+ my $branchCheck = ( $userenv_branch eq $reserve->{branchcode} );
+ if ( $reserve->{'ResFound'} eq "Reserved" && C4::Context->preference('HoldsAutoFill') ) {
+ my $item = Koha::Items->find( $itemnumber );
+ my $biblio = $item->biblio;
+
+ my $diffBranchSend = !$branchCheck ? $reserve->{branchcode} : undef;
+ ModReserveAffect( $reserve->{itemnumber}, $reserve->{borrowernumber}, $diffBranchSend, $reserve->{reserve_id} );
+ my ( $messages, $nextreservinfo ) = GetOtherReserves($reserve->{itemnumber});
+
+ my $patron = Koha::Patrons->find( $nextreservinfo );
+
+ $template->param(
+ hold_auto_filled => 1,
+ print_slip => C4::Context->preference('HoldsAutoFillPrintSlip'),
+ patron => $patron,
+ borrowernumber => $patron->id,
+ biblionumber => $biblio->id,
+ );
+
+ if ( $messages->{'transfert'} ) {
+ $template->param(
+ itemtitle => $biblio->title,
+ itemnumber => $item->itemnumber,
+ itembiblionumber => $biblio->biblionumber,
+ iteminfo => $biblio->author,
+ diffbranch => 1,
+ );
+ }
+ }
+ elsif ( $reserve->{'ResFound'} eq "Waiting" or $reserve->{'ResFound'} eq "Reserved" ) {
if ( $reserve->{'ResFound'} eq "Waiting" ) {
$template->param(
- waiting => ($userenv_branch eq $reserve->{'branchcode'} ? 1 : 0 ),
+ waiting => $branchCheck ? 1 : undef,
);
} elsif ( $reserve->{'ResFound'} eq "Reserved" ) {
$template->param(
- intransit => ($userenv_branch eq $reserve->{'branchcode'} ? 0 : 1 ),
- transfertodo => ($userenv_branch eq $reserve->{'branchcode'} ? 0 : 1 ),
+ intransit => $branchCheck ? undef : 1,
+ transfertodo => $branchCheck ? undef : 1,
reserve_id => $reserve->{reserve_id},
reserved => 1,
);
}
- # same params for Waiting or Reserved
- $template->param(
- # FIXME The full patron object should be passed to the template
- found => 1,
- name => $patron->surname . ", " . $patron->title . " " . $patron->firstname,
- borfirstname => $patron->firstname,
- borsurname => $patron->surname,
- borcategory => $patron->category->description,
- bortitle => $patron->title,
- borphone => $patron->phone,
- boremail => $patron->email,
- boraddress => $patron->address,
- boraddress2 => $patron->address2,
- streetnumber => $patron->streetnumber,
- city => $patron->city,
- zipcode => $patron->zipcode,
- state => $patron->state,
- country => $patron->country,
- borcnum => $patron->cardnumber,
- debarred => $patron->debarred,
- gonenoaddress => $patron->gonenoaddress,
- barcode => $barcode,
- destbranch => $reserve->{'branchcode'},
- borrowernumber => $reserve->{'borrowernumber'},
- itemnumber => $reserve->{'itemnumber'},
- reservenotes => $reserve->{'reservenotes'},
- reserve_id => $reserve->{reserve_id},
- bormessagepref => $holdmsgpreferences->{'transports'},
- );
} # else { ; } # error?
+
+ # same params for Waiting or Reserved
+ $template->param(
+ found => 1,
+ patron => $patron,
+ barcode => $barcode,
+ destbranch => $reserve->{'branchcode'},
+ itemnumber => $reserve->{'itemnumber'},
+ reservenotes => $reserve->{'reservenotes'},
+ reserve_id => $reserve->{reserve_id},
+ bormessagepref => $holdmsgpreferences->{'transports'},
+ );
}
# Error Messages
elsif ( $code eq 'NotIssued' ) {
$err{notissued} = 1;
$err{msg} = '';
- $err{msg} = $messages->{'IsPermanent'} if $messages->{'IsPermanent'};
}
elsif ( $code eq 'LocalUse' ) {
$err{localuse} = 1;
$err{withdrawn} = 1;
$exit_required_p = 1 if C4::Context->preference("BlockReturnOfWithdrawnItems");
}
- elsif ( ( $code eq 'IsPermanent' ) && ( not $messages->{'ResFound'} ) ) {
- if ( $messages->{'IsPermanent'} ne $userenv_branch ) {
- $err{ispermanent} = 1;
- $err{msg} = $messages->{'IsPermanent'};
- }
- }
elsif ( $code eq 'WrongTransfer' ) {
; # FIXME... anything to do here?
}
elsif ( $code eq 'ForeverDebarred' ) {
$err{foreverdebarred} = $messages->{'ForeverDebarred'};
}
+ elsif ( $code eq 'ItemLocationUpdated' ) {
+ $err{ItemLocationUpdated} = $messages->{ItemLocationUpdated};
+ }
elsif ( $code eq 'NotForLoanStatusUpdated' ) {
$err{NotForLoanStatusUpdated} = $messages->{NotForLoanStatusUpdated};
}
elsif ( $code eq 'DataCorrupted' ) {
$err{data_corrupted} = 1;
}
- else {
+ elsif ( $code eq 'ReturnClaims' ) {
+ $template->param( ReturnClaims => $messages->{ReturnClaims} );
+ } else {
die "Unknown error code $code"; # note we need all the (empty) elsif's above, or we die.
# This forces the issue of staying in sync w/ Circulation.pm
}
} else {
$ri{return_overdue} = 1 if (DateTime->compare($duedate, $dropboxdate) == -1);
}
- $ri{borrowernumber} = $patron->borrowernumber;
- $ri{borcnum} = $patron->cardnumber;
- $ri{borfirstname} = $patron->firstname;
- $ri{borsurname} = $patron->surname;
- $ri{bortitle} = $patron->title;
- $ri{bornote} = $patron->borrowernotes;
- $ri{borcategorycode}= $patron->categorycode;
- $ri{borissuescount} = Koha::Checkouts->count( { borrowernumber => $patron->borrowernumber } );
+ $ri{patron} = $patron,
+ $ri{borissuescount} = $patron->checkouts->count;
}
else {
$ri{borrowernumber} = $riborrowernumber{$_};
}
my $item = Koha::Items->find({ barcode => $bar_code });
+ next unless $item; # FIXME The item has been deleted in the meantime,
+ # we could handle that better displaying a message in the template
+
my $biblio = $item->biblio;
# FIXME pass $item to the template and we are done here...
$ri{itembiblionumber} = $biblio->biblionumber;
$ri{itemtitle} = $biblio->title;
+ $ri{subtitle} = $biblio->subtitle;
+ $ri{part_name} = $biblio->part_name;
+ $ri{part_number} = $biblio->part_number;
$ri{itemauthor} = $biblio->author;
$ri{itemcallnumber} = $item->itemcallnumber;
$ri{dateaccessioned} = $item->dateaccessioned;
- $ri{itemtype} = $item->effective_itemtype;
+ $ri{recordtype} = $biblio->itemtype;
+ $ri{itemtype} = $item->itype;
$ri{itemnote} = $item->itemnotes;
$ri{itemnotes_nonpublic} = $item->itemnotes_nonpublic;
$ri{ccode} = $item->ccode;
$ri{homebranch} = $item->homebranch;
$ri{holdingbranch} = $item->holdingbranch;
- $ri{location} = $biblio->{'location'};
+ $ri{location} = $item->location;
my $shelfcode = $ri{'location'};
$ri{'location'} = $shelflocations->{$shelfcode} if ( defined( $shelfcode ) && defined($shelflocations) && exists( $shelflocations->{$shelfcode} ) );
errmsgloop => \@errmsgloop,
exemptfine => $exemptfine,
dropboxmode => $dropboxmode,
- dropboxdate => output_pref($dropboxdate),
+ dropboxdate => $dropboxdate,
forgivemanualholdsexpire => $forgivemanualholdsexpire,
overduecharges => $overduecharges,
AudioAlerts => C4::Context->preference("AudioAlerts"),
);
-$itemnumber = GetItemnumberFromBarcode( $barcode );
-if ( $itemnumber ) {
- my ( $holdingBranch, $collectionBranch ) = GetCollectionItemBranches( $itemnumber );
- if ( $holdingBranch and $collectionBranch ) {
- $holdingBranch //= '';
- $collectionBranch //= $returnbranch;
- if ( ! ( $holdingBranch eq $collectionBranch ) ) {
- $template->param(
- collectionItemNeedsTransferred => 1,
- collectionBranch => $collectionBranch,
- itemnumber => $itemnumber,
- );
+if ( $barcode ) {
+ my $item_from_barcode = Koha::Items->find({barcode => $barcode }); # How many times do we fetch this item?!?
+ if ( $item_from_barcode ) {
+ $itemnumber = $item_from_barcode->itemnumber;
+ my ( $holdingBranch, $collectionBranch ) = GetCollectionItemBranches( $itemnumber );
+ if ( $holdingBranch and $collectionBranch ) {
+ $holdingBranch //= '';
+ $collectionBranch //= $returnbranch;
+ if ( ! ( $holdingBranch eq $collectionBranch ) ) {
+ $template->param(
+ collectionItemNeedsTransferred => 1,
+ collectionBranch => $collectionBranch,
+ itemnumber => $itemnumber,
+ );
+ }
}
}
}
projects / koha-ffzg.git / blobdiff