Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
standard_install()
manually updates various bits of configuration instead of shipping with proper configuration exports in config/install
. This promotes a bad practice for other installation profile authors.
Proposed resolution
Remove all configuration updates in favor of exported configuration files.
Comment | File | Size | Author |
---|---|---|---|
#9 | 2990776-9.patch | 6.97 KB | tstoeckler |
Comments
Comment #2
tstoecklerHere we go.
I could have sworn we already an issue for this somewhere, but couldn't find anything. I put some related prior art in the "Related issues" section, but I didn't find anything about this part in particular.
Just handling Standard here, to see how the issue goes. Umami suffers from the same problem, will open a separate issue for that.
Comment #4
tstoecklerAborted the test for #2 because I realized I failed to account for
node_install()
andFilterFormat::postSave()
. (Also fixes an incorrect ordering.)Comment #6
tstoecklerWell that's easy enough to fix...
Comment #7
chr.fritschThis is really cool and cleans up some code.
I only found one nitpick:
Comment #8
chr.fritschWe should also open a follow-up to do these changes in umami as well.
Comment #9
tstoecklerHere we go. Will open the follow-up now, and reference this one, so it should appear in the sidebar in a minute ;-)
Comment #10
chr.fritschThank you. 👍
Comment #11
catchCommitted 44920d2 and pushed to 8.7.x. Thanks!
Comment #13
tstoecklerAwesome, thank you! Just posted a patch in #2357215: Clean up hook_install() in Standard Install Profile which is now pretty easy...