Problem/Motivation
For testing the removal of Stable.
Steps to reproduce
Use @bbrala's excellent instructions as a basis: #3303780: Manually test QuickEdit module removal
Install Drupal 9.4
- Install Drupal 9.4
composer create-project drupal/recommended-project:9.4.* drupal-stable - Move into the directory
cd drupal-stable - Enable development settings
cp web/sites/example.settings.local.php web/sites/default/settings.local.php - Add Drush
composer require drush/drush - Install with Umami
vendor/bin/drush si demo_umami - Enable settings.local.php at the end of
web/sites/default/settings.php
Require and enable particles_orange which uses stable as base theme
composer require drupal/particles_orange- Log into the site and enable and set as default particles_orange.
- Go to articles and see the correct theme applies
Upgrade to Drupal 10 and test again
composer config minimum-stability dev- Make sure composer doesn't cry about the installed theme:
composer require mglaman/composer-drupal-lenient composer config --merge --json extra.drupal-lenient.allowed-list '["drupal/particles_orange"]' -
update composer.json to require drupal 10:
composer require drupal/core-composer-scaffold:10.0.x-dev drupal/core-project-message:10.0.x-dev drupal/core-recommended:10.0.x-dev drush/drush --no-update
Runcomposer update - Remove stable
rm -rf web/core/themes/stable vendor/bin/drush cr - Now check the site, the sites should be broken with a missing theme message
- Require contrib stable
composer require drupal/stable - Now check the site, the theme should load! Yay
Proposed resolution
Remaining tasks
none
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|
Comments
Comment #2
quietone commentedComment #3
bbralaJust tested classy: #3309319: Manually test upgrade core Classy to contrib Classy. Use that as base for the manual testing.
Comment #4
bbralaFind a theme with d9 compatibility here
Comment #5
bbralaOne thing to note. Classy was already removed, if stable has not yet been removed you need to either apply the removal patch or remove the theme directory when upgraded to 10 in the steps. This was done like this for the module removal testing also and vetted as a proper way by core comitters.
Comment #6
bbralaComment #7
bbralaComment #8
bbralaComment #9
bbralaWell, i thought, why not.
I've updated IS with steps, and went through them all. Everything works as expected, went from 9.4 to 10.0 succesfully without stable in core.
Comment #10
bbralaComment #11
kristen polI used ddev so my steps were a little different:
Install Drupal 9.4
mkdir drupal-stablecd drupal-stableddev config --project-type=drupal9 --docroot=web --create-docrootddev startddev composer create drupal/recommended-project:9.4.* --no-installddev composer require drush/drush --no-installddev composer installddev drush site:install demo_umami -yRequire and enable particles_orange which uses stable as base theme
Same as in issue summary.
Upgrade to Drupal 10 and test again
ddev composer config minimum-stability devddev composer require mglaman/composer-drupal-lenientI got to this point and then got a PHP version error so I'll see what I can do to get around this:
Comment #12
kristen polTrying above again but with bumping up php in ddev to get past the version issue:
Install Drupal 9.4
mkdir drupal-stablecd drupal-stableddev config --project-type=drupal9 --docroot=web --create-docrootddev startddev composer create drupal/recommended-project:9.4.* --no-installddev composer require drush/drush --no-installddev composer installddev drush site:install demo_umami -yRequire and enable particles_orange which uses stable as base theme
ddev composer require drupal/particles_orangeparticles_orange.Upgrade to Drupal 10 and test again
ddev composer config minimum-stability dev.ddev/config.yamlphp_versionddev restartddev composer require mglaman/composer-drupal-lenientddev composer config --merge --json extra.drupal-lenient.allowed-list '["drupal/particles_orange"]'ddev composer require drupal/core-composer-scaffold:10.0.x-dev drupal/core-project-message:10.0.x-dev drupal/core-recommended:10.0.x-dev drush/drush --no-updateddev composer updaterm -rf web/core/themes/stableddev drush crddev composer require drupal/stableddev drush crDrupal\Core\Theme\MissingThemeDependencyException: Base theme stable has not been installed. in Drupal\Core\Theme\ThemeInitialization->getActiveThemeByName() (line 115 of /var/www/html/web/core/lib/Drupal/Core/Theme/ThemeInitialization.php).Comment #13
kristen polI went through the steps in #12 again from scratch. One thing I didn't notice the first time is that my images aren't loading after upgrading to D10 and finishing step 12.
The logs show notices like:
Comment #14
kristen polI turned off mutagen and went through the steps again from #12 and the images show up fine so ignore the image thing above.
Note the steps in 12 are equivalent to the steps in the issue summary but just using ddev.
Everything works as expected other than there was a discussion on whether or not we could avoid the WSOD but I think the decision was that it would not be easy to handle that for the theme.
As such, marking RTBC. Thanks, everyone!
Comment #15
catchThanks for all the testing! I think we can mark this done.