Commit 8a23417a authored by catch's avatar catch
Browse files

Issue #3103529 by mcdruid, alexpott, Chris Burge, greg.1.anderson, rfay,...

Issue #3103529 by mcdruid, alexpott, Chris Burge, greg.1.anderson, rfay, anavarre, Gábor Hojtsy: Drupal 8.8.1+ and 9 can fail to install in the web browser due to cache pollution
parent fd8c9222
...@@ -618,6 +618,12 @@ public function discoverServiceProviders() { ...@@ -618,6 +618,12 @@ public function discoverServiceProviders() {
// Retrieve enabled modules and register their namespaces. // Retrieve enabled modules and register their namespaces.
if (!isset($this->moduleList)) { if (!isset($this->moduleList)) {
$extensions = $this->getConfigStorage()->read('core.extension'); $extensions = $this->getConfigStorage()->read('core.extension');
// If core.extension configuration does not exist then the container
// cannot be dumped because Drupal is yet to be installed.
if ($extensions === FALSE) {
$this->allowDumping = FALSE;
$this->containerNeedsDumping = FALSE;
}
$this->moduleList = isset($extensions['module']) ? $extensions['module'] : []; $this->moduleList = isset($extensions['module']) ? $extensions['module'] : [];
} }
$module_filenames = $this->getModuleFileNames(); $module_filenames = $this->getModuleFileNames();
......
...@@ -54,6 +54,15 @@ protected function prepareEnvironment() { ...@@ -54,6 +54,15 @@ protected function prepareEnvironment() {
mkdir($this->settings['settings']['config_sync_directory']->value, 0777, TRUE); mkdir($this->settings['settings']['config_sync_directory']->value, 0777, TRUE);
} }
/**
* Visits the interactive installer.
*/
protected function visitInstaller() {
// Should redirect to the installer.
$this->drupalGet($GLOBALS['base_url']);
$this->assertSession()->addressEquals($GLOBALS['base_url'] . '/core/install.php');
}
/** /**
* {@inheritdoc} * {@inheritdoc}
*/ */
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment