TYPO3 Core

Hm sounds more like an endless loop.

So a cache is more a "side effect", it is more a loop detection mechanism.
Is there any prevention from validation loops at all?

Thinking about A.theB = B; B.theA = A;

Hello Markus,

I just found it. There is already a cache inside. The cache is only valid for the current CollectionValidator and not for the whole Domainmodel. Here an example:

Day1:
- Event1
- Event2
- Event3

Day2:
- Event1
- Event3
- Event5

Day3:
- Event1
- Event2
- Event4

So, for each day a new Cache will be created. That means, that Event1 will be validated 3 times. Maybe this works good for 1:1 and 1:N, but not for N:M:

Event1:
Day1:
- Event1
---Day1:
---- Event1
-----Day1-3 again
---- Event2
---- Event3
---Day2:
---- Event1
-----Day1-3 again
---- Event3
---- Event5
---Day3:
---- Event1
-----Day1-3 again
---- Event2
---- Event4
-- Event2
-- Event3
Day2:
- Event1
---Day1:
---- Event1
-----Day1-3 again
---- Event2
---- Event3
---Day2:
---- Event1
-----Day1-3 again
---- Event3
---- Event5
---Day3:
---- Event1
---- Event2
---- Event4
-- Event3
-- Event5
Day3:
- Event1
---Day1:
---- Event1
-----Day1-3 again
---- Event2
---- Event3
---Day2:
---- Event1
---- Event3
---- Event5
---Day3:
---- Event1
---- Event2
---- Event4
-- Event2
-- Event4

Issue #53816 helped me a little bit. Defining event.days as ObjectStorage and day.events as LazyObjectStorage stops recursive endless validation.

What do you think to implement a "stopRecursion"-Annotation for Domainmodel-Properties?

Stefan

Hi Stefan.
We are aware of these problems as Nico de Haen already spent a lot of time with this issue.

I talked to Anja to have another code sprint just for fixing these validation issues in january.

Most probably we will just join the sprint that takes place with all active contributors. Personally I really struggle with analysing the validation api beside my daily work, so I like the idea of letting this issue rest until we find a weekend to handle this issue together.

sounds like related to https://jira.typo3.org/browse/FLOW-17 Persistence: object tree validation performance is really slow