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.
We should move $conf['cache_backends'] to a dedicated global (as the $databases array already is), so it can't be confused with with configuration overrides.
Comment | File | Size | Author |
---|---|---|---|
#5 | cache_backends.patch | 721 bytes | catch |
#1 | 1605414.patch | 2.7 KB | bleen |
Comments
Comment #1
bleen CreditAttribution: bleen commentedI think this handles it. Lets see what testbot has to say.
Comment #2
tim.plunkett#1: 1605414.patch queued for re-testing.
Comment #3
alexpottThis should use the Settings class rather than a new global
Comment #4
alexpottso... needs work :)
Comment #5
catchThis is a duplicate of the 'Make cache backends use the DIC' issue now - that issue removed the $conf variables. However I found a leftover so here's a patch for that.
Comment #6
tim.plunkettThat variable_get call is what led me here, thanks!
Comment #8
tim.plunkett#5: cache_backends.patch queued for re-testing.
Comment #9
BerdirYes, let's remove this :)
Comment #10
alexpottCommitted 77871c9 and pushed to 8.x. Thanks!