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.

Comments

dagmar’s picture

Status: Active » Needs review
StatusFileSize
new53.14 KB

Let use configstore by now.

dagmar’s picture

StatusFileSize
new66.99 KB

I forgot add the constants. Now all occurences of datastore were removed form the module.

dagmar’s picture

#2: 1510064.configstore.2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1510064.configstore.2.patch, failed testing.

dagmar’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
bojanz’s picture

Title: Change "datastore" to "configstore" or something better to match changes in D8 » Rename activestore and datastore
Status: Needs work » Active

Core 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.

bojanz’s picture

Status: Active » Needs review
StatusFileSize
new29.2 KB

So, something like this perhaps.

Note: the README would need to be rewritten.

bojanz’s picture

StatusFileSize
new36.47 KB

Here's some more, seems complete now.
Keep in mind this is completely untested.

Status: Needs review » Needs work

The last submitted patch, 1510064-rename-datastore-8.patch, failed testing.

bojanz’s picture

Status: Needs work » Needs review
StatusFileSize
new35.78 KB

Status: Needs review » Needs work

The last submitted patch, 1510064-rename-datastore-9.patch, failed testing.

bojanz’s picture

Status: Needs work » Needs review

#10: 1510064-rename-datastore-9.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1510064-rename-datastore-9.patch, failed testing.

bojanz’s picture

Status: Needs work » Needs review
StatusFileSize
new37.13 KB

Should fix the tests.