Feature #55778
Add a cache around getimagesize
| Status: | Resolved | Start date: | 2014-02-07 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Dominique Feyer | % Done: | 100% |
|
| Category: | - | |||
| Target version: | - |
Description
getimagesize need to read the full file from disk, so it impact performance a lots, a cache for those value can have a positive impact
Associated revisions
[FEATURE] Add cache for Image size and type
The function getimagesize need to load the file from disk on every
call. This patch introduce a new cache to store the height, width
and image type based on resource unique hash.
Change-Id: Ic50b52f3981e9031ef9e5de912713a3992464421
Resolves: #55778
Releases: master, 1.1
Reviewed-on: https://review.typo3.org/27428
Reviewed-by: Aske Ertmann
Reviewed-by: Dominique Feyer
Tested-by: Dominique Feyer
Reviewed-by: Robert Lemke
Tested-by: Robert Lemke
Reviewed-by: Christian Mueller
Tested-by: Christian Mueller
[FEATURE] Add cache for Image size and type
The function getimagesize need to load the file from disk on every
call. This patch introduce a new cache to store the height, width
and image type based on resource unique hash.
Change-Id: Ic50b52f3981e9031ef9e5de912713a3992464421
Resolves: #55778
Releases: master, 1.1
Reviewed-on: https://review.typo3.org/29347
Reviewed-by: Christian Mueller
Tested-by: Christian Mueller
History
#1 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.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#2 Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#3 Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#4 Updated by Gerrit Code Review over 1 year ago
Patch set 4 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#5 Updated by Gerrit Code Review over 1 year ago
Patch set 5 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#6 Updated by Gerrit Code Review over 1 year ago
Patch set 6 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#7 Updated by Gerrit Code Review over 1 year ago
Patch set 7 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#8 Updated by Gerrit Code Review over 1 year ago
Patch set 8 for branch master of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/27428
#9 Updated by Dominique Feyer over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 86e505b349b4a8e71f36e32c40babcbd0ab1b4b3.
#10 Updated by Gerrit Code Review over 1 year ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 1.1 of project Packages/TYPO3.Media has been pushed to the review server.
It is available at https://review.typo3.org/29347
#11 Updated by Dominique Feyer over 1 year ago
- Status changed from Under Review to Resolved
Applied in changeset 83b493e8cffbf16e0056679ff4d12611f9fe6efe.