Migrate system site i18n variables to config:

  • site_name
  • site_mail
  • site_slogan
  • site_frontpage
  • site_403
  • site_404
  • drupal_weight_select_max
  • admin_compact_mode
CommentFileSizeAuthor
#8 2637182-8.patch3.47 KBquietone
#7 2637182-7.patch3.59 KBquietone
#6 2637182-6.patch3.11 KBquietone
#3 2637182-3.patch4.74 KBquietone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone CreditAttribution: quietone commented
Status: Active » Postponed

This can proceed once the source and dest plugin are resolved in #2225477: Add migrate sources and destinations for D6 i18n variables.

quietone’s picture

quietone CreditAttribution: quietone commented
Issue tags: +i18n-migrate
FileSize
2637182-3.patch4.74 KB

This version works locally for the site name and site slogan. The other variables need to be added to the test fixture, the test and the migration template.

quietone’s picture

quietone CreditAttribution: quietone commented
Component: entity system » migration system

Mistakenly in wrong system, moved to migration system.

quietone’s picture

quietone CreditAttribution: quietone as a volunteer commented
Issue tags: +migrate-d6-d8
quietone’s picture

quietone CreditAttribution: quietone as a volunteer commented
FileSize
2637182-6.patch3.11 KB

Removed drupal6.php because of #2670170: Add i18n string & variable data to d6_dump and updated for new test data.

quietone’s picture

quietone CreditAttribution: quietone as a volunteer commented
FileSize
2637182-7.patch3.59 KB

Cleanup and added tests for another language.

quietone’s picture

quietone CreditAttribution: quietone as a volunteer commented
Version: 8.0.x-dev » 8.1.x-dev
Status: Postponed » Needs review
FileSize
2637182-8.patch3.47 KB

Test now that the dump has the i18n string data.

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Needs review » Reviewed & tested by the community 
+++ b/core/modules/config_translation/tests/src/Kernel/Migrate/d6/MigrateI18nSystemSiteTest.php
@@ -0,0 +1,53 @@
+/**
+ * @file
+ * Contains \Drupal\Tests\config_translation\Kernel\Migrate\d6\MigrateI18nSystemSiteTest.
+ */

Un-needed comment

The last submitted patch, 3: 2637182-3.patch, failed testing.

  • catch committed 6005468 on 8.1.x 
    Issue #2637182 by quietone: i18n Variable to config: system site...

  • catch committed 3dccd75 on 8.2.x 
    Issue #2637182 by quietone: i18n Variable to config: system site...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
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.