Problem/Motivation

While this seemed like a neat trick originally, @berdir and others found it confusing on #2212069: Non-English Drupal sites get default configuration in English, edited in English, originals not actually used if translated that the system.site language code is both used as a language code for that file itself and the default langcode for the whole site.

Proposed resolution

Introduce a default_langcode key that is used for the site default.

Remaining tasks

Do it. Ensure tests pass.

User interface changes

None.

API changes

None.

Files: 
CommentFileSizeAuthor
#4 interdiff.txt613 bytesGábor Hojtsy
#4 2457653-4.patch15.92 KBGábor Hojtsy
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 89,972 pass(es). View
#2 2457653.patch15.32 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 19,679 pass(es), 999 fail(s), and 997 exception(s). View

Comments

Gábor Hojtsy’s picture

Assigned: Unassigned » Gábor Hojtsy
Gábor Hojtsy’s picture

Status: Active » Needs review
FileSize
15.32 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 19,679 pass(es), 999 fail(s), and 997 exception(s). View

It is hard to tell all the instances 100% so here are the ones I found immediately. Let's see what fails with this.

Status: Needs review » Needs work

The last submitted patch, 2: 2457653.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
15.92 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 89,972 pass(es). View
613 bytes

Fix config schema.

alexpott’s picture

Issue tags: +D8 upgrade path

This looks good. It is a hard break for existing sites but very easy to fix.

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Yes, this looks good to me as well.

Berdir’s picture

Status: Reviewed & tested by the community » Needs work

Ups, change record :)

Gábor Hojtsy’s picture

Status: Needs work » Reviewed & tested by the community
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

Also speculatively opened head2head issue; #2459119: Upgrade path for #2457653 (system.site langcode).

  • catch committed 364a6e9 on
    Issue #2457653 by Gábor Hojtsy: System.site langcode is both used as a...
Gábor Hojtsy’s picture

Issue tags: -sprint

Thanks, published the change notice.

  • catch committed 079a52b on 8.0.x
    Revert "Issue #2457653 by Gábor Hojtsy: System.site langcode is both...

  • catch committed 3fbe92b on 8.0.x
    Revert "Revert "Issue #2457653 by Gábor Hojtsy: System.site langcode is...
catch’s picture

I reverted then unreverted this due to an unrelated regression in beta 8 so we could roll beta 9 with no schema/config changes from beta 8. Nothing to do with this issue directly.

epari.siva’s picture

epari.siva’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.