From 5fc0e13c57fd4f04c3c95ac51b5cca39d58aa856 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatetsolutions.com>
Date: Thu, 10 May 2018 08:12:04 -0400
Subject: [PATCH] Bug 20724 [Alternate]: Try to fix ReservesNeedReturns wrong
 behaviour

We need to call Koha::Hold->set_waiting to correctly
calculate the expiration date.

It fixes a regression introduced by
  commit 26634151dbce0bf39ff6c9eda5b58bb993996a2d
  Bug 12063 - Fix QA failures

The expiration date has to be set for waiting holds
---
 C4/Reserves.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/C4/Reserves.pm b/C4/Reserves.pm
index 73b86b606c..a919f7581c 100644
--- a/C4/Reserves.pm
+++ b/C4/Reserves.pm
@@ -203,6 +203,7 @@ sub AddReserve {
             itemtype       => $itemtype,
         }
     )->store();
+    $hold->set_waiting() if $found eq 'W';
 
     logaction( 'HOLDS', 'CREATE', $hold->id, Dumper($hold->unblessed) )
         if C4::Context->preference('HoldsLog');
-- 
2.15.1 (Apple Git-101)