Bug #85363
closedImageInfo must not be called in EM if no extension icon is provided
100%
Description
If an extension icon is missing, the extension manager tries the following path Environment::getPublicPath() . '/' . $properties['siteRelPath'] . $properties['ext_icon']
which leads to the following logged error
Fri, 22 Jun 2018 18:12:40 +0200 [WARNING] request="c2d3b63beee46" component="TYPO3.CMS.Core.Type.File.ImageInfo": I could not retrieve the image size for file /app/web/typo3conf/ext/gdpr_example
Updated by Gerrit Code Review over 6 years ago
- Status changed from Accepted 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/57342
Updated by Gerrit Code Review over 6 years 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/57342
Updated by Gerrit Code Review over 6 years 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/57342
Updated by Gerrit Code Review over 6 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57342
Updated by Gerrit Code Review over 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57342
Updated by Georg Ringer over 6 years ago
- Status changed from Under Review to Closed
no need for further changes
Updated by Nicole Cordes over 6 years ago
- Category set to Extension Manager
- Status changed from Closed to Accepted
- Assignee changed from Georg Ringer to Nicole Cordes
As the extension icon is not required, I think it would be good to prevent the warning.
Updated by Gerrit Code Review over 6 years ago
- Status changed from Accepted 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/57393
Updated by Oliver Hader over 6 years ago
- Status changed from Under Review to New
Updated by Oliver Hader over 6 years ago
- Status changed from New to Under Review
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57413
Updated by Nicole Cordes over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3ce9cc7ae879f86d3077a91abc453b64c0411402.