Fix get with body in ClientStatic #4376

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@robertprice

Changed ClientStatic to correctly accept a body. Before it was trying
to call setBody which didn't exist. Unit test added.

This change may be controversial as there is debate as to wether GET's
should have a body or not. However, the documentation for this class
says it is supported, so this fixes that broken functionality.

@robertprice robertprice Fix get with body in ClientStatic
Changed ClientStatic to correctly accept a body. Before it was trying
to call setBody which didn't exist. Unit test added.

This change may be controversial as there is debate as to wether GET's
should have a body or not. However, the documentation for this class
says it is supported, so this fixes that broken functionality.
0d48cf2
@weierophinney weierophinney was assigned May 2, 2013
@weierophinney weierophinney added a commit that closed this pull request May 2, 2013
@weierophinney weierophinney Merge branch 'hotfix/4376'
Close #4376
ed9279a
@weierophinney weierophinney added a commit that referenced this pull request May 2, 2013
@weierophinney weierophinney Merge branch 'hotfix/4376' into develop
Forward port #4376
5e33a40
@weierophinney weierophinney added a commit to zendframework/zend-http that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#4376 from robertprice/…
…ClientStatic-GET-with-body

Fix get with body in ClientStatic
c22b4c5
@weierophinney weierophinney added a commit to zendframework/zend-http that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4376' 9ecbeb2
@weierophinney weierophinney added a commit to zendframework/zend-http that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4376' into develop b229d0b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment