Project

General

Profile

Actions

Bug #89011

closed

InvalidArgumentException due to ActionController property annotation

Added by Sebastian Fischer over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Must have
Category:
Extbase
Start date:
2019-08-25
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

In the ActionController a property annotation causes an InvalidArgumentException in \phpDocumentor\Reflection\Fqsen::__construct. By removing the semicolon at the end of HashService this exception does not appear anymore.

/**
 * @var HashService;
*/
protected $hashService;
Actions #1

Updated by Alexander Schnitzler over 4 years ago

  • Status changed from New to Accepted
  • Assignee set to Alexander Schnitzler
  • Priority changed from Should have to Must have
  • Is Regression set to Yes
Actions #2

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61556

Actions #3

Updated by Benjamin Franzke over 4 years ago

Fully agree, that this is a bug, but one question, can you provide a backtrace?

I don't see where this Exception is actually triggered in core.
The InvalidArgument Exception is catched in https://github.com/TYPO3/TYPO3.CMS/blob/a73ba8f9f8ecd75d7fff59dfa992262ba0db49d2/typo3/sysext/extbase/Classes/Reflection/PropertyInfo/Extractor/PhpDocPropertyTypeExtractor.php#L117

Does this Exception happen in regular ActionController dispatches, or do you use phpDocumentor/reflection directly in own code?

Actions #4

Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61556

Actions #5

Updated by Anonymous over 4 years ago

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

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF