Merge pull request #17636 from MikeyMJCO/patch-5

[dev/core#750] Don't check server variables if we're running in CLI
civicrm · Jul 9, 2020 · f7d3b28 · f7d3b28
2 parents 8bd77e7 + 8a0e49b
commit f7d3b28
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/Civi/Install/Requirements.php b/Civi/Install/Requirements.php
@@ -28,12 +28,15 @@ class Requirements {
    */
   protected $system_checks = [
     'checkMemory',
-    'checkServerVariables',
     'checkMysqlConnectExists',
     'checkJsonEncodeExists',
     'checkMultibyteExists',
   ];
 
+  protected $system_checks_web = [
+    'checkServerVariables',
+  ];
+
   protected $database_checks = [
     'checkMysqlConnection',
     'checkMysqlVersion',
@@ -83,6 +86,12 @@ public function checkSystem(array $file_paths) {
       $errors[] = $this->$check();
     }
 
+    if (PHP_SAPI !== 'cli') {
+      foreach ($this->system_checks_web as $check) {
+        $errors[] = $this->$check();
+      }
+    }
+
     return $errors;
   }