Bug 30500: (QA follow-up) Separate code execution streams for obviously independent...
authorPetro Vashchuk <stalkernoid@gmail.com>
Fri, 5 Aug 2022 12:47:41 +0000 (15:47 +0300)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 5 Aug 2022 14:54:55 +0000 (11:54 -0300)
commit3a814ff573f9491156c252fa3b2b6bcb6b82a461
tree3adce4d7f87c1d60bbf640a97c90de3446750176
parent6215f9a0e17444d3178b85cfcc1df33fd4644e8f
Bug 30500: (QA follow-up) Separate code execution streams for obviously independent parameters

When $new_pickup_location gets passed as parameter it doesn't make any
sense to also check is_cancelable_from_opac.

Also this makes code more clearer as it seemed like it was possible for
item to get cancelled at the same time as request to change pickup
location.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
opac/opac-modrequest.pl