[Koha-bugs] [Bug 21765] AutoUnsuspendReserves manually sets holds fields instead of calling ->resume
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Mon Feb 11 20:10:03 CET 2019
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21765
--- Comment #3 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
Created attachment 85000
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85000&action=edit
Bug 21765: Make AutoUnsuspendReserves call Koha::Hold->resume
This patch makes the mentioned method use the biult-in method for
resuming a suspended hold, instead of manually setting the specific
attributes using the accessors.
The side effect for this is that HoldsLog is now honoured.
This patchset also refactors the tests a bit so they rely on the exposed
methods and thus don't rely on specific implementation details (like
suspended hold means a date in suspended_until plus suspended=1).
To test:
- Apply the regression tests patch
- Run:
$ kshell
k$ prove t/db_dependent/Reserves/AutoUnsuspendReserves.t
=> FAIL: It fails loudly because things are done manually
- Apply this patch
- Run:
k$ prove t/db_dependent/Reserves/AutoUnsuspendReserves.t
=> SUCCESS: Tests pass!
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list