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.
There is an issue #1957142: Replace config() with Drupal::config() to replace all calls to config() with Drupal::config(). However this is unlikely to be implemented until code freeze, therefore config() should be deprecated in the meantime.
Comment | File | Size | Author |
---|---|---|---|
#9 | config_deprecate_config-2028149-9.patch | 723 bytes | Eli-T |
#3 | config_deprecate_config-2028149-3.patch | 675 bytes | Eli-T |
#1 | config_deprecate_config-2028149-1.patch | 675 bytes | Eli-T |
Comments
Comment #1
Eli-TComment #2
damiankloip CreditAttribution: damiankloip commentedNice, i agree that we should mark this @deprecated and then deal with #1957142: Replace config() with Drupal::config(). Although I still don't see the point in making lots of issues out of that but hey, not a problem here :)
add parenthesis to config(), and we should use 'method' and not 'function'
Comment #3
Eli-TChanges suggested in #2 implemented.
Comment #4
damiankloip CreditAttribution: damiankloip commentedThank you. Looks good to me.
Comment #6
Eli-T#3: config_deprecate_config-2028149-3.patch queued for re-testing.
Comment #7
Eli-TReturning to rtbc following successful retest with no code changes.
Comment #8
alexpottNeed to add the version info see https://drupal.org/coding-standards/docs#deprecated
Comment #9
Eli-TAdded Drupal 8.0 as version at which the function was deprecated and added @see block to the replacement.
Comment #10
damiankloip CreditAttribution: damiankloip commentedLookin' good
Comment #11
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks.