Bugzilla – Attachment 99291 Details for
Bug 24688
Hold priority isn't adjusted correctly if checking out a middle priority hold
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24688: Check the priority of the fulfilled hold when fixing priorities
Bug-24688-Check-the-priority-of-the-fulfilled-hold.patch (text/plain), 1.67 KB, created by
Aleisha Amohia
on 2020-02-20 01:44:18 UTC
(
hide
)
Description:
Bug 24688: Check the priority of the fulfilled hold when fixing priorities
Filename:
MIME Type:
Creator:
Aleisha Amohia
Created:
2020-02-20 01:44:18 UTC
Size:
1.67 KB
patch
obsolete
>From 07ffb96775ffaa1af2fcbcd4d1501849e3d531e9 Mon Sep 17 00:00:00 2001 >From: Aleisha Amohia <aleishaamohia@hotmail.com> >Date: Thu, 20 Feb 2020 01:36:32 +0000 >Subject: [PATCH] Bug 24688: Check the priority of the fulfilled hold when > fixing priorities > >Currently when fixing priorities of existing holds, priorities aren't >fixed when checking out a middle priority hold because the hold has been >fulfilled and deleted. This patch adds a check for, if the hold can't be >found, an old hold with the required reserve_id. > >To test: >1. Place at least four holds on a biblio for different borrowers >2. Check out an item to the 3rd priority borrower >3. Check the existing holds table for the biblio. The priorities might be >out of order, or try clicking the dropdowns and there might be an empty >dropdown option, or open the browser developer tools and notice the option >values in the select are missing value=3. >4. Apply patch and restart memcached >5. Check out an item to the 2nd priority borrower >6. Check the existing holds table for the biblio. Confirm the priorities >have been fixed and are now consecutive with no gaps. > >Sponsored-by: Chartered Accountants Australia and New Zealand >--- > C4/Reserves.pm | 4 ++++ > 1 file changed, 4 insertions(+) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 00d921d..ea2088e 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -1485,6 +1485,10 @@ sub _FixPriority { > my $hold; > if ( $reserve_id ) { > $hold = Koha::Holds->find( $reserve_id ); >+ if (!defined $hold){ >+ # may have already been checked out and hold fulfilled >+ $hold = Koha::Old::Holds->find( $reserve_id ); >+ } > return unless $hold; > } > >-- >2.1.4
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 24688
:
99291
|
100435
|
101117