Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

$resource obj.$delete sends the resource as the request body #4280

Closed
bourey opened this issue Oct 4, 2013 · 0 comments
Closed

$resource obj.$delete sends the resource as the request body #4280

bourey opened this issue Oct 4, 2013 · 0 comments

Comments

@bourey
Copy link
Contributor

bourey commented Oct 4, 2013

Angular's $resource delete method sends the resource as the request body. This behavior is a change from angular 1.0.x, and appears to contradict the intention articulated in #3207.

This issues specifically affects the obj.$delete instance action and is not demonstrated via $resource.delete.

bourey added a commit to bourey/angular.js that referenced this issue Oct 4, 2013
Prevent the  obj.$delete instance method from sending the resource as the request body.  This commit uses the existing hasBody boolean to only set httpConfig.data for methods which should have a request body.

Closes angular#4280
jeffbcross pushed a commit to jeffbcross/angular.js that referenced this issue Oct 4, 2013
Prevent the  obj.$delete instance method from sending the resource as the request body.  This commit uses the existing hasBody boolean to only set httpConfig.data for methods which should have a request body.

Closes angular#4280
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this issue Jan 27, 2014
Prevent the  obj.$delete instance method from sending the resource as the request body.  This commit uses the existing hasBody boolean to only set httpConfig.data for methods which should have a request body.

Closes angular#4280
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this issue Jan 27, 2014
Prevent the  obj.$delete instance method from sending the resource as the request body.  This commit uses the existing hasBody boolean to only set httpConfig.data for methods which should have a request body.

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

Successfully merging a pull request may close this issue.

1 participant