Variable to config: system.menu [d7]

working on this.

I can't find a variable in D6 to map with the unique variable in d8 active_menus_default

Please include the name of the variable we are upgrading from. It is not in the issue summary as that's just from a grep update_variables_to_config and because you are not including it I need to look up the name of the variable that you've already looked up.

Ie. the following comment would've been a lot more helpful: I can't find the variable menu_default_active_menus in D6. The D8 variable name is active_menus_default.

Reviewing the functionality, it is indeed D7 only.

Moving to the core queue to consolidate issues now that we're doing all the work there.

Hi there,

Here is a first version of the patch with the new test.

Regards,
David.

Removed whitespaces.

+++ b/core/modules/migrate_drupal/lib/Drupal/migrate_drupal/Tests/Dump/Drupal7SystemMenu.php
@@ -0,0 +1,31 @@
+        'name' => 'menu_default_active_menus',
+        'value' => serialize(array('SystemMenu' => 'TestSystemMenuMigration')),
+      ))

These are indented wrong, other than that looks good.

Fixed issue reported by benjy

Committed to 8.x, I moved the migration into the install folder when I committed it.

Will need to be submitted again to Core since moving from sandbox.

Needs to be merged into the parent issue.

The system.menu.active_menus_default configuration setting no longer appears to be a thing in Drupal 8.