From 039250924a1d68fe9f04444745399dd5b931f3bc Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 19 Jan 2016 14:16:22 +0000 Subject: [PATCH] Bug 14310 [QA Follow] - Remove FIXME, add explanation comment instead --- C4/Reserves.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 8794ea8..f981be1 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -1176,8 +1176,8 @@ sub ModReserve { $suspend_until = eval { dt_from_string( $suspend_until ) }; $hold->suspend_hold( $suspend_until ); } else { - # FIXME: Why are we doing this? Should this be resuming the hold, - # or maybe suspending it indefinitely? + # If the hold is suspended leave the hold suspended, but convert it to an indefinite hold. + # If the hold is not suspended, this does nothing. $hold->set( { suspend_until => undef } )->store(); } } -- 2.1.4