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.
Follow-up to #2384675: Deprecate conf_path()
The parent issue missed usages of the deprecated conf_path() in InstallerExistingSettingsNoProfileTest which should be removed.
Comment | File | Size | Author |
---|---|---|---|
#4 | remove_usages_of-2503015-4.patch | 1.3 KB | cilefen |
#2 | 2503015-2.patch | 1.2 KB | joshi.rohit100 |
Comments
Comment #1
cilefen CreditAttribution: cilefen commentedComment #2
joshi.rohit100Comment #3
alexpottLet's call
DrupalKernel::getSitePath()
only once.Comment #4
cilefen CreditAttribution: cilefen commentedComment #5
cilefen CreditAttribution: cilefen commentedOh I didn't realize ::getSitePath will probably work. Oh no it won't. It isn't a static method.
Comment #7
alexpott@cilefen yep you're right - at this point - no container nothing we definitely have to use
::findSitePath()
Comment #8
alexpottI've merged this patch into #2487592: CMI: don't ship with a default "active" directory that is empty in most Drupal installations to prevent unnecessary re-rolling and added the contributors to this patch to the suggested commit message on the other issue.
Comment #9
alexpottNot a performance issue this one :)