Task #91500

Remove runtimeActivatedPackages left overs after deprecation removal

Added by Benjamin Franzke about 2 months ago. Updated about 1 month ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-05-27
Due date:
% Done:

100%

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

Description

https://review.typo3.org/c/Packages/TYPO3.CMS/+/64584

From what I see right now there are some remaining places where runtimeActivatesPackages are still taken into account:

typo3/sysext/core/Classes/Utility/ExtensionManagementUtility.php calculates $GLOBALS['TYPO3_CONF_VARS']['EXT']['runtimeActivatedPackages'] into cache hashes.

typo3/sysext/core/Configuration/DefaultConfiguration.php contains the default value.

typo3/sysext/extensionmanager/Classes/ViewHelpers/ToggleExtensionInstallationStateViewHelper.php contains logic regarding runtimeActivatedPackages.

See:
git grep runtimeActivatedPackages | grep -v "/Documentation/\|/ExtensionScanner/"

Also the Bootstrapping could be adapted to remove hacks that where needed for runtime activated packages.


Related issues

Related to TYPO3 Core - Task #91477: Remove deprecated code in Bootstrap Resolved 2020-05-26

Associated revisions

Revision 4af5abc5 (diff)
Added by Benjamin Franzke about 1 month ago

[TASK] Cleanup runtimeActivatedPackages deprecation removal

Remove a stale comment wrt PackageManager->activePackageDuringRuntime()
as that method has been removed in #91477. Drop the setSingletonInstance
workaround which was only needed for the runtime activated packages
feature to work.

Also remove some left over runtimeActivatedPackages logic in various
places.

Resolves: #91500
Related: #91477
Related: #91473
Releases: master
Change-Id: Ia2830c230e2a8b7443bb24a79c81dbf7f2f86226
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/64588
Tested-by: Benni Mack <>
Tested-by: TYPO3com <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Benni Mack <>
Reviewed-by: Andreas Fernandez <>

History

#1 Updated by Benjamin Franzke about 2 months ago

  • Related to Task #91477: Remove deprecated code in Bootstrap added

#2 Updated by Gerrit Code Review about 2 months 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/+/64588

#3 Updated by Gerrit Code Review about 1 month 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/+/64588

#4 Updated by Benjamin Franzke about 1 month ago

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

Also available in: Atom PDF