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

Comments

quietone created an issue. See original summary.

quietone’s picture

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 commented
Issue tags: +i18n-migrate
StatusFileSize
new 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 commented
Component: entity system » migration system

Mistakenly in wrong system, moved to migration system.

quietone’s picture

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

quietone commented
StatusFileSize
new 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 commented
StatusFileSize
new 2637182-7.patch3.59 KB

Cleanup and added tests for another language.

quietone’s picture

quietone commented
Version: 8.0.x-dev » 8.1.x-dev
Status: Postponed » Needs review
StatusFileSize
new 2637182-8.patch3.47 KB

Test now that the dump has the i18n string data.

benjy’s picture

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