Bug ID: 21478
Summary: Koha::Hold->suspend_hold allows suspending in transit
Change sponsored?: ---
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: [hidden email] Reporter: [hidden email] QA Contact: [hidden email]
My understanding is that both 'waiting' and 'in transit' holds are 'found', and
in both cases suspending them should be disallowed. I haven't looked at the
places in which this could happen, but the code implementing the checks in
Koha::Hold clearly misses the use case.
The method should be renamed to the new guidelines (->suspend) and it should
raise exceptions in both cases.