Feature #55778

Add a cache around getimagesize

Added by Dominique Feyer over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Should have
Target version:
-
Start date:
2014-02-07
Due date:
% Done:

100%


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

Revision 86e505b3 (diff)
Added by Dominique Feyer over 5 years ago

[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

Revision 83b493e8 (diff)
Added by Dominique Feyer over 5 years ago

[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 5 years 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 5 years 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 5 years 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 5 years 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 5 years 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 5 years 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 5 years 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 5 years 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 5 years ago

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

#10 Updated by Gerrit Code Review over 5 years 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 5 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF