Project

General

Profile

Actions

Task #38972

closed

Improve method makeCategorizable to be used everywhere

Added by Fabien Udriot over 11 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
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 open1 closed)

Related to TYPO3 Core - Bug #48381: after git pull db compare -> CREATE TABLE ( ( categories int(11) NOT NULL default '0' );ClosedAlexander Opitz2013-05-19

Actions
Actions #1

Updated by Gerrit Code Review over 11 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 #2

Updated by Gerrit Code Review over 11 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 #3

Updated by Gerrit Code Review over 11 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 #4

Updated by Helmut Hummel about 11 years ago

  • Target version changed from 6.0.0-alpha3 to 6.0.1
Actions #5

Updated by Gerrit Code Review about 11 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 #6

Updated by Gerrit Code Review about 11 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 #7

Updated by Gerrit Code Review about 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 #8

Updated by Gerrit Code Review almost 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 #9

Updated by Gerrit Code Review almost 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 #10

Updated by Fabien Udriot almost 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 #11

Updated by Gerrit Code Review almost 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 #12

Updated by Gerrit Code Review almost 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 #13

Updated by Francois Suter almost 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 #14

Updated by Fabien Udriot almost 11 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #15

Updated by Gerrit Code Review almost 11 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/20767

Actions #16

Updated by Fabien Udriot almost 11 years ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF