Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

Some suggestion on DELETE Method #49

Open
snowwolf007cn opened this issue Jun 10, 2014 · 2 comments
Open

Some suggestion on DELETE Method #49

snowwolf007cn opened this issue Jun 10, 2014 · 2 comments

Comments

@snowwolf007cn
Copy link

  1. delete() and deleteList() method in Resource objects needs accept data from request body, since in such scenario, user need to give comments on why this action happens and provides other related information.
  2. I think it's better to return 202 Accepted as response for DELETE method, better than 204 no content.
@weierophinney
Copy link
Member

Agreed with (1); I'll work on that for either 1.0.3 or 1.1.0.

As for (2), 202 indicates the request has been accepted, but not processed. In the majority of cases, this is likely entirely incorrect, and 204 is the more accurate case (as there is no content returned, as the resource no longer exists to be represented).

@michalbundyra
Copy link
Member

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at laminas-api-tools/api-tools#68.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants