[Koha-bugs] [Bug 27330] Wrong return status when no enrollments in club holds

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Jan 4 15:37:19 CET 2021


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27330

--- Comment #3 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
Created attachment 114800
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114800&action=edit
Bug 27330: (follow-up) Cleanup missed on bug 23843

This patch makes some cleanup missed on bug 23843. The catch condition
on the controller was copied and pasted from other controller and should
be avoided unless there's a known case that needs special handling.
Otherwise it will be catched by the $c->unhandled_exception call.

All the old mappings were inadvertedly kept in the controller as well as
the (unused) _to_api method.

The base class is also cleaned from unnecessary mappings. Only mapped
things need to be added.

To test:
1. Apply this patch
2. Run:
   $ kshell
  k$ prove t/db_dependent/api/v1/clubs_holds.t \
           t/db_dependent/Koha/Club/Hold.t
=> SUCCESS: Tests pass!
3. Sign off :-D

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.


More information about the Koha-bugs mailing list