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
In conf_path()
in bootstrap.inc
, the return value is named $conf
. This is confusing, since it would seem to refer to the $conf
array instead of what actually is: the string containing the $conf_path
used for settings.php
.
Proposed resolution
Rename the variable to $conf_path
.
Comment | File | Size | Author |
---|---|---|---|
#6 | rename-conf_path-1960764-6.patch | 1.15 KB | foopang |
#4 | rename-conf_path-1960764-4.patch | 938 bytes | foopang |
#1 | rename-conf_path-1960764-1.patch | 942 bytes | foopang |
Comments
Comment #1
foopang CreditAttribution: foopang commentedComment #2
xjm#1: rename-conf_path-1960764-1.patch queued for re-testing.
Oops, I think this needs a reroll already following #1949724: Allow simpletest child sites to additionally load a test-specific settings.php to allow testing anonymous and configless updates.
Comment #4
foopang CreditAttribution: foopang commentedRerolling the patch.
Comment #5
xjmThe new hunk will need to be updated too:
Comment #6
foopang CreditAttribution: foopang commentedOops, sorry. Rerolled the patch again.
Comment #7
foopang CreditAttribution: foopang commentedComment #8
xjmHere's the whole function with the latest patch applied:
Thanks @foopang!
Comment #9
alexpottCommitted dad84f0 and pushed to 8.x. Thanks!