Jonathan Druart
92dbe80902
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> |
||
---|---|---|
.. | ||
acquisitions_funds.t | ||
acquisitions_orders.t | ||
acquisitions_vendors.t | ||
advanced_editor_macros.t | ||
auth.t | ||
auth_authenticate_api_request.t | ||
auth_basic.t | ||
biblios.t | ||
checkouts.t | ||
cities.t | ||
clubs_holds.t | ||
holds.t | ||
illrequests.t | ||
import_batch_profiles.t | ||
items.t | ||
libraries.t | ||
oauth.t | ||
patrons.t | ||
patrons_accounts.t | ||
patrons_password.t | ||
return_claims.t | ||
smtp_servers.t | ||
stockrotationstage.t |