https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38253 --- Comment #25 from Johanna Räisä <johanna.raisa@koha-suomi.fi> --- (In reply to Tomás Cohen Arazi (tcohen) from comment #23)
(In reply to Tomás Cohen Arazi (tcohen) from comment #3)
I don't think having a verb in the resource path is RESTful, and we should avoid it for consistency.
This can be solved by getting rid of the 'toggle' nature of the route and the method used. For example:
``` PUT /holds/{hold_id}/lowest_priority true ```
or
``` PUT /holds/{hold_id}/lowest_priority false ```
This is not what's been implemented. And it should.
I made it similar as in request.pl. It uses C4::Reserves::ToggleLowestPriority($hold_id), so only the hold_id is needed. I can change it so the boolean value is required if that is the case. -- You are receiving this mail because: You are watching all bug changes.