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.I've been noticing this on Drupal 8 status pages lately:
Drupal 8 no longer uses the $settings['install_profile'] value in settings.php and it can be removed.
Seems to be coming from the change record The install profile has moved from settings.php to the core.extension configuration object.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | provision-stop_setting_installation_profile_in_settings_php-3035707-2.patch | 1.33 KB | colan |
Comments
Comment #2
colanThis will prevent it from being set, but do we also need a DB update hook to set it in the configuration? There's a note about this in the change record, but I'm not sure if it applies.
Also, I removed some related variables that appear to be unused. They're not mentioned anywhere in the original referenced issue.
Comment #3
spidermanReading the changelog, it seems that a `hook_update()` would only be required for an already-installed site. Under normal operation, the `core.extension` config to track install profile would get created when the site is installed, as I understand it. As such, this patch seems sufficient to me.
That said, I haven't actually tested this patch, but it's pretty straightforward, so assuming @colan has proved it works, I think we can commit it :)
Comment #5
colanThanks for review and feedback. As it works fine with testing, and doesn't appear to break anything, I've committed it.