Project

General

Profile

Actions

Task #94324

closed

Story #94322: Inconsistencies in showing page TSconfig in Info module

Remove "Module Web>Modules" from select list in Info > Page TSconfig

Added by Sybille Peters over 3 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Backend User Interface
Target version:
-
Start date:
2021-06-12
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

This corresponds to mod.web_modules which no longer exists.

Actions #1

Updated by Sybille Peters over 3 years ago

I am not familiar with the "Modules" module and mod.web_modules. I searched in the core for "web_modules" and it only exists in the select list in "Info > Page TSconfig".

There is no change log (grep for "web_modules").

I am not sure if extensions might still be using this.

I could find:

The Direct Mail configuration properties are set in the Page TSConfig of the Direct Mail folder under key mod.web_modules.dmail .

Currently, I am not sure what the correct convention for extensions is (for extending TSconfig with their own settings). If the mod.web_modules refers to a "Modules" module which no longer exists, it should be removed. If this is rather a placeholder for extension configuration, this is something different.

Not sure if this is a breaking change and direct_mail is using the settings incorrectly (should be mod.dmail or mod.tx_dmail ?) but someone else should verify.

Examples:

  • linkvalidator: mod.linkvalidator (although, shouldn't mod.web_info.linkvalidator be more correct?)
  • news: directly (not in mod): tx_news
  • direct_mail: mod.web_modules.dmail
Actions #2

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/69468

Actions #3

Updated by Gerrit Code Review about 3 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/c/Packages/TYPO3.CMS/+/69468

Actions #4

Updated by Sybille Peters about 3 years ago

  • Tracker changed from Bug to Task
Actions #5

Updated by Gerrit Code Review about 3 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/c/Packages/TYPO3.CMS/+/69468

Actions #6

Updated by Sybille Peters about 3 years ago

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

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF