Bugzilla – Attachment 87281 Details for
Bug 17978
Include 'Next available'/title level holds in holds count when placing holds (opac and staff)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17978: Check if hold can be placed before placing on placerequest.pl
Bug-17978-Check-if-hold-can-be-placed-before-placi.patch (text/plain), 2.40 KB, created by
Martin Renvoize (ashimema)
on 2019-04-01 14:15:55 UTC
(
hide
)
Description:
Bug 17978: Check if hold can be placed before placing on placerequest.pl
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2019-04-01 14:15:55 UTC
Size:
2.40 KB
patch
obsolete
>From 67b4d605e4d59f4a26f19629dee04942d152ce84 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Mon, 24 Dec 2018 14:33:51 +0000 >Subject: [PATCH] Bug 17978: Check if hold can be placed before placing on > placerequest.pl > >To test: > 1 - Set an All/All/All rule with reserves limited 2 to > 2 - Search in the staff side > 3 - Select all records (or more than 2) from the results > 4 - Click 'Place hold' > 5 - Find a patron, place holds > 6 - You get more holds than you should > 7 - Delete those holds > 8 - Apply patch > 9 - Search and select more than 2 records >10 - Find patron, place holds >11 - Only 2 holds are placed > >Signed-off-by: Liz Rea <wizzyrea@gmail.com> >Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> >--- > reserve/placerequest.pl | 14 +++++++++----- > 1 file changed, 9 insertions(+), 5 deletions(-) > >diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl >index c614200a67..1e49a4bd78 100755 >--- a/reserve/placerequest.pl >+++ b/reserve/placerequest.pl >@@ -97,14 +97,18 @@ if ( $type eq 'str8' && $borrower ) { > > if ($multi_hold) { > my $bibinfo = $bibinfos{$biblionumber}; >- AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,[$biblionumber], >- $bibinfo->{rank},$startdate,$expirationdate,$notes,$bibinfo->{title},$checkitem,$found); >+ if ( CanBookBeReserved($borrower->{'borrowernumber'}, $biblionumber)->{status} eq 'OK' ) { >+ AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,[$biblionumber], >+ $bibinfo->{rank},$startdate,$expirationdate,$notes,$bibinfo->{title},$checkitem,$found); >+ } > } else { > # place a request on 1st available > for ( my $i = 0 ; $i < $holds_to_place_count ; $i++ ) { >- AddReserve( $branch, $borrower->{'borrowernumber'}, >- $biblionumber, \@realbi, $rank[0], $startdate, $expirationdate, $notes, $title, >- $checkitem, $found, $itemtype ); >+ if ( CanBookBeReserved($borrower->{'borrowernumber'}, $biblionumber)->{status} eq 'OK' ) { >+ AddReserve( $branch, $borrower->{'borrowernumber'}, >+ $biblionumber, \@realbi, $rank[0], $startdate, $expirationdate, $notes, $title, >+ $checkitem, $found, $itemtype ); >+ } > } > } > } >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17978
:
69927
|
72547
|
73161
|
83476
|
83477
|
86829
|
86830
|
86831
|
87280
| 87281 |
87282