Task #38972: Improve method makeCategorizable to be used everywhere - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Task #38972

closed

Improve method makeCategorizable to be used everywhere

Added by Fabien Udriot over 12 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Fabien Udriot

Category:

Target version:

6.2.0

Start date:

2012-07-16

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.0

PHP Version:

5.3

Tags:

Complexity:

Sprint Focus:

Description

Method makeCategorizable is in charge of registering a table to be categorizable in the BE. The current implementation has the drawback to force the call of this method after the TCA of the table. If called before, the "categorization" wouldn't be applied because of timing issue. The patch improve the situation in this regard by postponing the application of the TCA responsible of categorization.

For more details about the method refer to t3lib_extMgm::makeCategorizable.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Gerrit Code Review over 12 years ago

Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review about 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review about 12 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12812

Actions

Copy link

Updated by Helmut Hummel almost 12 years ago

Target version changed from 6.0.0-alpha3 to 6.0.1

Actions

Copy link

Updated by Gerrit Code Review almost 12 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review almost 12 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review over 11 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review over 11 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

Updated by Gerrit Code Review over 11 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

#10

Updated by Fabien Udriot over 11 years ago

Target version changed from 6.0.1 to 6.2.0
PHP Version set to 5.3

To test the patch you could make this experiment. Before applying the patch, in extension "foo", add this line at the top of ext_tables.php and notice there are no categories key in TCA['foo']['columns'] Edit: Actually after a second test, it appears categorization is totally broken as of 6.1 following TCA refactoring. Patch resolves it.

# Actually, I have taken EXT:news/ext_tables.php as example
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::makeCategorizable('news', 'tx_news_domain_model_file');

After applying the patch, you will see a field "categories" in TCA of table "foo" proving the use of the patch.

Actions

Copy link

#11

Updated by Gerrit Code Review over 11 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

#12

Updated by Gerrit Code Review over 11 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12812

Actions

Copy link

#13

Updated by Francois Suter over 11 years ago

While looking at your patch, I discovered the [SYS][defaultCategorizedTables] setting which isn't documented in the wiki. It doesn't work at all unless your patch is applied.

By default it is set to "pages,tt_content". If your patch is backported as is to 6.1, categories will suddenly appear in the pages and tt_content tables upon the next minor upgrade, but the database will not have been updated, thus creating SQL errors. I would suggest to remove this default setting from 6.1 and leave it only for 6.2.

I will open a thread in the core list about this.

Actions

Copy link

#14