This is a sub-issue of #1938580: [META] Make active config save format match the default yml file (order and quotes).

Files need to be fixed
locale.setttings.yml

Files fixed already

Files: 
CommentFileSizeAuthor
#6 1942148-locale-config-fix-6.patch625 bytesSwapS
PASSED: [[SimpleTest]]: [MySQL] 55,413 pass(es). View
#6 1942148-diff-2-6.txt567 bytesSwapS
#2 1942148-locale-config-fix-2.patch719 bytessandipmkhairnar
PASSED: [[SimpleTest]]: [MySQL] 54,367 pass(es). View
#1 1942148-locale-config-fix-1.patch744 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 53,118 pass(es). View

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
744 bytes
PASSED: [[SimpleTest]]: [MySQL] 53,118 pass(es). View
sandipmkhairnar’s picture

FileSize
719 bytes
PASSED: [[SimpleTest]]: [MySQL] 54,367 pass(es). View

Thanks @vijaycs85. As discussed on IRC updated directory variable.

vijaycs85’s picture

#2: 1942148-locale-config-fix-2.patch queued for re-testing.

vijaycs85’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

alexpott’s picture

Status: Reviewed & tested by the community » Needs review
+++ b/core/modules/locale/config/locale.settings.ymlundefined
@@ -2,11 +2,11 @@ cache_strings: '1'
   directory: 'languages'

I don't get why the quotes are back?

+++ b/core/modules/locale/config/locale.settings.ymlundefined
@@ -2,11 +2,11 @@ cache_strings: '1'
-  path: ''
+  path: null

Also as we convert everything (atm) to strings... are you sure about this?

SwapS’s picture

FileSize
567 bytes
625 bytes
PASSED: [[SimpleTest]]: [MySQL] 55,413 pass(es). View

Thanks for Review @alexpott.
Here is the patch.

LinL’s picture

Status: Needs review » Closed (duplicate)

These changes have been done in #2106459: Core config has everything as string so closing this as a duplicate.