Screen is black and completely unreadable in Configure page after install on standard profile

Lovely! I like how Drupal tries to be environmental friendly these days.e

Hmm, using Minimal profile I got to the configure page OK, but when clicking the submit at the end of the configure form I just get taken to a server 500 error page.

Seeing this error:

Warning: strtotime(): It is not safe to rely on the system's timezone settings. You are *required* to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected the timezone 'UTC' for now, but please set date.timezone to select your timezone. in Symfony\Component\Yaml\Inline::evaluateScalar() (line 503 of core/vendor/symfony/yaml/Symfony/Component/Yaml/Inline.php).

Confirmed. A refresh of the page makes the style appear. Cache issue somewhere?

@jhodgdon my bad, will do

This has only occurred in the last week or so - I first noticed it Monday. I'm going to try to find last known good this morning and figure out what happened.

Also because on that issue given on #9, you cannot install a module with default configurations for optional modules which could be nice.

Note that if you refresh the "configuration page" which is black during the install process, it get's fine as usual and thus usable. This leverages the issue I guess.

@Dom.

Also because on that issue given on #9, you cannot install a module with default configurations for optional modules which could be nice.

Can you post on #2090115: Don't install a module when its default configuration has unmet dependencies as to want you mean. I think this is probably just because the configuration being provided by the module has not been moved to the correct location. Optional config needs to go in config/optional not config/install. That's the whole point of that change.

Oups!

The problem is to do with how the active theme is set in the installer - the code in _drupal_maintenance_theme is a bit skewiff. This patch fixes the bug and adds a test. In a followup I will combine StandardInstallerTest and StandardTest but changing StandardTest to use InstallerTestBase instead of WebTestBase is a bit tricky and lots of out-of-context changes would be needed.

Ah, now this is looking much better!

+++ b/core/includes/install.core.inc
@@ -1611,21 +1611,13 @@ function install_profile_modules(&$install_state) {
-  $theme_handler = \Drupal::service('theme_handler');
-
-  // ThemeHandler::install() resets the current list of themes. The theme used
-  // in the installer is not necessarily in the list of themes to install, so
-  // retain the current list.
-  // @see _drupal_maintenance_theme()
-  $current_themes = $theme_handler->listInfo();
-
   // Install the themes specified by the installation profile.
   $themes = $install_state['profile_info']['themes'];
-  $theme_handler->install($themes);
+  \Drupal::service('theme_handler')->install($themes);
 
-  foreach ($current_themes as $theme) {
-    $theme_handler->addTheme($theme);
-  }
+  // Ensure that the install profile's theme is used.
+  // @see _drupal_maintenance_theme()
+  \Drupal::service('theme.manager')->resetActiveTheme();

Its really good to know that the theme manager is working properly in the early stages here.

Hi,

Patch applied from fresh Drupal and tested manually: install went well, no black screen.
Also had a look at the code : Drupal code convention followed (check with PHP CodeSniffer).

However, I'm not able to judge code solution itself, axelpott #13 defines code as "skewiff" and induce some further work to remain on test conversion. For this reason I'm not taking responsability for flaging status to reviewed and tested just adding this : RTBC+1 for me !

So who is going to take the next step with this issue then?

Btw the screenshot in the issue summary is the what bartik looks like when used as the installation theme :) - I think we can safely say that Bartik is not meant for that :)

Committed/pushed to 8.0.x, thanks!

Created followup #2459699: Merge StandardInstallerTest and StandardTest