Bug #105493
open
typo3/cms-extensionmanager still declared as partOfMinimalUsableSystem: true
0%
Description
According to this changelog entry: https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.0/Breaking-102935-OverhauledExtensionInstallationInExtensionManager.html
The
EXT:extensionmanagerextension is now optional.
However, in its composer.json:
"Package": {
"protected": true,
"partOfFactoryDefault": true,
"partOfMinimalUsableSystem": true
},
What is correct here? An optional extension is surely not "partOfMinimalUsableSystem".
Updated by Oliver Bartsch about 1 month ago
- Related to Feature #102935: Add PackageInitialisationEvent added
Updated by Andreas Kienast about 1 month ago
IMO it's correct to have partOfMinimalUsableSystem here. In non-Composer scenarios, there would be no way to enable EXT:extensionmanager ever again.
Updated by Gerrit Code Review about 1 month ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86809
Updated by Benjamin Franzke about 1 month ago
- Priority changed from Should have to Could have
- TYPO3 Version changed from 13 to 14
Good catch, although there is more work to be done to change that.
First and foremost we need to adapt https://github.com/TYPO3/typo3/blob/84e9083c50ad4bd4c42d4a846f78d470e379ab87/typo3/sysext/core/Classes/Package/PackageManager.php#L214 to respect partOfMinimalUsableSystem instead of protected, otherwise a change to partOfMinimalUsableSystem = false would have no effect, as the package would still always be activated due to it being present in classic mode.
With that in mind we need to think about the following usecases:
PackageStates.php is corrupt and therefore removed and needs to be recreated by the installtool:
In this case we still need the extension-manager to be enabled by default, as the integrator would otherwise not be able to activate it later.
(We'd need a minimimal extension-manager in EXT:install in order to allow extensionmanager activation from within install tool)
It would probably be fine to deactivate the extension manger via CLI in classic mode, but that CLI tool is currently provided by extensionmanager itself.
That means this tool would need to be moved somewhere (where?) else.
That's something that could be addressed in v14, but in v13 extensionmanager is optional for composer mode and will probably keep always-active in classic mode.