Task #56108
Epic #55070: Workpackages
Epic #55065: WP: Overall System Performance (Backend and Frontend)
Cleanup \TYPO3\CMS\Recordlist\RecordList\DatabaseRecordList and friends
Status: | Closed | Start date: | 2014-02-19 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | - | % Done: | 0% |
|
Category: | Code Cleanup | Spent time: | - | |
Target version: | 6.2.0 | |||
TYPO3 Version: | 6.2 | Complexity: | easy | |
PHP Version: | 5.3 | Sprint Focus: | On Location Sprint |
Description
- Checking values to be TRUE or FALSE instead of casting to boolean
- Go over all loops to see if we can move repeatedly called methods on unchanging values outside of the loop
- Go over htmlspecialchars usage
- See if we can cache the fetching of the icon code
History
#1 Updated by Ingo Schmitt over 1 year ago
- Parent task set to #55070
#2 Updated by Ingo Schmitt over 1 year ago
- Parent task changed from #55070 to #55065
#3 Updated by Tymoteusz Motylewski over 1 year ago
- File AbstractRecordList.png added
Attached class diagram (the only class missing on the diagram is the DatabaseRecordList from dbal).
#4 Updated by Michiel Roos over 1 year ago
- File 56108.patch
added
I'm abandoning this issue since there are only a handfull of micro optimisations that can be had here. And the word is that we are not doing any more of those so close before release in fear of breaking stuff.
Please find attached a patch with my work so far.
Some good came of this: https://review.typo3.org/#/c/27713/
Also the heaviest hogs in the extended db list are the preview icons in a page list. Those fetch the TSConfig for each page before rendering the appropriate page+eye icon. It would be nice to disable that icon using TSConfig. But that would be a feature. So maybe this issue will be revisited for a later version of TYPO3.
#5 Updated by Anja Leichsenring 6 months ago
- Sprint Focus set to On Location Sprint
#6 Updated by Benjamin Mack about 1 month ago
- Status changed from New to Closed
see comment by michiel