[TASK] Remove fileinfo as dependency in SystemEnvironment/Check 63/47063/3
authorTimo Schmidt <timo.schmidt@dkd.de>
Fri, 4 Mar 2016 15:08:22 +0000 (16:08 +0100)
committerNicole Cordes <typo3@cordes.co>
Fri, 4 Mar 2016 16:50:57 +0000 (17:50 +0100)
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 <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
INSTALL.md
typo3/sysext/install/Classes/SystemEnvironment/Check.php

index 8d72ef7..f314426 100644 (file)
@@ -69,7 +69,6 @@ Your PHP needs to support the following extensions. Install will
 check if these are available.
 
 * These are usually part of the standard PHP package on most distributions:
-  * fileinfo
   * filter
   * hash
   * openssl
index 8338f74..a7876b0 100644 (file)
@@ -52,7 +52,6 @@ class Check
      * @var array List of required PHP extensions
      */
     protected $requiredPhpExtensions = array(
-        'fileinfo',
         'filter',
         'gd',
         'hash',