Bug #58004
closedCategories not visible
Added by Philipp Wrann over 10 years ago. Updated almost 6 years ago.
100%
Description
Since last update i cant see the categories tree anymore.
I added the TCA definitions like this:
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable
Files
example.zip (2.2 KB) example.zip | example to reproduce | Philipp Wrann, 2014-04-17 15:47 |
Updated by Philipp Wrann over 10 years ago
This is the concrete call in ext_tables.php:
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable($_EXTKEY,'tx_regionalobject_domain_model_regionalobject','categories');
the field exists in ext_tables.sql and of course in the database
Updated by Philipp Wrann over 10 years ago
I think the TCA definitions are just built too early.
Updated by Stefan Neufeind over 10 years ago
He meant: works with 6.2.0, broke since 6.2.1. Might be related to the introduction of TCA-overrides (#57942) though that should not break existing things.
Updated by Ernesto Baschny over 10 years ago
- Is Regression changed from No to Yes
Updated by Philipp Wrann over 10 years ago
- File example.zip example.zip added
I added the TCA definitions like in the example attached.
Updated by Andreas Allacher over 10 years ago
Same problem. Most likely related to overrides.
Because from Bootstrap->executeExtTablesAdditionalFile() the applyTca() is gone.
It is now done in applyTcaOverrides.
However instead of using
\TYPO3\CMS\Core\Category\CategoryRegistry::getInstance()
The registry is injected and is therefore a different instance then the one where makeCategorizable adds the category information.
Updated by Stefan Neufeind over 10 years ago
CategoryRegistry is a singleton. So I guess you should receive the same instance.
Updated by Helmut Hummel over 10 years ago
- Status changed from Accepted to Needs Feedback
Hey guys, thanks for your feedback and sorry for any inconvenience caused by this change.
However, calling \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable in ext_tables.php is wrong and instead the call should be made in ext_localconf.php
It is true nonetheless that having it in ext_tables.php worked in TYPO3 6.1 and also 6.2.0
It worked because CategoryRegistry->applyTca was called so late in the bootstrap, that it was after ext_tables.php files were included and after the extTables.php file was read.
That meant that the resulting TCA could related to categories could not be modified at all
Additionally, adding to the registry and applying the TCA changes of the registry could not be cached, although the result was/is pretty "static"
The thing is, that the design of \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable changed since TYPO3 6.1 to a pure registry with no side effects when adding categorization requests. Such API is not something to be placed in ext_tables.php because in this file direct manipulation of global state (e.g. TCA) is required to be effective. Since \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable does not manipulate the global TCA. Thus, such call has to go into ext_localconf.php. It has been made to work by putting it late into the bootstrap, but that had the downsides described above.
My recommendation: put the \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable call in ext_localconf.php as this will work in 6.1.x and 6.2.0 if you need backwards compatibility (you can also add it to ext_tables.php if you do not care about logged warnings in 6.1.x or wrap it in a condition to only call if version lower than 6.1).
If you want the benefit of caching and do not need backwards compatibility to 6.1, then put it in a *.php file (choose a name as you like) in <extKey>/Configuration/TCA/Overrides/ (see #57951)
Let me elaborate on the current order of execution during bootstrap:
- ext_localconf.php files from all extensions are read
- <extKey>/Configuration/TCA/<table_name>.php files are read and put into TCA[<table_name>]
- <extKey>/Configuration/TCA/Overrides/<name>.php are read (API calls in there are meant to manipulate TCA and nothing else is useful as these files are only included when TCA is not cached)
- CategoryRegistry changes are applied
- Complete TCA is cached
- ext_tables.php files from all extensions are read (Here still TCA can be manipulate for BC reasons or e.g. for changing category registry related things)
- extTables.php
I post here my suggestions already made on https://review.typo3.org/29484 If you have another option or proposal what we should do in this area please add your comments:
This is a limitation in the category registry design. We can do the following:- Apply category registry TCA before override TCA from extensions are included. Then the makeCategorizable calls from extension can only work in ext_localconf.php files and are executed in every BE and FE request.
- Kick the registry design and apply TCA changes on every ->add call directly and apply default core categorization before TCA overrides are included. by doing so makeCategorizable must not be placed in ext_localconf.php files as this would not work any more, but default TCA can be overridden and still cached.
- Leave it like that, as it is better than before (except that now makeCategorizable will not work in ext_tables.php and extTables.php any more, but only in TCA overrides and ext_localconf.php files)
- any other option?
Updated by Philipp Wrann over 10 years ago
Just my thoughts on that:
A functional change between one big release and a patch (eg. 6.2.0 -> 6.2.1) should never take place, mark it as deprecated and use the new approach in 6.3
If someone uses the new approach the benefits will be given i think, so ensure backwards compatibility within version 6.2. All Extensions that work with 6.2.0 should also work with 6.2.*.
Splitting up TCA definitions in 2 files anyway is bad, (i hated that weird dynamicConfigFile bullshit too) especially for new developers.
Maybe it would be best to have something like that in your tables TCA:
$TCA['table']['columns']['categories']['config'] = ExtensionManagementUtility::getCategoryFieldTCAConfig('table','categories',array());
Like you can use:
ExtensionManagementUtility::getFileFieldTCAConfig
Updated by Helmut Hummel over 10 years ago
Philipp Wrann wrote:
A functional change between one big release and a patch (eg. 6.2.0 -> 6.2.1) should never take place, mark it as deprecated and use the new approach in 6.3
If someone uses the new approach the benefits will be given i think, so ensure backwards compatibility within version 6.2. All Extensions that work with 6.2.0 should also work with 6.2.*.
I totally agree in general, but in this case, how should it have been done?
The category fields added were not changeable because applyTca was executed after ext_tables.php, if we move applyTca before ext_tables.php then we could change category fields there again, but cannot add them in ext_tables.php because TCA is already applied. See the chicken/egg problem here?
Would be great if you had an idea how to theoretically fix that without breaking something, then I'm happy to implement it.
This really is a chicken/egg problem: applyTca before ext_tables.php then you cannot register any more, applyTca after ext_tables.php then you cannot modify the applied TCA any more.
So where should we put it without breaking something?
Splitting up TCA definitions in 2 files anyway is bad, (i hated that weird dynamicConfigFile bullshit too) especially for new developers.
Maybe it would be best to have something like that in your tables TCA:$TCA['table']['columns']['categories']['config'] = ExtensionManagementUtility::getCategoryFieldTCAConfig('table','categories',array());
Like you can use:
ExtensionManagementUtility::getFileFieldTCAConfig
Yes. This is what should have been done, it hasn't because the goal has been to also create the database field automatically (I guess).
Nevertheless such API would be good to have in exchange (or in addition) to the current API.
Updated by Helmut Hummel over 10 years ago
Helmut Hummel wrote:
Splitting up TCA definitions in 2 files anyway is bad, (i hated that weird dynamicConfigFile bullshit too) especially for new developers.
Maybe it would be best to have something like that in your tables TCA:$TCA['table']['columns']['categories']['config'] = ExtensionManagementUtility::getCategoryFieldTCAConfig('table','categories',array());
Like you can use:
ExtensionManagementUtility::getFileFieldTCAConfigYes. This is what should have been done, it hasn't because the goal has been to also create the database field automatically (I guess).
Nevertheless such API would be good to have in exchange (or in addition) to the current API.
Ok, thought about this again. The scope of something like ExtensionManagementUtility::getCategoryFieldTCAConfig and ExtensionManagementUtility::makeCategorizable is really different.
If you introduce a new table and want to have it categorizable, then just write the TCA for it (currently no support for you in that regard) and put it into Configuration/TCA/<table_name>.php and you are set.
Do not use ExtensionManagementUtility::makeCategorizable when introducing tables yourself.
ExtensionManagementUtility::makeCategorizable fits perfectly to easily add a category field to an existing table. This can be done in ext_localconf.php and /Configuration/TCA/Overrides/
However I would be fine introducing ExtensionManagementUtility::getCategoryFieldTCAConfig in 6.2.x if somebody wants to do this.
Updated by Stefan Neufeind over 10 years ago
When I write something that should be categorizable maybe I don't want to think about what TCA makes it categorizable but just use the function for that? That's also how extension-builder (soon to be released version) does it - independent of whether it's an existing table or if you just defined it in the very same class.
I guess you mean it just won't work? So if you define your own TCA and want to use makeCategorizable, would the makeCategorizable() then need to go into a separate TCA-overrides-file (inside the same extension) to make it work?
Updated by Philipp Gampe over 10 years ago
Ok, thought about this again. The scope of something like ExtensionManagementUtility::getCategoryFieldTCAConfig and ExtensionManagementUtility::makeCategorizable is really different.
If you introduce a new table and want to have it categorizable, then just write the TCA for it (currently no support for you in that regard) and put it into Configuration/TCA/<table_name>.php and you are set.
I agree. I think we need both functions. One for adding TCA to an existing table and one to help extension authors to provide category fields themselves as part of their extension (e.g. tx_news).
Updated by Helmut Hummel over 10 years ago
Stefan Neufeind wrote:
When I write something that should be categorizable maybe I don't want to think about what TCA makes it categorizable but just use the function for that?
True, but this never worked.
That's also how extension-builder (soon to be released version) does it - independent of whether it's an existing table or if you just defined it in the very same class.
We need to fix the core first before we can say what code the extension builder should create.
I guess you mean it just won't work? So if you define your own TCA and want to use makeCategorizable, would the makeCategorizable() then need to go into a separate TCA-overrides-file (inside the same extension) to make it work?
Yes. Introducing a new table and at the same time making it categorizable never worked.
Updated by Helmut Hummel over 10 years ago
[EDIT] Behavior also changed from 6.1.0 to 6.1.1
Here a matrix that will help on changes in that area. Any change should add more + than we had before.
Judging the change introduced in 6.2.1: it added many + but to do so we had to give up one, that is use makeCategorizable in ext_tables.php
Features | TYPO3 6.0 | TYPO3 6.1.0 | TYPO3 6.1.1 | TYPO3 6.2.0 | TYPO3 6.2.1 |
---|---|---|---|---|---|
use makeCategorizable in ext_tables.php | + | + | + | + | - |
use makeCategorizable in ext_localconf.php | - | - | + | + | + |
use makeCategorizable in Overrides/file.php | - | - | - | - | + |
override TCA of makeCategorizable | + | + | - | - | + |
use makeCategorizable for newly introduced table1 | - | - | + | + | + |
use makeCategorizable for new table in same TCA file | - | - | - | - | - |
have TCA of makeCategorizable cached | - | - | - | - | + |
have changes to TCA of makeCategorizable cached4 | - | - | - | - | - |
database definition automatically added | (+)[2] | (+)[2] | (+)[2] | + | + |
defaultCategorizedTables | - | + | + | + | + |
defaultCategorizedTables overridable | - | - | - | - | +[3] |
defaultCategorizedTables cachable | - | - | - | - | + |
defaultCategorizedTables overrides cachable | - | - | - | - | - |
API to return category field configuration | - | - | - | - | - |
[1] In same extension
[2] Was buggy for EM
[3] in ext_tables.php
[4] in 6.2.1 changes to TCA created by CategoryRegistry can only be overridden in ext_tables.php
Updated by Helmut Hummel over 10 years ago
[EDIT] [6] and [7] is fixed with #58039
[EDIT-2] Added failing case to table
[EDIT-3] Fixed failing case to table
Features | TYPO3 6.0 | TYPO3 6.1.0 | TYPO3 6.1.1 | TYPO3 6.2.0 | TYPO3 6.2.1 | TYPO3 6.2.2-patch |
---|---|---|---|---|---|---|
use makeCategorizable in ext_tables.php | + | + | + | + | - | + |
use makeCategorizable in ext_localconf.php | - | - | + | + | + | + |
use makeCategorizable in Overrides/file.php | - | - | - | - | + | + |
override TCA of makeCategorizable | + | + | - | - | + | + |
use makeCategorizable for newly introduced table1 | - | - | + | + | + | + |
makeCategorizable new table, old stlye "dynamic" TCA1 | - | - | + | + | - | +[8] |
use makeCategorizable for new table in same TCA file | - | - | - | - | - | +[5] |
have TCA of makeCategorizable cached | - | - | - | - | (+)[7] | + |
have changes to TCA of makeCategorizable cached4 | - | - | - | - | - | + |
database definition automatically added | (+)[2] | (+)[2] | (+)[2] | + | + | + |
defaultCategorizedTables | - | + | + | + | + | + |
defaultCategorizedTables overridable | - | - | - | - | +[3] | + |
defaultCategorizedTables cachable | - | - | - | - | + | + |
defaultCategorizedTables overrides cachable | - | - | - | - | - | + |
API to return category field configuration | - | - | - | - | - | +[5] |
[1] In same extension
[2] Was buggy for EM
[3] in ext_tables.php
[4] in 6.2.1 changes to TCA created by CategoryRegistry can only be overridden in ext_tables.php
[5] new API: CategoryRegistry::getTcaFieldConfiguration()
[6] not the case for new API and also not the case when makeCategorizable is used in Overrides/file.php
[7] This is true when calling makeCategorizable from Overrides/file.php which has the downside, that the DB field will not be added during DB compare any more and has to be added manually
[8] TCA ctrl section of table must be defined before calling makeCategorizable !
Updated by Gerrit Code Review over 10 years ago
- Status changed from Needs Feedback 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/29560
Updated by Gerrit Code Review over 10 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/29560
Updated by Gerrit Code Review over 10 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/29560
Updated by Gerrit Code Review over 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29560
Updated by Gerrit Code Review over 10 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29560
Updated by Gerrit Code Review over 10 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29560
Updated by Helmut Hummel over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 4b3e3cce4f1bdcedc7f6ee23fb1bea952569a5ca.
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed