Problem/Motivation

i18n user settings are not migrated.

Proposed resolution

Migrate i18n variable, user settings to config.

References:

Remaining tasks

Write migration.
Write tests, includes updating the drupal6 test fixture. See Generating database fixtures for D8 Migrate tests.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#4 2655346-4.patch10.92 KBquietone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quietone created an issue. See original summary.

quietone’s picture

Issue tags: +i18n-migrate

Added tag.

quietone’s picture

Issue tags: +migrate-d6-d8
quietone’s picture

Version: 8.0.x-dev » 8.1.x-dev
Status: Active » Needs review
Related issues: +#2655348: i18n Variable to config: user mail [d6]
FileSize
10.92 KB

On closer look at MigrateUserConfigsTest, it tests both the user settings and user mail settings. So, moving the work already done in #2655348: i18n Variable to config: user mail [d6] to this issue, so the resulting test is consistent with the corresponding d6 one. And there is no need for two issues for these straight forward migration tests.

benjy’s picture

Status: Needs review » Reviewed & tested by the community

I think we may as well get these in so people can start testing them.

+++ b/core/modules/config_translation/tests/src/Kernel/Migrate/d6/MigrateI18nUserConfigsTest.php
@@ -0,0 +1,79 @@
+/**
+ * @file
+ * Contains \Drupal\Tests\config_translation\Kernel\Migrate\d6\MigrateI18nUserConfigsTest.
+ */

Un-needed comment

  • catch committed 87f6b35 on 8.2.x
    Issue #2655346 by quietone, benjy: i18n Variable to config: user...

  • catch committed 032ebbb on 8.1.x
    Issue #2655346 by quietone, benjy: i18n Variable to config: user...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

Status: Fixed » Closed (fixed)

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