]> git.koha-community.org Git - koha.git/commit
Bug 38175: (QA follow-up) DRY out PATCH handling
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 17 Oct 2024 11:02:10 +0000 (12:02 +0100)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Thu, 24 Oct 2024 16:27:24 +0000 (18:27 +0200)
commite8185e14cc2b2ad9f28f3b34153342f46d107304
tree2b7de399359ea390424146c32902703a0842f880
parent35136c234d61f784b23e8fa01a5537deffb1c67d
Bug 38175: (QA follow-up) DRY out PATCH handling

We were repeating some code for the PATCH endpoint instead of using the
existing update handling code.  This lead to two issues:

1) We weren't catching status updates on standard updates, only the
   patch.
2) We were limiting the PATCH endpoint to just status updates when it
   could happily be used for other fields too.

This patch removes the introduction of the 'edit' method from both the
REST controller and the corresponding object and moves the logic into
the store method of the object where it's easily testable.

We also DRY out the notice sending code a little for re-use.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Koha/Booking.pm
Koha/Object.pm
Koha/REST/V1/Bookings.pm
api/v1/swagger/definitions/booking_patch.yaml [new file with mode: 0644]
api/v1/swagger/paths/bookings.yaml
api/v1/swagger/swagger.yaml
t/db_dependent/Koha/Booking.t
t/db_dependent/api/v1/bookings.t