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.
In #1827112: Convert 'install_profile' variable to Settings we forgot to update the function doc block fordrupal_get_profile()
Critical because we mention variable_get() :)
Part of #1775842: [meta] Convert all variables to state and/or config systems
Comment | File | Size | Author |
---|---|---|---|
#4 | 2112261.4.patch | 795 bytes | alexpott |
#2 | 2112261.2.patch | 793 bytes | alexpott |
#1 | 2112261.1.patch | 820 bytes | alexpott |
Comments
Comment #1
alexpottHere's a patch
Comment #2
alexpottDon't really like the test in #1 it's too verbose and doesn't really add value.
Comment #3
damiankloip CreditAttribution: damiankloip commentedShould we wrap this setting name in quotes?
Howcome you changed from using settings() to settings.php - out of interest :)
Comment #4
alexpott1. Sure
2. Because everyone knows where their settings.php file is :)
Comment #5
damiankloip CreditAttribution: damiankloip commented2. zing! Sounds fair to me :)
Comment #6
jhodgdonThanks! Committed to 8.x.
Comment #7.0
(not verified) CreditAttribution: commentedUpdated issue summary.