MAGETWO-44635: Running module:enable --all via CLI before installation prevents running further commands

- Checking if deployment config is present based on install/date key

Author: mazhalai

lib/internal/Magento/Framework/App/DeploymentConfig.php

@@ -6,6 +6,8 @@
 
 namespace Magento\Framework\App;
 
+use Magento\Framework\Config\ConfigOptionsListConstants;
+
 /**
  * Application deployment configuration
  */
@@ -39,13 +41,6 @@ class DeploymentConfig
      */
     private $overrideData;
 
-    /**
-     * Availability of deployment config file
-     *
-     * @var bool
-     */
-    private $isAvailable;
-
     /**
      * Constructor
      *
@@ -85,7 +80,7 @@ public function isAvailable()
     {
         $this->data = null;
         $this->load();
-        return $this->isAvailable;
+        return isset($this->flatData[ConfigOptionsListConstants::CONFIG_PATH_INSTALL_DATE]);
     }
 
     /**
@@ -128,7 +123,6 @@ private function load()
     {
         if (null === $this->data) {
             $this->data = $this->reader->load();
-            $this->isAvailable = !empty($this->data);
             if ($this->overrideData) {
                 $this->data = array_replace($this->data, $this->overrideData);
             }

lib/internal/Magento/Framework/App/Test/Unit/DeploymentConfigTest.php

@@ -7,6 +7,7 @@
 namespace Magento\Framework\App\Test\Unit;
 
 use \Magento\Framework\App\DeploymentConfig;
+use \Magento\Framework\Config\ConfigOptionsListConstants;
 
 class DeploymentConfigTest extends \PHPUnit_Framework_TestCase
 {
@@ -88,7 +89,9 @@ public function testGetters()
 
     public function testIsAvailable()
     {
-        $this->reader->expects($this->once())->method('load')->willReturn(['a' => 1]);
+        $this->reader->expects($this->once())->method('load')->willReturn([
+            ConfigOptionsListConstants::CONFIG_PATH_INSTALL_DATE => 1
+        ]);
         $object = new DeploymentConfig($this->reader);
         $this->assertTrue($object->isAvailable());
     }
@@ -103,7 +106,9 @@ public function testNotAvailable()
     public function testNotAvailableThenAvailable()
     {
         $this->reader->expects($this->at(0))->method('load')->willReturn([]);
-        $this->reader->expects($this->at(1))->method('load')->willReturn(['a' => 1]);
+        $this->reader->expects($this->at(1))->method('load')->willReturn([
+            ConfigOptionsListConstants::CONFIG_PATH_INSTALL_DATE => 1
+        ]);
         $object = new DeploymentConfig($this->reader);
         $this->assertFalse($object->isAvailable());
         $this->assertTrue($object->isAvailable());

lib/internal/Magento/Framework/Module/ModuleList.php

@@ -139,7 +139,7 @@ public function isModuleInfoAvailable()
      */
     private function loadConfigData()
     {
-        if (null === $this->configData && ($this->config->isAvailable())) {
+        if (null === $this->configData && null !== $this->config->get(ConfigOptionsListConstants::KEY_MODULES)) {
             $this->configData = $this->config->get(ConfigOptionsListConstants::KEY_MODULES);
         }
     }