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.
DatabaseStorage can create the table itself. And this will make the patch over in #2262861: Add concept of collections to config storages smaller since that changes the DatabaseStorage schema.
Comment | File | Size | Author |
---|---|---|---|
remove-config-snapshot-schema.patch | 6.09 KB | alexpott | |
Comments
Comment #1
BerdirThere was some discussion about this in the issue that switched to the database backend, @sun was against this if I remember correctly, but since it was required on some cases, we now have a mixed case..
Looks fine to me but we should check the discussion/reason in the issue that added it..
Comment #2
alexpottSo we added the auto table creation in #2161591: Change default active config from file storage to DB storage - config_snapshot is not mentioned at all in that issue. Auto generation was added in comment #59 - @sun expressed reservations but in back #88 it was re-introduced by @pwolanin after discussion with @sun.
Comment #3
BerdirYes, you're right, this relies on the functionality of the database storage to create whatever table name you pass it to, and if we do it for the normal table, then it's only consistent to do it here too.
Comment #4
catchCommitted/pushed to 8.x, thanks!