View | Details | Raw Unified | Return to bug 14310
Collapse All | Expand All

(-)a/C4/Reserves.pm (-3 / +2 lines)
Lines 1176-1183 sub ModReserve { Link Here
1176
                $suspend_until = eval { dt_from_string( $suspend_until ) };
1176
                $suspend_until = eval { dt_from_string( $suspend_until ) };
1177
                $hold->suspend_hold( $suspend_until );
1177
                $hold->suspend_hold( $suspend_until );
1178
            } else {
1178
            } else {
1179
                # FIXME: Why are we doing this? Should this be resuming the hold,
1179
                # If the hold is suspended leave the hold suspended, but convert it to an indefinite hold.
1180
                # or maybe suspending it indefinitely?
1180
                # If the hold is not suspended, this does nothing.
1181
                $hold->set( { suspend_until => undef } )->store();
1181
                $hold->set( { suspend_until => undef } )->store();
1182
            }
1182
            }
1183
        }
1183
        }
1184
- 

Return to bug 14310