Project

General

Profile

Actions

Task #78702

closed

Install Tool: System environment check should display a warning if PHP fileinfo extension is not loaded

Added by Florian Wessels over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
Install Tool
Target version:
-
Start date:
2016-11-15
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
easy
Sprint Focus:
On Location Sprint

Description

If the PHP extension "fileinfo" is not loaded, the system environment check in the install tool should throw an error because extracting of file metadata (e.g. mime-types) will fail (if the function 'mime_content_type' (deprecated) also not exists).


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Task #77153: Mention fileinfo PHP extensionClosedMathias Brodala2016-07-19

Actions
Related to TYPO3 Core - Task #74177: Remove fileinfo as dependency in SystemEnvironment/CheckClosedTimo Hund2016-03-04

Actions
Actions #1

Updated by Gerrit Code Review over 7 years 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 https://review.typo3.org/50662

Actions #2

Updated by Frans Saris over 7 years ago

  • Status changed from Under Review to Needs Feedback

See #74177 there is was removed as required extension. It should indeed be mentioned somewhere. But it isn't required correct?

Actions #3

Updated by Alexander Opitz about 7 years ago

  • Status changed from Needs Feedback to Closed

No feedback within the last 90 days => closing this issue.

If you think that this is the wrong decision or experience this issue again, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.

Actions #4

Updated by Ralf Merz almost 7 years ago

  • Priority changed from Should have to Must have

Hi guys,

I want you to bring back the finfo check back into install tool, because without that, the following happens:

We hav a customer hosting its website on their own, where they use a Suse Linux. I do no exactly know how PHP 7.0 has been installed, but the class finfo was not there.
This leads to the problem that ImageMagick does not know how big an image is (size in pixels) and it renders images to 0x0px. So in frontend you have a correct src to "_processed_/blabla", but in the image-tag itself, width and height will be rendered as
width=0 and height=0.

I found out that the size of the image wants to be fetched via "vendor/typo3/cms/typo3/sysext/core/Classes/Type/File/FileInfo.php". There is a check if the method exists. If not, the returned $mimeType is empty which causes confusing errors in the frontend.

So the install tool must check if this class is around, as it is possible that PHP does to have finfo included.

Thanks and greets
Ralf aka merzilla

Actions #5

Updated by Frans Saris almost 7 years ago

  • Status changed from Closed to Accepted
  • Priority changed from Must have to Should have

Is should be a system warning in the install tool

Actions #6

Updated by Susanne Moog over 6 years ago

  • Sprint Focus set to On Location Sprint
Actions #7

Updated by Jonathan IROULIN over 6 years ago

  • Assignee set to Jonathan IROULIN
Actions #8

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/54823

Actions #9

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/54823

Actions #10

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/54823

Actions #11

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/54823

Actions #12

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/54823

Actions #13

Updated by Gerrit Code Review over 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54823

Actions #14

Updated by Gerrit Code Review over 6 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54823

Actions #15

Updated by Gerrit Code Review over 6 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54823

Actions #16

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/54833

Actions #17

Updated by Gerrit Code Review over 6 years ago

Patch set 2 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/54833

Actions #18

Updated by Jonathan IROULIN over 6 years ago

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

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

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/50662

Actions #20

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/50662

Actions #21

Updated by Markus Klein over 6 years ago

  • Status changed from Under Review to Resolved
Actions #22

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF