From 9828d0d29cd3c39e24434bcfe713297358c44198 Mon Sep 17 00:00:00 2001 From: Dane Powell Date: Tue, 14 Apr 2020 13:30:48 -0700 Subject: [PATCH] Fixes #4108: Dev Desktop doctor check missing env var. --- src/Robo/Doctor/DevDesktopCheck.php | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/src/Robo/Doctor/DevDesktopCheck.php b/src/Robo/Doctor/DevDesktopCheck.php index ad1e6a39d..7d66d7115 100644 --- a/src/Robo/Doctor/DevDesktopCheck.php +++ b/src/Robo/Doctor/DevDesktopCheck.php @@ -19,7 +19,7 @@ public function performAllChecks() { */ protected function checkDevDesktop() { if ($this->getInspector()->isDevDesktopInitialized()) { - if (getenv('DEVDESKTOP_DRUPAL_SETTINGS_DIR')) { + if (!getenv('DEVDESKTOP_DRUPAL_SETTINGS_DIR')) { $this->logProblem(__FUNCTION__, [ "DevDesktop usage is enabled, but \$DEVDESKTOP_DRUPAL_SETTINGS_DIR is not set in your environmental variables.", "", @@ -33,16 +33,6 @@ protected function checkDevDesktop() { "Add `export DEVDESKTOP_DRUPAL_SETTINGS_DIR=\"\$HOME/.acquia/DevDesktop/DrupalSettings\"` to ~/.bash_profile or equivalent for your system.`", ], 'error'); } - - $variables_order = ini_get('variables_order'); - $php_ini_file = php_ini_loaded_file(); - if (!strstr($variables_order, 'E')) { - $this->logProblem(__FUNCTION__, [ - "DevDesktop usage is enabled, but variables_order does support environmental variables.", - "", - "Define variables_order = \"EGPCS\" in $php_ini_file", - ], 'error'); - } } }