Bug #87773

ClassSchema doesn't treat unsignated classes anymore

Added by Anja Leichsenring 3 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2019-02-24
Due date:
% Done:

0%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

with #87632, the default value from ClassSchema was changed from Entity to zero, but this causes ClassSchema to not treat disguised ValueObjects anymore.

With the change applied, TYPO3\CMS\Belog\Domain\Model\Constraint is not handled anymore and causes the BE Log Module to fail with

InvalidArgumentException
There is no @var annotation for property "userOrGroup" in class "TYPO3\CMS\Belog\Domain\Model\Constraint".

Setting the default value in ClassSchema to 1 (Value Object) also omits the expensive followup relation resolving, but brings back the class schema resolving for classes not explicitely extending AbstractValueObject.


Related issues

Related to TYPO3 Core - Bug #87632: Do not treat all classes as entities Closed 2019-02-04

History

#1 Updated by Anja Leichsenring 3 months ago

  • Related to Bug #87632: Do not treat all classes as entities added

#2 Updated by Gerrit Code Review 3 months ago

  • Status changed from New 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/+/59793

#3 Updated by Gerrit Code Review 3 months 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/+/59793

#4 Updated by Gerrit Code Review 3 months 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/+/59793

#5 Updated by Susanne Moog 3 months ago

  • Status changed from Under Review to Closed

Patch was abandoned as original patch was reverted.

#6 Updated by Riccardo De Contardi 3 months ago

  • Assignee deleted (Anja Leichsenring)

Also available in: Atom PDF