From 039250924a1d68fe9f04444745399dd5b931f3bc 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

---
 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