From: Łukasz Uznański <l.uznanski@macopedia.pl>
Date: Sat, 31 Mar 2018 23:33:34 +0000 (+0200)
Subject: [BUGFIX] Do not require */ext to be writable in Composer Mode
X-Git-Tag: v9.5.1~54
X-Git-Url: http://git.typo3.org/Packages/TYPO3.CMS.git/commitdiff_plain/c3e576b8a437af37acb7cca84dc25bcb9b50f08d?hp=342e7bff84927406f7b63cc846ce759b8a437926

[BUGFIX] Do not require */ext to be writable in Composer Mode

Resolves: #81857
Releases: master
Change-Id: I2dc07e8ac23edad2c7bd4880c4fd12ebcf2e40d6
Reviewed-on: https://review.typo3.org/57772
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
---

diff --git a/typo3/sysext/install/Classes/Report/InstallStatusReport.php b/typo3/sysext/install/Classes/Report/InstallStatusReport.php
index 05b0898404e8..8e2e06a1765f 100644
--- a/typo3/sysext/install/Classes/Report/InstallStatusReport.php
+++ b/typo3/sysext/install/Classes/Report/InstallStatusReport.php
@@ -76,14 +76,17 @@ class InstallStatusReport implements \TYPO3\CMS\Reports\StatusProviderInterface
             $varPath . '/charset/' => 2,
             $varPath . '/lock/' => 2,
             $sitePath . '/typo3conf/' => 2,
-            Environment::getExtensionsPath() => 0,
             Environment::getLabelsPath() => 0,
             $sitePath . '/' . $GLOBALS['TYPO3_CONF_VARS']['BE']['fileadminDir'] => -1,
             $sitePath . '/' . $GLOBALS['TYPO3_CONF_VARS']['BE']['fileadminDir'] . '_temp_/' => 0,
         ];
 
-        if ($GLOBALS['TYPO3_CONF_VARS']['EXT']['allowGlobalInstall']) {
-            $checkWritable[Environment::getBackendPath() . '/ext/'] = -1;
+        // Check for writable extension folder files in non-composer mode only
+        if (!Environment::isComposerMode()) {
+            $checkWritable[Environment::getExtensionsPath()] = 0;
+            if ($GLOBALS['TYPO3_CONF_VARS']['EXT']['allowGlobalInstall']) {
+                $checkWritable[Environment::getBackendPath() . '/ext/'] = -1;
+            }
         }
 
         foreach ($checkWritable as $path => $requirementLevel) {