Task #56109
Epic #55070: Workpackages
Epic #55065: WP: Overall System Performance (Backend and Frontend)
Cleanup \TYPO3\CMS\Lang\LanguageService
Status: | Resolved | Start date: | 2014-02-19 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | - | % Done: | 100% |
|
Category: | - | Spent time: | - | |
Target version: | 6.2.0 | |||
TYPO3 Version: | 6.2 | Complexity: | medium | |
PHP Version: | 5.3 | Sprint Focus: |
Description
Amongst other things:
- See if we can also cache the HSC values on sL() calls in the \TYPO3\CMS\Lang\LanguageService::$LL_labels_cache
- loadSingleTableDescription calls methods inside nested foreach loops. Each of the methods test unchanging values for being an array etc. over and over.
- Replace count() calls with !empty() if we are dealing with objects known to be an array
Associated revisions
[TASK] Cleanup \TYPO3\CMS\Lang\LanguageService
- Cache hsc values of calls to sL()
- Code cleanup
- Use !empty() instead of count() for objects known to be an array
Change-Id: Ie015f9755b98c41df185865325ea15ce42179e23
Resolves: #56109
Releases: 6.2
Reviewed-on: https://review.typo3.org/27745
Reviewed-by: Steffen Ritter
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Oliver Klee
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
History
#1 Updated by Ingo Schmitt over 1 year ago
- Parent task set to #55065
#2 Updated by Gerrit Code Review over 1 year 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/27745
#3 Updated by Gerrit Code Review over 1 year 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/27745
#4 Updated by Gerrit Code Review over 1 year 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/27745
#5 Updated by Michiel Roos over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 16c5c810c4051d1de38283045d62cc86a83c72b7.