From 3908248f58f105c15eba34b4018d00ccfe78b6aa Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 13 Jul 2018 14:45:55 +0000 Subject: [PATCH] Bug 21075: AutoUnsuspendHolds should unsuspend holds <= today To recreate: 1 - Place a hold in your system 2 - Set that hold (or all holds) to expire today update reserves set suspend_until=CONCAT( CURDATE()," 00:00:00"); 3 - Run misc/cronjobs/holds/auto_unsuspend_holds.p 4 - Note the hold is sitll suspended 5 - Visit /cgi-bin/koha/reserve/request.pl for the book with the hold 6 - Note the suspend date is tomorrow (and cannot be set to today] 7 - Click update holds - the date in db is now set to tomorrow 8 - Reset to today 9 - Apply patch 10 - Run the cron again 11 - Visit the page and note hold is unsuspended Signed-off-by: Katrin Fischer Signed-off-by: Nick Clemens (cherry picked from commit c0f7aed84d663029b2f1b37ce0391b5a40b10b97) Signed-off-by: Martin Renvoize (cherry picked from commit 9e67cff276297fe52e2f8c58ab559758ad5043f6) Signed-off-by: Fridolin Somers --- C4/Reserves.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index ff35f37806..9054771338 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -842,7 +842,7 @@ Unsuspends all suspended reserves with a suspend_until date from before today. sub AutoUnsuspendReserves { my $today = dt_from_string(); - my @holds = Koha::Holds->search( { suspend_until => { '<' => $today->ymd() } } ); + my @holds = Koha::Holds->search( { suspend_until => { '<=' => $today->ymd() } } ); map { $_->suspend(0)->suspend_until(undef)->store() } @holds; } -- 2.39.5