Open mmalecki opened 12 years ago
Basically, basing on various REST resources (linked further down), we're not really doing it RESTfully.
PUT shouldn't be used for performing update operations (at least, not in the way resourceful does it - it merges updates with the existing document).
PUT
resourceful
PUT to /creatures/someId should result in:
/creatures/someId
I believe we could also use PATCH HTTP method for performing partial updates, but it introduces some risks related to concurrency.
PATCH
Resources:
Sounds right to me.
If you can make a branch with new test coverage that would cover any proposed changes, I can gladly make the core changes needed to implement your proposed API.
Basically, basing on various REST resources (linked further down), we're not really doing it RESTfully.
PUT
shouldn't be used for performing update operations (at least, not in the wayresourceful
does it - it merges updates with the existing document).PUT
to/creatures/someId
should result in:/creatures/someId
if it doesn't exist, with data supplied in the request/creatures/someId
if it existsI believe we could also use
PATCH
HTTP method for performing partial updates, but it introduces some risks related to concurrency.Resources: