Revert "[BUGFIX] Disable not required checks on cli" 74/50874/2
authorAlexander Stehlik <alexander.stehlik@gmail.com>
Sat, 3 Dec 2016 14:46:07 +0000 (15:46 +0100)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Sat, 3 Dec 2016 15:13:30 +0000 (16:13 +0100)
This reverts commit c02d4e37e44d12125969657e4e5706ff68eb1808.

The condition for checking for CLI mode in ext_tables.php of
the install tool is correct in the 7.6 branch.

The additional checks are not required.

Change-Id: I4d364fc08c68c7c25d9210c08b813247bee7d47f
Resolves: #78854
Relates: #78786
Reviewed-on: https://review.typo3.org/50874
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/install/Classes/SystemEnvironment/Check.php

index 2c365cc..2270f3e 100644 (file)
@@ -160,9 +160,6 @@ class Check
             $status = new Status\WarningStatus();
             $status->setTitle('Trusted hosts pattern is insecure');
             $status->setMessage('Trusted hosts pattern is configured to allow all header values. Check the pattern defined in Install Tool -> All configuration -> System -> trustedHostsPattern and adapt it to expected host value(s).');
-        } elseif (PHP_SAPI === 'cli') {
-            $status = new Status\NoticeStatus();
-            $status->setTitle('Trusted hosts pattern cannot be checked on the CLI');
         } else {
             if (GeneralUtility::hostHeaderValueMatchesTrustedHostsPattern($_SERVER['HTTP_HOST'])) {
                 $status = new Status\OkStatus();
@@ -345,11 +342,8 @@ class Check
     {
         $minimumMaximumExecutionTime = 30;
         $recommendedMaximumExecutionTime = 240;
-        $currentMaximumExecutionTime = (int)ini_get('max_execution_time');
-        if (PHP_SAPI === 'cli' && $currentMaximumExecutionTime === 0) {
-            $status = new Status\NoticeStatus();
-            $status->setTitle('Infinite PHP script execution time detected, which is admissible on the CLI');
-        } elseif ($currentMaximumExecutionTime === 0) {
+        $currentMaximumExecutionTime = ini_get('max_execution_time');
+        if ($currentMaximumExecutionTime == 0) {
             $status = new Status\WarningStatus();
             $status->setTitle('Infinite PHP script execution time');
             $status->setMessage(
@@ -385,7 +379,6 @@ class Check
             $status->setTitle('Maximum PHP script execution time is equal to or more than '
                 . $recommendedMaximumExecutionTime);
         }
-
         return $status;
     }