Bug #31134

Functional tests of controllers don't reset arguments

Added by Christopher Hlubek almost 4 years ago. Updated over 3 years ago.

Status:Resolved Start date:2011-10-20
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:MVC
Target version:-
PHP Version: Complexity:
Has patch: Affected Flow version:FLOW3 1.0.0

Description

Using sendWebRequest to one controller with two different arguments arrays will not reset the array in the controller.

Not tested against current trunk but I'm quite sure it won't work there too.

Associated revisions

Revision a3201354
Added by Christopher Hlubek over 3 years ago

[BUGFIX] Reset argument list before initializing action arguments

This change fixes a problem with functional tests of controller
actions. It's also needed to reset the state of the controller when
multiple requests are dispatched.

It also refactors a unit test to use the actual arguments class instead
of a mock which is more expressive.

Change-Id: Ia7579436acf258b61f98c9b1e69ee78fe65b0cbf
Fixes: #31134
Releases: 1.0, 1.1

Revision 4cafbe10
Added by Christopher Hlubek over 3 years ago

[BUGFIX] Reset argument list before initializing action arguments

This change fixes a problem with functional tests of controller
actions. It's also needed to reset the state of the controller when
multiple requests are dispatched.

It also refactors a unit test to use the actual arguments class instead
of a mock which is more expressive.

Change-Id: I861e3d75e0dff43738cd4275ce142de797bb87d9
Fixes: #31134
Releases: 1.0, 1.1

History

#1 Updated by Mr. Hudson almost 4 years ago

  • Status changed from New to Under Review

Patch set 1 of change Ia7579436acf258b61f98c9b1e69ee78fe65b0cbf has been pushed to the review server.
It is available at http://review.typo3.org/6119

#2 Updated by Karsten Dambekalns almost 4 years ago

  • Affected Flow version set to FLOW3 1.0.0

#3 Updated by Mr. Hudson over 3 years ago

Patch set 2 of change Ia7579436acf258b61f98c9b1e69ee78fe65b0cbf has been pushed to the review server.
It is available at http://review.typo3.org/6119

#4 Updated by Christopher Hlubek over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF