Bugzilla – Attachment 31140 Details for
Bug 12630
Prioritizing "Hold starts on date" -holds causes all other holds to be prioritized as well!
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 12630 - Prioritizing "Hold starts on date" -holds causes all other holds to be prioritized as well!
SIGNED-OFF-Bug-12630---Prioritizing-Hold-starts-on.patch (text/plain), 2.15 KB, created by
Kyle M Hall (khall)
on 2014-08-25 14:21:03 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 12630 - Prioritizing "Hold starts on date" -holds causes all other holds to be prioritized as well!
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2014-08-25 14:21:03 UTC
Size:
2.15 KB
patch
obsolete
>From d9c4ed3a26dc2a6043387a4878e1082206eeba3f Mon Sep 17 00:00:00 2001 >From: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> >Date: Wed, 23 Jul 2014 08:41:47 +0300 >Subject: [PATCH] [SIGNED-OFF] Bug 12630 - Prioritizing "Hold starts on date" -holds causes all other holds to be prioritized as well! > >------------------------- >-- REPLICATE LIKE THIS -- >------------------------- > >0. Enable AllowHoldDateInFuture-system preference! > >1. Select a biblio with some holds. >2. Place a hold with the "Hold starts on date"-attribute set to future. >3. More the specific hold up on the priority queue. >4. Add another normal hold, observe how it is prioritized with the "Hold starts on date"-hold, leaving old holds to the prioritization queue tail. > >Unfair eh? > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > C4/Reserves.pm | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index a98eed4..779f554 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -163,7 +163,7 @@ sub AddReserve { > } > if ( C4::Context->preference( 'AllowHoldDateInFuture' ) ) { > # Make room in reserves for this before those of a later reserve date >- $priority = _ShiftPriorityByDateAndPriority( $biblionumber, $resdate, $priority ); >+ $priority = _ShiftPriorityByDateAndPriority( $biblionumber, $priority ); > } > my $waitingdate; > >@@ -1975,12 +1975,12 @@ the sub accounts for that too. > =cut > > sub _ShiftPriorityByDateAndPriority { >- my ( $biblio, $resdate, $new_priority ) = @_; >+ my ( $biblio, $new_priority ) = @_; > > my $dbh = C4::Context->dbh; >- my $query = "SELECT priority FROM reserves WHERE biblionumber = ? AND ( reservedate > ? OR priority > ? ) ORDER BY priority ASC LIMIT 1"; >+ my $query = "SELECT priority FROM reserves WHERE biblionumber = ? AND priority > ? ORDER BY priority ASC LIMIT 1"; > my $sth = $dbh->prepare( $query ); >- $sth->execute( $biblio, $resdate, $new_priority ); >+ $sth->execute( $biblio, $new_priority ); > my $min_priority = $sth->fetchrow; > # if no such matches are found, $new_priority remains as original value > $new_priority = $min_priority if ( $min_priority ); >-- >1.7.2.5
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 12630
:
29962
|
31140
|
31141
|
33238
|
33239
|
33240
|
135939
|
135940
|
135941
|
135942