Project

General

Profile

Actions

Task #56538

closed

Epic #55070: Workpackages

Epic #55065: WP: Overall System Performance (Backend and Frontend)

Bug #52949: Speed decrease since 4.5

Bug #56307: Findings from Bootstrap/PackageManager/ClassLoader analyses

Cache the $GLOBALS['TYPO3_LOADED_EXT'] as an array

Added by Ernesto Baschny about 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Category:
Performance
Target version:
Start date:
2014-03-04
Due date:
% Done:

100%

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

Description

Several core components still access $GLOBALS['TYPO3_LOADED_EXT'] directly. Basically it's used to loop through the activated extensions and also to check if some extension is installed.

Since the Package Management, this array is being simulated by an object (LoadedExtensionsArray). Plan was to be able to add deprecation messages for accessing this array in future.

When accessing a FE with one USER_INT, accessing this object takes about 10% of the time.

It should be considered if we cannot generate the original array on first request, cache it, and then make the GLOBAL array available again to whoever uses it. It's just a list of extensions. The gain of the future possibility of throwing a deprecation message opposed to simply having this array as an array and be fast is probably not worth it.


Related issues 1 (0 open1 closed)

Has duplicate TYPO3 Core - Task #56341: Evaluate if it is really reasonable to have TYPO3\CMS\Core\Compatibility\LoadedExtensionsArrayClosedSteffen Ritter2014-02-26

Actions
Actions #1

Updated by Thomas Maroschik about 10 years ago

  • Status changed from New to Accepted
  • Assignee set to Thomas Maroschik
Actions #2

Updated by Michiel Roos about 10 years ago

It would be great if \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::isLoaded() could then just return a isset($GLOBALS['TYPO3_LOADED_EXT']['extension']).

Actions #3

Updated by Markus Klein about 10 years ago

Actually this is a regression.

Using basic functions like array_keys() does not work on objects like this, so this is was a breaking change.

Actions #4

Updated by Gerrit Code Review about 10 years ago

  • Status changed from Accepted to Under Review

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28329

Actions #5

Updated by Gerrit Code Review about 10 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28329

Actions #6

Updated by Thomas Maroschik about 10 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF