Bug 27004: Prevent claim resolution to fail
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 21 Dec 2020 10:05:26 +0000 (11:05 +0100)
committerVictor Grousset/tuxayo <victor@tuxayo.net>
Fri, 15 Jan 2021 01:02:38 +0000 (02:02 +0100)
commit29d4cfbdb8c448db81338d301733d5b13fa353ec
treebf33df8423c10bd1a52c72a986c96775e4b0471e
parent3c6577d239c6341e23152c75e102b8ffd0997797
Bug 27004: Prevent claim resolution to fail

If the account of the patron who created a claim is removed, then the
claim won't be resolvable.
Trying to resolve the claim will result in a 500:
[2020/12/21 10:05:55] [ERROR] PUT /api/v1/return_claims/1/resolve: unhandled exception (Koha::Exceptions::Checkouts::ReturnClaims::NoCreatedBy)<<[created_by is mandatory]>> Koha::REST::Plugin::Exceptions::__ANON

We should not check if created_by exist when we update an existing hold,
only when we create it.

Test plan:
- Login with patron A
- set a value to syspref ClaimReturnedLostValue
- Check an item out
- Claim returned it
- Login with patron B
- Delete patron A
- Resolve the claim
- ERROR in api-error.log an UI never confirming the resolution
- apply patch & restart services
- Resolve the claim
- it works

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 92dbe809021ea1f7ee44e51387b48d9ffbda1fea)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 8e3f33e5d6e79d2df1a5077e4dc68364867566c9)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
(cherry picked from commit a5c20fc188c634fc8c7b6d00f542dc7589d2e9a8)
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Koha/Checkouts/ReturnClaim.pm
t/db_dependent/api/v1/return_claims.t