Bug #30351

Can't use AOP with Repository

Added by Fernando Arconada almost 4 years ago. Updated almost 4 years ago.

Status:Resolved Start date:2011-09-27
Priority:Must have Due date:
Assigned To:Robert Lemke % Done:

100%
Category:AOP
Target version:TYPO3 Flow Base Distribution - 1.0.0
PHP Version: Complexity:
Has patch: Affected Flow version:

Description

If you write an aspect of a Repository and this Repository is injected via DI in your controller then FLOW3 fires an Exception when you load your controller

Proof of Concept package and exception screenshot in the next URL

http://goo.gl/ntfGm

It happens FLOW3 master commit 87863f3dd1db5786c03ba9c2ee4f91b7b718ec59

Associated revisions

Revision 3e36de83
Added by Robert Lemke almost 4 years ago

[BUGFIX] AOP proxies are no longer broken if __call() exists

If a class contains a magic __call() method, AOP proxy classes are
broken due to a is_callable() use – method_exists() in addition would
be correct.

Change-Id: Icf6bdf3f789162afbca61d7cf915dbb7ecd583d5
Fixes: #30351

History

#1 Updated by Robert Lemke almost 4 years ago

  • Project changed from TYPO3 Flow Base Distribution to TYPO3.Flow
  • Status changed from New to Accepted
  • Assigned To set to Robert Lemke

#2 Updated by Robert Lemke almost 4 years ago

  • Category set to AOP
  • Target version set to 1.0.0

#3 Updated by Mr. Hudson almost 4 years ago

  • Status changed from Accepted to Under Review

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

#4 Updated by Mr. Hudson almost 4 years ago

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

#5 Updated by Mr. Hudson almost 4 years ago

Patch set 3 of change Icf6bdf3f789162afbca61d7cf915dbb7ecd583d5 has been pushed to the review server.
It is available at http://review.typo3.org/5409

#6 Updated by Robert Lemke almost 4 years ago

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

Also available in: Atom PDF