Thanks to visit codestin.com
Credit goes to github.com

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

Allow multidimensional params in AbstractControllerTestCase#6637

Closed
ahoppen wants to merge 2 commits into
zendframework:masterfrom
ahoppen:zfBugfix6636
Closed

Allow multidimensional params in AbstractControllerTestCase#6637
ahoppen wants to merge 2 commits into
zendframework:masterfrom
ahoppen:zfBugfix6636

Conversation

@ahoppen
Copy link
Copy Markdown
Contributor

@ahoppen ahoppen commented Sep 4, 2014

Fixes bug #6636

Added test method testCanHandleMultidimensionalParams to \ZendTest\Test\PHPUnit\Controller\AbstractControllerTestCaseTest that isolates the issue
Fixed the bug by using http_build_query instead of the self-written code to generate the Request-Content

weierophinney and others added 2 commits August 20, 2014 08:18
…al params array

Added test method testCanHandleMultidimensionalParams to \ZendTest\Test\PHPUnit\Controller\AbstractControllerTestCaseTest that isolates the issue
Fixed the bug by using http_build_query instead of the self-written code to generate the Request-Content
@Ocramius Ocramius added this to the 2.3.4 milestone Dec 24, 2014
@Ocramius Ocramius self-assigned this Dec 24, 2014
@Ocramius Ocramius closed this in 247c567 Dec 30, 2014
Ocramius added a commit that referenced this pull request Dec 30, 2014
…t-multidimensional-urlencoded-params' into develop

Close #6636
Close #6637
Forward port #6636
Forward port #6637
@Ocramius
Copy link
Copy Markdown
Member

@ahoppen rebased and merged, thanks!

master: 247c567
develop: 0a3cc00

gianarb pushed a commit to zendframework/zend-test that referenced this pull request May 15, 2015
…dding `@group` annotations for newly introduced tests
gianarb pushed a commit to zendframework/zend-test that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-test that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants