Bug 29741: (follow-up) Make DELETE /patrons use the new validation method
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 20 Dec 2021 18:11:02 +0000 (15:11 -0300)
committerKyle Hall <kyle@bywatersolutions.com>
Fri, 7 Jan 2022 11:27:56 +0000 (06:27 -0500)
commit339d199be7ed303b30598ddb2d754698e01e0ddc
treebbda720dd22e7659fac793d9cce94632d3451dff
parent5a04907c54971ee2830ceb3827e020fc37c6bb01
Bug 29741: (follow-up) Make DELETE /patrons use the new validation method

This patch adapts the route so it uses the newly introduced
Koha::Patron->safe_to_delete method.

To test:
1. Run:
   $ kshell
  k$ prove t/db_dependent/api/v1/patrons.t
=> SUCCESS: Tests pass
2. Apply this patch
3. Repeat 1
=> SUCCESS: Tests still pass!
4. Sign off :-D

Note: There's a trivial behavior change, in which the 'anonymous patron'
use case is caugh eariler than the ->delete call. I left the exception
catch block just in case, who knows :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Koha/REST/V1/Patrons.pm
t/db_dependent/api/v1/patrons.t