Bug #85363: ImageInfo must not be called in EM if no extension icon is provided - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #85363

closed

ImageInfo must not be called in EM if no extension icon is provided

Added by Georg Ringer almost 6 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Nicole Cordes

Category:

Extension Manager

Target version:

Start date:

2018-06-23

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

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

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Georg Ringer almost 6 years ago

Status changed from Under Review to Closed

no need for further changes

Actions

Copy link

Updated by Nicole Cordes almost 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.

Actions

Copy link

Updated by Gerrit Code Review almost 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

Actions

Copy link

Updated by Oliver Hader almost 6 years ago

Status changed from Under Review to New

Actions

Copy link

#10

Updated by Oliver Hader almost 6 years ago

Status changed from New to Under Review

Actions

Copy link

#11

Updated by Gerrit Code Review almost 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

Actions

Copy link

#12