Attachment 42659 Details for Bug 14733 – Bug 14733 - Prevent a record from having holds with duplicate priorities

[patch] Bug 14733 - Prevent a record from having holds with duplicate priorities

Bug-14733---Prevent-a-record-from-having-holds-wit.patch (text/plain), 1.42 KB, created by Nicole C. Engard on 2015-09-17 12:55:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicole C. Engard

Created: 2015-09-17 12:55:54 UTC

Size: 1.42 KB

patch

obsolete

>From 64f5ae86fe1001e3d59f6bc8ffce5e9b603202b1 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 26 Aug 2015 11:43:08 -0400
>Subject: [PATCH] Bug 14733 - Prevent a record from having holds with
> duplicate priorities
>
>It is possible to create holds with duplicate priorities.
>
>The reason for this is that typically the priority is calculated before
>placing the hold. When the hold is placed the priority is calculated.
>This can easily be shown by opening up two browser windows and starting
>to place a hold for a record in each one. You'll see that both list the
>same priority. If you than place the hold in each window, both holds
>will have the same priority!
>
>Test Plan:
>1) Run unit tests pre-patch, note they fail
>2) Run unit tests post-patch, note they succeed
>
>Signed-off-by: Heather Braum <hbraum@nekls.org>
>---
> C4/Reserves.pm |    2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 5f0bf64..80ebf0a 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -193,6 +193,8 @@ sub AddReserve {
>     my $fee = GetReserveFee( $borrowernumber, $biblionumber );
>     ChargeReserveFee( $borrowernumber, $fee, $title );
> 
>+    _FixPriority({ biblionumber => $biblionumber});
>+
>     # Send e-mail to librarian if syspref is active
>     if(C4::Context->preference("emailLibrarianWhenHoldIsPlaced")){
>         my $borrower = C4::Members::GetMember(borrowernumber => $borrowernumber);
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 14733: 41996 | 41997 | 41998 | 42658 | 42659 | 42734 | 42735 | 42736 | 42880