Revisit update route - Githubissues

senaite / senaite.jsonapi

RESTful JSON API for SENAITE

https://www.senaite.com

GNU General Public License v2.0

11 stars 19 forks source link

Revisit update route #40

Closed xispa closed 4 years ago

xispa commented 4 years ago

Description of the issue/feature this PR addresses

This PR reviews the behavior of update route and adds doctests as well.

The following has been fixed:

Objects were not updated when only the route update was used (without neither resource nor uid. The reason is that the route /<string:resource> was masking the route /<any(" + ACTIONS + "):action>. https://github.com/senaite/senaite.jsonapi/commit/0e5377c02eb2b17f7beca1bc818d228060853bbb

The following has been added:

IUpdate for custom update adapters

And the following restrictions have been added:

A 401 Unauthorized error is rised when trying to update an object from inside portal root.
A 401 Unauthorized error is rised when trying to update an object from inside setup folder
Do never update field id

-- I confirm I have tested the PR thoroughly and coded it according to PEP8 standards.