Change 47369 killed features in flux
Updated by Wouter Wolters over 7 years ago
- Status changed from New to Needs Feedback
Did you read what the exception message contains?
Nesting type=inline in container sections is not supported. Throw an exception if configured.
Nesting sections is not supported. Throw an exception if configured.
Nesting flex form sections or inline within section containers is not supported.
To not let developers run into hard to track issues the patch adds an exception to detect these cases.
Additionally, GroupElement contains a preparation that is done by TcaGroup data provider already,
so this part is cleaned up along the way since this is unreachable code if FormEngine is used without nested sections.
Updated by Christian Kuhn over 7 years ago
- Status changed from Needs Feedback to Rejected
Working with inline in section or with nested sections can result in various data integrity issues. Removing that exception would expose the system to that risk again. Core must stay on the safe side here.
This issue can only be closed at the moment.
Updated by Claus Due over 7 years ago
I agree with the decisions and Christian is completely correct about the data integrity issues (see every Flux issue concerning sorting of section objects). I would add that Flux contained some workarounds to avoid those issues but we fully accept the decision to block these structures and any others that may have similar problems.