Bug #31452

IgnoreValidation with argumentName does not work for ActionController

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

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

100%

Category:MVC
Target version:TYPO3 Flow Base Distribution - 1.0.1
PHP Version: Complexity:
Has patch:No Affected Flow version:FLOW3 1.0.0

Description

The following code will not work as expected:

/**
 * @param \TYPO3\FLOW3\Tests\Functional\MVC\Fixtures\Controller\TestObjectArgument $argument
 * @FLOW3\IgnoreValidation(argumentName="$argument")
 */
public function showObjectArgumentAction(TestObjectArgument $argument) {
    return $argument->getEmailAddress();
}

The ActionController still use getMethodTagsValues to get ignored arguments.


Related issues

related to TYPO3.Flow - Bug #31270: Proxy classes loose @return and @throws annotations Resolved 2011-10-25

Associated revisions

Revision 1c92bdd4
Added by Christopher Hlubek almost 4 years ago

[BUGFIX] Use annotations in ActionController for IgnoreValidation

Use annotations for the action method instead of method tags values
which return wrong results if an explicit argumentName is given
to the annotation. See the functional test that is added.

Change-Id: I1c6b2a24f452b313e541e14f47194e046629786a
Fixes: #31452
Releases: 1.0, 1.1

Revision 52d1e370
Added by Christopher Hlubek over 3 years ago

[BUGFIX] Use annotations in ActionController for IgnoreValidation

Use annotations for the action method instead of method tags values
which return wrong results if an explicit argumentName is given
to the annotation. See the functional test that is added.

Change-Id: I4662423f2bfa0c83bff13121a2d2928d4b0dd40b
Fixes: #31452
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 I1c6b2a24f452b313e541e14f47194e046629786a has been pushed to the review server.
It is available at http://review.typo3.org/6393

#2 Updated by Christopher Hlubek almost 4 years ago

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

Also available in: Atom PDF