Project

General

Profile

Actions

Task #80837

closed

Update comments in ExtensionManagementUtility about use in ext_tables

Added by Jigal van Hemert over 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Category:
Code Cleanup
Target version:
Start date:
2017-04-13
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
no-brainer
Sprint Focus:

Description

In v8 the Install Tool has a wizard to check for TCA modifying functions in ext_tables.php files. The comments of these functions however, often mention that they are for use in ext_tables.php or TCA/Override/*.php files.
The comments obviously need to be adapted to the new situation as usage in ext_tables may break the frontend.

Actions #1

Updated by Gerrit Code Review over 7 years ago

  • Status changed from New 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/52437

Actions #2

Updated by Gerrit Code Review over 7 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/52437

Actions #3

Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52632

Actions #4

Updated by Jigal van Hemert over 7 years ago

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

Updated by Gone With the Wind over 7 years ago

Question: Any chance to add more clarity to the documentation?

https://typo3.org/api/typo3cms/class_t_y_p_o3_1_1_c_m_s_1_1_core_1_1_utility_1_1_extension_management_utility.html#a13ee6921557e1123c328a7dd3df9c576

Here, for example, it doesn't really say that the usage in ext_tables is discouraged...

EDIT:
Seems to be done already:
https://review.typo3.org/#/c/52437/

Actions #6

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF