Project

General

Profile

Actions

Bug #87838

closed

ClassSchema must analyze all property doc blocks

Added by Alexander Schnitzler over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
Extbase
Start date:
2019-01-17
Due date:
2019-01-17
% Done:

100%

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

Description

While introducing symfony/property-info with https://github.com/TYPO3/TYPO3.CMS/commit/bcde6c0177aba345b0bd363c4519892a132ef279, the analyzing of property doc blocks changed. Before, all properties had been analyzed but with that commit, the analyzing had been reduced to inject properties and collection properties. This however breaks the validation which is based on knowing the type of properties.


Related issues 1 (0 open1 closed)

Follows TYPO3 Core - Feature #87457: Use symfony/property-info to gather doc block informationClosedAlexander Schnitzler2019-01-16

Actions
Actions #1

Updated by Alexander Schnitzler over 5 years ago

  • Due date set to 2019-01-17
  • Start date changed from 2019-03-04 to 2019-01-17
  • Follows Feature #87457: Use symfony/property-info to gather doc block information added
Actions #2

Updated by Gerrit Code Review over 5 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/+/59846

Actions #3

Updated by Gerrit Code Review over 5 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/+/59846

Actions #4

Updated by Gerrit Code Review over 5 years ago

Patch set 3 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/+/59846

Actions #5

Updated by Anonymous over 5 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF