|
Lines 22-27
use Modern::Perl;
Link Here
|
| 22 |
use DateTime; |
22 |
use DateTime; |
| 23 |
use POSIX qw( floor ); |
23 |
use POSIX qw( floor ); |
| 24 |
use Encode; |
24 |
use Encode; |
|
|
25 |
use JSON; |
| 25 |
|
26 |
|
| 26 |
use C4::Context; |
27 |
use C4::Context; |
| 27 |
use C4::Stats qw( UpdateStats ); |
28 |
use C4::Stats qw( UpdateStats ); |
|
Lines 317-322
A recall for this item was found, and the item needs to be transferred to the re
Link Here
|
| 317 |
|
318 |
|
| 318 |
=cut |
319 |
=cut |
| 319 |
|
320 |
|
|
|
321 |
my $query = CGI->new; |
| 322 |
|
| 323 |
my $stickyduedate = $query->param('stickyduedate'); |
| 324 |
my $duedatespec = $query->param('duedatespec'); |
| 325 |
my $restoreduedatespec = $query->param('restoreduedatespec') || $duedatespec; |
| 326 |
if ( $restoreduedatespec && $restoreduedatespec eq "highholds_empty" ) { |
| 327 |
undef $restoreduedatespec; |
| 328 |
} |
| 329 |
my $issueconfirmed = $query->param('issueconfirmed'); |
| 330 |
my $cancelreserve = $query->param('cancelreserve'); |
| 331 |
my $cancel_recall = $query->param('cancel_recall'); |
| 332 |
my $recall_id = $query->param('recall_id'); |
| 333 |
my $debt_confirmed = $query->param('debt_confirmed') || 0; # Don't show the debt error dialog twice |
| 334 |
my $charges = $query->param('charges') || q{}; |
| 335 |
|
| 320 |
sub transferbook { |
336 |
sub transferbook { |
| 321 |
my $params = shift; |
337 |
my $params = shift; |
| 322 |
my $tbr = $params->{to_branch}; |
338 |
my $tbr = $params->{to_branch}; |
|
Lines 801-809
sub CanBookBeIssued {
Link Here
|
| 801 |
my $circ_library = Koha::Libraries->find( _GetCircControlBranch($item_object, $patron) ); |
817 |
my $circ_library = Koha::Libraries->find( _GetCircControlBranch($item_object, $patron) ); |
| 802 |
|
818 |
|
| 803 |
my $now = dt_from_string(); |
819 |
my $now = dt_from_string(); |
|
|
820 |
my $message; |
| 821 |
my @message; |
| 822 |
|
| 804 |
$duedate ||= CalcDateDue( $now, $effective_itemtype, $circ_library->branchcode, $patron ); |
823 |
$duedate ||= CalcDateDue( $now, $effective_itemtype, $circ_library->branchcode, $patron ); |
|
|
824 |
|
| 805 |
if (DateTime->compare($duedate,$now) == -1 ) { # duedate cannot be before now |
825 |
if (DateTime->compare($duedate,$now) == -1 ) { # duedate cannot be before now |
| 806 |
$needsconfirmation{INVALID_DATE} = $duedate; |
826 |
$needsconfirmation{INVALID_DATE} = $duedate; |
|
|
827 |
if ($issueconfirmed) { |
| 828 |
if ($message) { |
| 829 |
$message = "$message + sticky due date is invalid or due date in the past"; |
| 830 |
} else { |
| 831 |
$message = "sticky due date is invalid or due date in the past"; |
| 832 |
} |
| 833 |
push( @message, "sticky due date is invalid or due date in the past" ); |
| 834 |
} |
| 835 |
|
| 807 |
} |
836 |
} |
| 808 |
|
837 |
|
| 809 |
my $fees = Koha::Charges::Fees->new( |
838 |
my $fees = Koha::Charges::Fees->new( |
|
Lines 848-853
sub CanBookBeIssued {
Link Here
|
| 848 |
} |
877 |
} |
| 849 |
if ( $patron->is_debarred ) { |
878 |
if ( $patron->is_debarred ) { |
| 850 |
$issuingimpossible{DEBARRED} = 1; |
879 |
$issuingimpossible{DEBARRED} = 1; |
|
|
880 |
if ($issueconfirmed) { |
| 881 |
if ($message) { |
| 882 |
$message = "$message + borrower is restricted"; |
| 883 |
} else { |
| 884 |
$message = "borrower is restricted"; |
| 885 |
} |
| 886 |
push( @message, "borrower is restricted" ); |
| 887 |
} |
| 851 |
} |
888 |
} |
| 852 |
|
889 |
|
| 853 |
if ( $patron->is_expired ) { |
890 |
if ( $patron->is_expired ) { |
|
Lines 914-923
sub CanBookBeIssued {
Link Here
|
| 914 |
else { |
951 |
else { |
| 915 |
if ( $non_issues_charges > $amountlimit && $allowfineoverride ) { |
952 |
if ( $non_issues_charges > $amountlimit && $allowfineoverride ) { |
| 916 |
$needsconfirmation{DEBT} = $non_issues_charges; |
953 |
$needsconfirmation{DEBT} = $non_issues_charges; |
|
|
954 |
if ($issueconfirmed) { |
| 955 |
if ($message) { |
| 956 |
$message = "$message + borrower had amend"; |
| 957 |
} else { |
| 958 |
$message = "borrower had amend"; |
| 959 |
} |
| 960 |
push( @message, "borrower had amend" ); |
| 961 |
} |
| 917 |
} elsif ( $non_issues_charges > $amountlimit && !$allowfineoverride) { |
962 |
} elsif ( $non_issues_charges > $amountlimit && !$allowfineoverride) { |
| 918 |
$issuingimpossible{DEBT} = $non_issues_charges; |
963 |
$issuingimpossible{DEBT} = $non_issues_charges; |
| 919 |
} elsif ( $non_issues_charges > 0 && $allfinesneedoverride ) { |
964 |
} elsif ( $non_issues_charges > 0 && $allfinesneedoverride ) { |
| 920 |
$needsconfirmation{DEBT} = $non_issues_charges; |
965 |
$needsconfirmation{DEBT} = $non_issues_charges; |
|
|
966 |
if ($issueconfirmed) { |
| 967 |
if ($message) { |
| 968 |
$message = "$message + borrower had amend"; |
| 969 |
} else { |
| 970 |
$message = "borrower had amend"; |
| 971 |
} |
| 972 |
push( @message, "borrower had amend" ); |
| 973 |
} |
| 921 |
} |
974 |
} |
| 922 |
} |
975 |
} |
| 923 |
|
976 |
|
|
Lines 994-1006
sub CanBookBeIssued {
Link Here
|
| 994 |
} else { |
1047 |
} else { |
| 995 |
if ( C4::Context->preference('AutoReturnCheckedOutItems') ) { |
1048 |
if ( C4::Context->preference('AutoReturnCheckedOutItems') ) { |
| 996 |
$alerts{RETURNED_FROM_ANOTHER} = { patron => $patron }; |
1049 |
$alerts{RETURNED_FROM_ANOTHER} = { patron => $patron }; |
| 997 |
} |
1050 |
} else { |
| 998 |
else { |
|
|
| 999 |
$needsconfirmation{ISSUED_TO_ANOTHER} = 1; |
1051 |
$needsconfirmation{ISSUED_TO_ANOTHER} = 1; |
| 1000 |
$needsconfirmation{issued_firstname} = $patron->firstname; |
1052 |
$needsconfirmation{issued_firstname} = $patron->firstname; |
| 1001 |
$needsconfirmation{issued_surname} = $patron->surname; |
1053 |
$needsconfirmation{issued_surname} = $patron->surname; |
| 1002 |
$needsconfirmation{issued_cardnumber} = $patron->cardnumber; |
1054 |
$needsconfirmation{issued_cardnumber} = $patron->cardnumber; |
| 1003 |
$needsconfirmation{issued_borrowernumber} = $patron->borrowernumber; |
1055 |
$needsconfirmation{issued_borrowernumber} = $patron->borrowernumber; |
|
|
1056 |
if ($issueconfirmed) { |
| 1057 |
if ($message) { |
| 1058 |
$message = "$message + item is checked out for someone else "; |
| 1059 |
} else { |
| 1060 |
$message = "item is checked out for someone else"; |
| 1061 |
} |
| 1062 |
push( @message, "item is checked out for someone else" ); |
| 1063 |
} |
| 1004 |
} |
1064 |
} |
| 1005 |
} |
1065 |
} |
| 1006 |
} |
1066 |
} |
|
Lines 1024-1029
sub CanBookBeIssued {
Link Here
|
| 1024 |
$needsconfirmation{current_loan_count} = $toomany->{count}; |
1084 |
$needsconfirmation{current_loan_count} = $toomany->{count}; |
| 1025 |
$needsconfirmation{max_loans_allowed} = $toomany->{max_allowed}; |
1085 |
$needsconfirmation{max_loans_allowed} = $toomany->{max_allowed}; |
| 1026 |
$needsconfirmation{circulation_rule_TOO_MANY} = $toomany->{circulation_rule}; |
1086 |
$needsconfirmation{circulation_rule_TOO_MANY} = $toomany->{circulation_rule}; |
|
|
1087 |
if ($issueconfirmed) { |
| 1088 |
if ($message) { |
| 1089 |
$message = "$message + too many checkout"; |
| 1090 |
} else { |
| 1091 |
$message = "too many checkout"; |
| 1092 |
} |
| 1093 |
push( @message, "too many checkout" ); |
| 1094 |
} |
| 1027 |
} else { |
1095 |
} else { |
| 1028 |
$issuingimpossible{TOO_MANY} = $toomany->{reason}; |
1096 |
$issuingimpossible{TOO_MANY} = $toomany->{reason}; |
| 1029 |
$issuingimpossible{current_loan_count} = $toomany->{count}; |
1097 |
$issuingimpossible{current_loan_count} = $toomany->{count}; |
|
Lines 1051-1056
sub CanBookBeIssued {
Link Here
|
| 1051 |
}else{ |
1119 |
}else{ |
| 1052 |
$needsconfirmation{NOT_FOR_LOAN_FORCING} = 1; |
1120 |
$needsconfirmation{NOT_FOR_LOAN_FORCING} = 1; |
| 1053 |
$needsconfirmation{item_notforloan} = $item_object->notforloan; |
1121 |
$needsconfirmation{item_notforloan} = $item_object->notforloan; |
|
|
1122 |
if ($issueconfirmed) { |
| 1123 |
if ($message) { |
| 1124 |
$message = "$message + item not for loan"; |
| 1125 |
} else { |
| 1126 |
$message = "item not for loan"; |
| 1127 |
} |
| 1128 |
push( @message, "item not for loan" ); |
| 1129 |
} |
| 1054 |
} |
1130 |
} |
| 1055 |
} |
1131 |
} |
| 1056 |
else { |
1132 |
else { |
|
Lines 1097-1102
sub CanBookBeIssued {
Link Here
|
| 1097 |
my $code = $av->count ? $av->next->lib : ''; |
1173 |
my $code = $av->count ? $av->next->lib : ''; |
| 1098 |
$needsconfirmation{ITEM_LOST} = $code if ( C4::Context->preference("IssueLostItem") eq 'confirm' ); |
1174 |
$needsconfirmation{ITEM_LOST} = $code if ( C4::Context->preference("IssueLostItem") eq 'confirm' ); |
| 1099 |
$alerts{ITEM_LOST} = $code if ( C4::Context->preference("IssueLostItem") eq 'alert' ); |
1175 |
$alerts{ITEM_LOST} = $code if ( C4::Context->preference("IssueLostItem") eq 'alert' ); |
|
|
1176 |
if ($issueconfirmed) { |
| 1177 |
if ($message) { |
| 1178 |
$message = "$message + item lost"; |
| 1179 |
} else { |
| 1180 |
$message = "item lost"; |
| 1181 |
} |
| 1182 |
push( @message, "item lost" ); |
| 1183 |
} |
| 1100 |
} |
1184 |
} |
| 1101 |
if ( C4::Context->preference("IndependentBranches") ) { |
1185 |
if ( C4::Context->preference("IndependentBranches") ) { |
| 1102 |
my $userenv = C4::Context->userenv; |
1186 |
my $userenv = C4::Context->userenv; |
|
Lines 1193-1198
sub CanBookBeIssued {
Link Here
|
| 1193 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
1277 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
| 1194 |
$needsconfirmation{'reswaitingdate'} = $res->{'waitingdate'}; |
1278 |
$needsconfirmation{'reswaitingdate'} = $res->{'waitingdate'}; |
| 1195 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
1279 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
|
|
1280 |
if ($issueconfirmed) { |
| 1281 |
if ($message) { |
| 1282 |
$message = |
| 1283 |
"$message + item is on reserve and waiting, but has been reserved by some other patron."; |
| 1284 |
} else { |
| 1285 |
$message = "item is on reserve and waiting, but has been reserved by some other patron."; |
| 1286 |
} |
| 1287 |
push( @message, "item is on reserve and waiting, but has been reserved by some other patron" ); |
| 1288 |
} |
| 1196 |
} |
1289 |
} |
| 1197 |
elsif ( $restype eq "Reserved" ) { |
1290 |
elsif ( $restype eq "Reserved" ) { |
| 1198 |
# The item is on reserve for someone else. |
1291 |
# The item is on reserve for someone else. |
|
Lines 1204-1209
sub CanBookBeIssued {
Link Here
|
| 1204 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
1297 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
| 1205 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
1298 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
| 1206 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
1299 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
|
|
1300 |
if ($issueconfirmed) { |
| 1301 |
if ($message) { |
| 1302 |
$message = "$message + item is on reserve for someone else"; |
| 1303 |
} else { |
| 1304 |
$message = "item is on reserve for someone else"; |
| 1305 |
} |
| 1306 |
push( @message, "item is on reserve for someone else" ); |
| 1307 |
} |
| 1207 |
} |
1308 |
} |
| 1208 |
elsif ( $restype eq "Transferred" ) { |
1309 |
elsif ( $restype eq "Transferred" ) { |
| 1209 |
# The item is determined hold being transferred for someone else. |
1310 |
# The item is determined hold being transferred for someone else. |
|
Lines 1215-1220
sub CanBookBeIssued {
Link Here
|
| 1215 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
1316 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
| 1216 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
1317 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
| 1217 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
1318 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
|
|
1319 |
if ($issueconfirmed) { |
| 1320 |
if ($message) { |
| 1321 |
$message = "$message + item is determined hold being transferred for someone else"; |
| 1322 |
} else { |
| 1323 |
$message = "item is determined hold being transferred for someone else"; |
| 1324 |
} |
| 1325 |
push( @message, "item is determined hold being transferred for someone else" ); |
| 1326 |
} |
| 1218 |
} |
1327 |
} |
| 1219 |
elsif ( $restype eq "Processing" ) { |
1328 |
elsif ( $restype eq "Processing" ) { |
| 1220 |
# The item is determined hold being processed for someone else. |
1329 |
# The item is determined hold being processed for someone else. |
|
Lines 1226-1231
sub CanBookBeIssued {
Link Here
|
| 1226 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
1335 |
$needsconfirmation{'resbranchcode'} = $res->{branchcode}; |
| 1227 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
1336 |
$needsconfirmation{'resreservedate'} = $res->{reservedate}; |
| 1228 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
1337 |
$needsconfirmation{'reserve_id'} = $res->{reserve_id}; |
|
|
1338 |
if ($issueconfirmed) { |
| 1339 |
if ($message) { |
| 1340 |
$message = "$message + item is determined hold being processed for someone else"; |
| 1341 |
} else { |
| 1342 |
$message = "item is determined hold being processed for someone else"; |
| 1343 |
} |
| 1344 |
push( @message, "item is determined hold being processed for someone else" ); |
| 1345 |
} |
| 1229 |
} |
1346 |
} |
| 1230 |
} |
1347 |
} |
| 1231 |
} |
1348 |
} |
|
Lines 1269-1274
sub CanBookBeIssued {
Link Here
|
| 1269 |
if ( $daysToAgeRestriction && $daysToAgeRestriction > 0 ) { |
1386 |
if ( $daysToAgeRestriction && $daysToAgeRestriction > 0 ) { |
| 1270 |
if ( C4::Context->preference('AgeRestrictionOverride') ) { |
1387 |
if ( C4::Context->preference('AgeRestrictionOverride') ) { |
| 1271 |
$needsconfirmation{AGE_RESTRICTION} = "$agerestriction"; |
1388 |
$needsconfirmation{AGE_RESTRICTION} = "$agerestriction"; |
|
|
1389 |
if ($message) { |
| 1390 |
$message = "$message + age restriction"; |
| 1391 |
} else { |
| 1392 |
$message = "age restriction"; |
| 1393 |
} |
| 1394 |
push( @message, "age restriction" ); |
| 1272 |
} |
1395 |
} |
| 1273 |
else { |
1396 |
else { |
| 1274 |
$issuingimpossible{AGE_RESTRICTION} = "$agerestriction"; |
1397 |
$issuingimpossible{AGE_RESTRICTION} = "$agerestriction"; |
|
Lines 1325-1330
sub CanBookBeIssued {
Link Here
|
| 1325 |
} |
1448 |
} |
| 1326 |
} |
1449 |
} |
| 1327 |
|
1450 |
|
|
|
1451 |
my $borrower = $patron; |
| 1452 |
my $user = C4::Context->userenv->{number}; |
| 1453 |
my $branchcode = C4::Context->userenv->{branch}; |
| 1454 |
my $item = Koha::Items->find( { barcode => $barcode } ); |
| 1455 |
my $itemnumber = $item->itemnumber; |
| 1456 |
|
| 1457 |
# action, cardnumber, barcode, date, heure, user, branche |
| 1458 |
if ($issueconfirmed) { |
| 1459 |
|
| 1460 |
my $infos = ( |
| 1461 |
{ |
| 1462 |
message => \@message, |
| 1463 |
borrowernumber => $borrower->borrowernumber, |
| 1464 |
barcode => $barcode, |
| 1465 |
manager_id => $user, |
| 1466 |
branchcode => $branchcode, |
| 1467 |
} |
| 1468 |
); |
| 1469 |
|
| 1470 |
my $json_infos = JSON->new->utf8->pretty->encode($infos); |
| 1471 |
$json_infos =~ s/"/'/g; |
| 1472 |
|
| 1473 |
logaction( |
| 1474 |
"CIRCULATION", "ISSUE", |
| 1475 |
$borrower->{'borrowernumber'}, |
| 1476 |
$json_infos, |
| 1477 |
) if C4::Context->preference("IssueLog"); |
| 1478 |
} |
| 1479 |
|
| 1480 |
|
| 1328 |
return ( \%issuingimpossible, \%needsconfirmation, \%alerts, \%messages, ); |
1481 |
return ( \%issuingimpossible, \%needsconfirmation, \%alerts, \%messages, ); |
| 1329 |
} |
1482 |
} |
| 1330 |
|
1483 |
|
|
Lines 2343-2351
sub AddReturn {
Link Here
|
| 2343 |
if ( $transfer->tobranch eq $branch ) { |
2496 |
if ( $transfer->tobranch eq $branch ) { |
| 2344 |
$transfer->receive; |
2497 |
$transfer->receive; |
| 2345 |
$messages->{'TransferArrived'} = $transfer->frombranch; |
2498 |
$messages->{'TransferArrived'} = $transfer->frombranch; |
|
|
2499 |
|
| 2346 |
# validTransfer=1 allows us returning the item back if the reserve is cancelled |
2500 |
# validTransfer=1 allows us returning the item back if the reserve is cancelled |
| 2347 |
$validTransfer = 1 |
2501 |
$validTransfer = 1 if $transfer->reason eq 'Reserve'; |
| 2348 |
if defined $transfer->reason && $transfer->reason eq 'Reserve'; |
|
|
| 2349 |
} |
2502 |
} |
| 2350 |
else { |
2503 |
else { |
| 2351 |
$messages->{'WrongTransfer'} = $transfer->tobranch; |
2504 |
$messages->{'WrongTransfer'} = $transfer->tobranch; |
|
Lines 3053-3058
sub CanBookBeRenewed {
Link Here
|
| 3053 |
return ( 0, "on_reserve" ) |
3206 |
return ( 0, "on_reserve" ) |
| 3054 |
if ( $item->current_holds->search( { non_priority => 0 } )->count ); |
3207 |
if ( $item->current_holds->search( { non_priority => 0 } )->count ); |
| 3055 |
|
3208 |
|
|
|
3209 |
my $issuing_rule = Koha::CirculationRules->get_effective_rules( |
| 3210 |
{ |
| 3211 |
categorycode => $patron->categorycode, |
| 3212 |
itemtype => $item->effective_itemtype, |
| 3213 |
branchcode => $branchcode, |
| 3214 |
rules => [ |
| 3215 |
'renewalsallowed', |
| 3216 |
'lengthunit', |
| 3217 |
'unseen_renewals_allowed' |
| 3218 |
] |
| 3219 |
} |
| 3220 |
); |
| 3221 |
|
| 3222 |
return ( 0, "too_many" ) |
| 3223 |
if not $issuing_rule->{renewalsallowed} |
| 3224 |
or $issuing_rule->{renewalsallowed} <= $issue->renewals_count; |
| 3056 |
|
3225 |
|
| 3057 |
my ( $status, $matched_reserve, $possible_holds ) = CheckReserves($item); |
3226 |
my ( $status, $matched_reserve, $possible_holds ) = CheckReserves($item); |
| 3058 |
my @fillable_holds = (); |
3227 |
my @fillable_holds = (); |
|
Lines 3181-3188
fallback to a true value
Link Here
|
| 3181 |
|
3350 |
|
| 3182 |
C<$automatic> is a boolean flag indicating the renewal was triggered automatically and not by a person ( librarian or patron ) |
3351 |
C<$automatic> is a boolean flag indicating the renewal was triggered automatically and not by a person ( librarian or patron ) |
| 3183 |
|
3352 |
|
| 3184 |
C<$skip_record_index> is an optional boolean flag to indicate whether queuing the search indexing |
3353 |
C<$skip_record_index> is an optional boolean flag to indicate whether queuing the search indexing should be skipped for this renewal. |
| 3185 |
should be skipped for this renewal. |
|
|
| 3186 |
|
3354 |
|
| 3187 |
=cut |
3355 |
=cut |
| 3188 |
|
3356 |
|
|
Lines 3384-3392
sub AddRenewal {
Link Here
|
| 3384 |
}); |
3552 |
}); |
| 3385 |
}); |
3553 |
}); |
| 3386 |
|
3554 |
|
| 3387 |
unless( $skip_record_index ){ |
3555 |
unless ($skip_record_index) { |
|
|
3556 |
|
| 3388 |
# We index now, after the transaction is committed |
3557 |
# We index now, after the transaction is committed |
| 3389 |
my $indexer = Koha::SearchEngine::Indexer->new({ index => $Koha::SearchEngine::BIBLIOS_INDEX }); |
3558 |
my $indexer = Koha::SearchEngine::Indexer->new( { index => $Koha::SearchEngine::BIBLIOS_INDEX } ); |
| 3390 |
$indexer->index_records( $item_object->biblionumber, "specialUpdate", "biblioserver" ); |
3559 |
$indexer->index_records( $item_object->biblionumber, "specialUpdate", "biblioserver" ); |
| 3391 |
} |
3560 |
} |
| 3392 |
|
3561 |
|