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
Currently \Drupal\Core\Config\StorageInterface::read()
returns FALSE when the configuration object does not exist.
Proposed resolution
Loading entities return NULL when the entity does not exist. Change \Drupal\Core\Config\StorageInterface::read()
to match.
Remaining tasks
Write patch and fix tests
User interface changes
None
API changes
Change return type of \Drupal\Core\Config\StorageInterface::read()
Comment | File | Size | Author |
---|---|---|---|
#14 | storage-interface-return-null-2260961-14.patch | 7.96 KB | nico.pinos |
Comments
Comment #1
alexpottThis came up in #2224887: Language configuration overrides should have their own storage
Comment #2
mtiftSo like this?
Comment #4
nico.pinos CreditAttribution: nico.pinos commentedI think CachedStorage is decoupled from this.
Something like this would be enough?
Comment #6
nico.pinos CreditAttribution: nico.pinos commentedComment #8
BerdirThat would be array|null then?
Comment #9
nico.pinos CreditAttribution: nico.pinos commentedYes, that's an easy fix. I'm getting some errors during the tests though. I'm working on some updates.
Comment #10
nico.pinos CreditAttribution: nico.pinos commentedUpdating tests.
Comment #12
nico.pinos CreditAttribution: nico.pinos commentedComment #14
nico.pinos CreditAttribution: nico.pinos commentedComment #16
andypostIs this still valid?
Comment #22
andypost