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
When re-saving configs in hook_update_n we have to trust the data, if we do not do it then there might errors during updates as the schema might not be up to date atm..
Proposed resolution
Trust the data when saving configs in system_update_8200 for sites which have not been updated yet.
Remaining tasks
Review & Commit.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#2 | 2838370-2.patch | 577 bytes | hchonov |
Comments
Comment #2
hchonovComment #3
hchonovComment #4
tstoecklerThis is caused by #2625212: Add ConfigSchemaChecker to development.services.yml. Pasting part of my comment from there here, as well:
I guess it should still be harmless to do this, but it is a bit weird to change this after this has run on most sites...
Comment #6
hchonovMost but not all .. so it is better to fix than to leave it like this forever
Comment #8
stevekeiretsu CreditAttribution: stevekeiretsu commentedhi
Attempting to update from 8.1.7 to 8.2.4 and database update fails:
system module
Update #8200
Failed: Drupal\Core\Config\ConfigNameException: Missing namespace in Config object name this. in Drupal\Core\Config\ConfigBase::validateName() (line 97 of /Users/myusername/Sites/mj2/core/lib/Drupal/Core/Config/ConfigBase.php).
Is this relevant, or am I totally on the wrong thread? This seems to be the only thing turning up googling "drupal system 8200" and the like. Applying the patch in this thread did not change anything, #2625212 does not seem to be applicable either as my (development.)services.yaml contains nothing about ConfigSchema in the first place.
Comment #10
lukasss CreditAttribution: lukasss commented#8 also for 8.3.3
Comment #18
quietone CreditAttribution: quietone at PreviousNext commentedIs anyone experiencing this problem? If so, set the status to 'Active' and add a comment.
Thanks!
Comment #20
longwavesystem_update_8200()
is long gone, closing as outdated.