Bug #66465
FileList makes 10 fileExists() calls for each file
| Status: | Rejected | Start date: | 2015-04-17 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 0% |
|
| Category: | File Abstraction Layer (FAL) | Spent time: | - | |
| Target version: | - | |||
| TYPO3 Version: | 7 | Is Regression: | No | |
| PHP Version: | Sprint Focus: | |||
| Complexity: |
Description
When opening the file list in the backend, each file displayed by the FileList leads to 10 fileExists() calls in the FAL driver.
This is ok for the local driver, but not for remote storages.
Somehow those data should be cached.
Related issues
History
#1 Updated by Christian Weiske 4 months ago
I was able to reduce the file list loading time with a remote FAL adapter from 1.6s to 0.6s by caching fileExists(), folderExists() and getPermissions().
#2 Updated by Christian Weiske 4 months ago
The 10 fileExists() calls were in TYPO3 6.2.
In git master (the soon-to-be 7.2), they are gone.
What stays are the getPermissions() calls; 7 per file/folder.
#3 Updated by Gerrit Code Review 4 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 http://review.typo3.org/38762
#4 Updated by Gerrit Code Review 4 months ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38762
#5 Updated by Christian Kuhn 3 months ago
- Status changed from Under Review to Rejected
Having a general caching wrapper around the drivers is not a solid approach. The patch was abandoned and this issue will be closed as rejected for now.