-
Notifications
You must be signed in to change notification settings - Fork 214
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUGFIX] ContentTypeBuilder LTS #1333
[BUGFIX] ContentTypeBuilder LTS #1333
Conversation
@NamelessCoder What I just think: can we put the call to CompatibilityRegistry for the DEFAULT_SHOWITEM into an if condition with TYPO3_MODE === 'BE' ? |
3954a69
to
78afcf0
Compare
I have made an update which now checks for TYPO3_MODE === 'BE' |
Please wait: I'll better remove the TCA data from ext_localconf.php to reduce the memory consumption. It's better to use a compatibility class for the TCA data with CompatibilityRegistry. |
78afcf0
to
20322ff
Compare
@NamelessCoder maybe we can combine the same TCA configuration in a separate class and can use it for fluidcontent and flux. It's redundant data. |
20322ff
to
62ed8b7
Compare
I changed to a public static property for |
Or should we use |
If possible use addBoilerplateTableConfiguration - also if that means we nedd to add an argument for example. |
16c1f81
to
ad751bc
Compare
@NamelessCoder Hi Claus. $GLOBALS['TCA']['tt_content']['columns']['pi_flexform']['ds']['*,' . $contentType] = []; had no effect all the time, because it should be $GLOBALS['TCA']['tt_content']['columns']['pi_flexform']['config']['ds']['*,' . $contentType] = []; but then the empty array is wrong I have removed it and now 'default' => '
<T3DataStructure>
<ROOT>
<type>array</type>
<el>
<!-- Repeat an element like "xmlTitle" beneath for as many elements you like. Remember to name them uniquely -->
<xmlTitle>
<TCEforms>
<label>The Title:</label>
<config>
<type>input</type>
<size>48</size>
</config>
</TCEforms>
</xmlTitle>
</el>
</ROOT>
</T3DataStructure>
' |
You can skip the new class with static properties containing TCA (generally speaking, this pattern is not desired; metadata should not be configured in a class). Instead, use the CompatibilityRegistry in the TCA override file for tt_content and apply the TCA there as well. The result then gets cached which is fine in this case (better than not caching, really). |
ad751bc
to
6067fef
Compare
Now the CompatibilityRegistry is used in TCA override file for tt_content. |
Using CompatibilityRegistry in ext_localconf for defaultShowItem for TCA generation.
6067fef
to
3e2e029
Compare
move showItem configuration into ext_localconf.php - using CompatibilityRegistry
convert arrays in ext_localconf.php into short arrays