Bug #30914

Proxy methods for methods with "by reference" return type are broken

Added by Andreas Förthner almost 4 years ago. Updated about 3 years ago.

Status:Resolved Start date:2011-10-14
Priority:Should have Due date:
Assigned To:Christian Müller % Done:

100%

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

Description

The proxy class builder ignores the by-reference & in the proxy method, and therefore you get an PHP fatal error.


Related issues

related to TYPO3.Flow - Bug #13333: AOP for methods with references do not work. Resolved 2011-02-24

Associated revisions

Revision fa086785
Added by Christian Müller over 3 years ago

[BUGFIX] Proxy methods with referenced arguments should work

Arguments given by reference would result in an exception if
the method was proxied due to AOP.

Change-Id: I647006669608a98a4fb9bc289d89c3c801502df1
Fixes: #30914
Fixes: #13333
Releases: 1.0,1.1

Revision fed005bf
Added by Christian Müller over 3 years ago

[BUGFIX] Proxy methods with referenced arguments should work

Arguments given by reference would result in an exception if
the method was proxied due to AOP.

Change-Id: I647006669608a98a4fb9bc289d89c3c801502df1
Fixes: #30914
Fixes: #13333
Releases: 1.0,1.1

History

#1 Updated by Karsten Dambekalns almost 4 years ago

  • Affected Flow version set to FLOW3 1.0.0

#2 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9651

#3 Updated by Christian Müller over 3 years ago

  • Assigned To set to Christian Müller
  • Has patch set to No

#4 Updated by Christian Müller over 3 years ago

  • Target version changed from 1.1 to 1.0.4

#5 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9651

#6 Updated by Christian Müller over 3 years ago

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

#7 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/10248

#8 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.0.4 to 1.0.5

#9 Updated by Karsten Dambekalns about 3 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF