Task #74177

Remove fileinfo as dependency in SystemEnvironment/Check

Added by Timo Hund over 3 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Install Tool
Target version:
Start date:
2016-03-04
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

Currenty 'fileinfo' is listed as required php extension in SystemEnvironment/Check
but it is currently not really required and prevents from installing TYPO3 on Systems where it is not present (e.g. Mircosoft Azure Cloud)

The class is currently only used once in the core in "TYPO3\CMS\Core\Type\File\FileInfo" and only when it exists, so it is not really a hard dependency.

Also grepping on an installation folder does not find any match where \finfo is really required.


Related issues

Related to TYPO3 Core - Task #77153: Mention fileinfo PHP extension Closed 2016-07-19
Related to TYPO3 Core - Task #78702: Install Tool: System environment check should display a warning if PHP fileinfo extension is not loaded Closed 2016-11-15
Related to TYPO3 Core - Bug #88571: php extension fileinfo not required Rejected 2019-06-17
Follows TYPO3 Core - Bug #69495: System environment check is not standalone Closed 2015-09-03

Associated revisions

Revision a9f38804 (diff)
Added by Timo Schmidt over 3 years ago

[TASK] Remove fileinfo as dependency in SystemEnvironment/Check

Currenty 'fileinfo' is listed as required php extension in
SystemEnvironment/Check but it is currently not really required
and prevents from installing TYPO3 on Systems where it is not
present (e.g. Mircosoft Azure Cloud)

The class is currently only used once in the core in
"TYPO3\CMS\Core\Type\File\FileInfo" and only when it exists,
so it is not really a hard dependency.

Also grepping on an installation folder does not find any match
where \finfo is really required.

Resolves: #74177
Releases: master, 7.6
Change-Id: I125cd3e8d0eb6f208641375ea71b89917284f8b3
Reviewed-on: https://review.typo3.org/47063
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

Revision 9a76d437 (diff)
Added by Timo Schmidt over 3 years ago

[TASK] Remove fileinfo as dependency in SystemEnvironment/Check

Currenty 'fileinfo' is listed as required php extension in
SystemEnvironment/Check but it is currently not really required
and prevents from installing TYPO3 on Systems where it is not
present (e.g. Mircosoft Azure Cloud)

The class is currently only used once in the core in
"TYPO3\CMS\Core\Type\File\FileInfo" and only when it exists,
so it is not really a hard dependency.

Also grepping on an installation folder does not find any match
where \finfo is really required.

Resolves: #74177
Releases: master, 7.6
Change-Id: I125cd3e8d0eb6f208641375ea71b89917284f8b3
Reviewed-on: https://review.typo3.org/47069
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

History

#1 Updated by Timo Hund over 3 years ago

  • Category changed from File Abstraction Layer (FAL) to Install Tool

#2 Updated by Mathias Brodala over 3 years ago

Please note that work on this should wait until #69495 is done.

#3 Updated by Gerrit Code Review over 3 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/47063

#4 Updated by Gerrit Code Review over 3 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/47063

#5 Updated by Gerrit Code Review over 3 years ago

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

#6 Updated by Anonymous over 3 years ago

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

#7 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

#8 Updated by Oliver Hader 5 months ago

  • Related to Bug #88571: php extension fileinfo not required added

Also available in: Atom PDF