https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38253
--- Comment #25 from Johanna Räisä <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
