From b86143b7b8144eaa6d0a42e512935e78ed7425e0 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 19 Jan 2016 14:16:22 +0000
Subject: [PATCH]  Bug 14310 [QA Follow] - Remove FIXME, add explanation
 comment instead

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
---
 C4/Reserves.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/C4/Reserves.pm b/C4/Reserves.pm
index 8794ea8657..f981be1afb 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.39.5