baugarten/node-restful

Patch method support

Satak opened this issue · 5 comments

Satak commented

Does node-restful support 'Patch' REST method?

stickwithjosh commented

Sadly, it looks like that's not yet implemented:

node-restful/lib/model.js

Line 8 in d5277ae

var methods = ['get', 'post', 'put', 'delete'], // All HTTP methods, PATCH not currently supported

baugarten commented

This is true

camilomontoyau commented

But if you only send the property you want to change via put, it only chages it and persist the other ones

OmgImAlexis commented

Currently PATCH is supported but it's called PUT, PUT should actually override the whole object with the new one that's provided whereas PATCH should only change the field you provide. It looks like @baugarten got these backwards.

OmgImAlexis commented

If you guys want to try this out I've opened a PR to possibly fix this.
Ref: #152