Task #78702
closedInstall Tool: System environment check should display a warning if PHP fileinfo extension is not loaded
100%
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).
Updated by Gerrit Code Review almost 8 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
Updated by Frans Saris almost 8 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?
Updated by Alexander Opitz over 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.
Updated by Ralf Merz over 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
Updated by Frans Saris over 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
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
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
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
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
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
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
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
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
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
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
Updated by Jonathan IROULIN over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 4337087a3d05022760c968deae4b7a5341755bf8.
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
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
Updated by Markus Klein over 6 years ago
- Status changed from Under Review to Resolved
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed