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.
The term "datastore" referring to the data in file is confusing and has been changed to "configstore" in D8. We should change this to match or possibly even consider better names as proposed by the community.
Comment | File | Size | Author |
---|---|---|---|
#14 | 1510064-rename-datastore-14.patch | 37.13 KB | bojanz |
#10 | 1510064-rename-datastore-9.patch | 35.78 KB | bojanz |
#8 | 1510064-rename-datastore-8.patch | 36.47 KB | bojanz |
#7 | 1510064-rename-datastore.patch | 29.2 KB | bojanz |
#2 | 1510064.configstore.2.patch | 66.99 KB | dagmar |
Comments
Comment #1
dagmarLet use configstore by now.
Comment #2
dagmarI forgot add the constants. Now all occurences of datastore were removed form the module.
Comment #3
dagmar#2: 1510064.configstore.2.patch queued for re-testing.
Comment #5
dagmarComment #6
bojanz CreditAttribution: bojanz commentedCore doesn't use "configstore" anymore.
Current terms are bad for UX, they are not immediately obvious.
The terms I see used in core are "active configuration" and "staged configuration", so maybe we should go down that road (giving us labels such as "Write active configuration to staging" and "Import from staging")
Alternatively, perhaps using "database store" and "filesystem store" (or storage) would be obvious enough, but matching core has obvious benefits.
Comment #7
bojanz CreditAttribution: bojanz commentedSo, something like this perhaps.
Note: the README would need to be rewritten.
Comment #8
bojanz CreditAttribution: bojanz commentedHere's some more, seems complete now.
Keep in mind this is completely untested.
Comment #10
bojanz CreditAttribution: bojanz commentedComment #12
bojanz CreditAttribution: bojanz commented#10: 1510064-rename-datastore-9.patch queued for re-testing.
Comment #14
bojanz CreditAttribution: bojanz commentedShould fix the tests.