Bug #90974
closedForm documentation advises to check for TYPO3_MODE === 'BE' in ext_localconf.php
100%
Description
I believe, this is highly discouraged due to caching reasons, see
It is recommended to AVOID checks for values on TYPO3_MODE or TYPO3_REQUESTTYPE constants (e.g. if (TYPO3_MODE === 'BE')) within these files as it limits the functionality to cache the whole systems’ configuration. Any extension author should remove the checks if not explicitly necessary, and re-evaluate if these context-depending checks could go inside the hooks / caller function directly.
if (TYPO3_MODE === 'BE') {
see
- 10.2: https://docs.typo3.org/c/typo3/cms-form/10.2/en-us/I/Concepts/Configuration/Index.html#yaml-registration-for-the-backend-via-addtyposcriptsetup
- 9.5: https://docs.typo3.org/c/typo3/cms-form/8.7/en-us/Concepts/Configuration/Index.html#yaml-registration-for-the-backend-via-addtyposcriptsetup
- etc.
If it is necessary to do that in that case, maybe add an explanation. In any case I find this confusing to have contradicting information.
Also, in the core "form" extension (at least for master), this check is not performed.
Updated by Gerrit Code Review about 4 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/+/65511
Updated by Gerrit Code Review about 4 years ago
Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65487
Updated by Oliver Bartsch about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset da657436e08d81422c2b5ddf5fadfbedf87a298a.
Updated by Gerrit Code Review about 4 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65532
Updated by Oliver Bartsch about 4 years ago
- Status changed from Under Review to Resolved
Applied in changeset d9f0fe6605160e3b878d3beed490e1d7d298a152.