Bug #31042

Throwing an exception throws an exception....

Added by Adrian Föder almost 4 years ago. Updated almost 4 years ago.

Status:Resolved Start date:2011-10-18
Priority:Must have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

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

Description

after running the master branch intended to become RC1, I have Uncaught Exception in FLOW3 #1: Warning: current() expects parameter 1 to be array, null given in [...]Packages\Framework\TYPO3.FLOW3\Classes\Reflection\ReflectionService.php line 932. The mentioned line is

throw new \TYPO3\FLOW3\Reflection\Exception(sprintf('Classes tagged as entity or value object must be of scope prototype, however, %s is declared as %s.', $className, current($classTagValues)), 1264103349);

In fact, the $classTagValues variable is not set in this function.

Associated revisions

Revision 9c486183
Added by Karsten Dambekalns almost 4 years ago

[BUGFIX] Exception when entities/valueobjects are not prototype

Change-Id: I936356d0c19fafbf3051cd10a9f92d97e64c4974
Fixes: #31042

History

#1 Updated by Adrian Föder almost 4 years ago

..maybe the current() invocation should be replaced with $scopeAnnotation->value, so that the whole block should read

if ($scopeAnnotation !== NULL && $scopeAnnotation->value !== 'prototype') {
    throw new \TYPO3\FLOW3\Reflection\Exception(sprintf('Classes tagged as entity or value object must be of scope prototype, however, %s is declared as %s.', $className, $scopeAnnotation->value), 1264103349);
}

#2 Updated by Karsten Dambekalns almost 4 years ago

  • Status changed from New to Accepted
  • Assigned To set to Karsten Dambekalns

#3 Updated by Mr. Hudson almost 4 years ago

  • Status changed from Accepted to Under Review

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

#4 Updated by Mr. Hudson almost 4 years ago

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

#5 Updated by Karsten Dambekalns almost 4 years ago

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

Also available in: Atom PDF