Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
modules/menu/menu.install:103: update_variables_to_config('menu.settings'
modules/menu/menu.install:58: update_variables_to_config('menu.settings'
Renamed variables
menu_primary_links_source => [default value]
- D6: menu_primary_links_source => 'primary-links'
- D7: menu_main_links_source => 'main-menu'
- D8: menu.settings.main_links => 'main'
menu_secondary_links_source => [default value]
- D6: menu_secondary_links_source => 'secondary-links'
- D7: menu_secondary_links_source => 'user-menu'
- D8: menu.settings.secondary_links => 'account'
menu_default_node_menu => [default value]
- D6: menu_default_node_menu => 'primary-links'
- D7: menu_options_{$content_type} => array('main-menu' => 'main-menu')
- D8: same as D7 -- not yet converted to config
Thanks in advance for helping many hands to make light work!
See #2125745: [meta] Variables to config migration [D6] for instructions
Comment | File | Size | Author |
---|---|---|---|
#13 | imp_2130315-13_migrate-menu.settings-D6-to-D8.patch | 5.03 KB | jessehs |
#7 | imp_2130315-1_migrate-menu.settings-D6-to-D8.patch | 4.31 KB | adamdicarlo |
Comments
Comment #1
adamdicarlo CreditAttribution: adamdicarlo commentedComment #2
adamdicarlo CreditAttribution: adamdicarlo commentedComment #3
fastangel CreditAttribution: fastangel commentedHey adamdicarlo. Do you need help you with this?
Comment #4
adamdicarlo CreditAttribution: adamdicarlo commentedhi fastangel. Thanks - I'm on IRC now (#drupal-contribute) but I *think* I have it covered. I'm doing the sprint in Portland today. I almost finished the patch for this issue.
Comment #5
adamdicarlo CreditAttribution: adamdicarlo commentedComment #6
adamdicarlo CreditAttribution: adamdicarlo commentedFirst patch attached. It migrates just three variables from D6 -> D8:
menu_primary_links_source
menu_secondary_links_source
menu_override_parent_selector
There are still variables in menu.module (in D8) not converted to CMI, however.
Comment #7
adamdicarlo CreditAttribution: adamdicarlo commentedComment #8
adamdicarlo CreditAttribution: adamdicarlo commentedWon't have time to work on this soon, so unassigning. I hope my patch helps!
Comment #9
chx CreditAttribution: chx commentedCrosslinking issues. Holding this up until.
Comment #10
chx CreditAttribution: chx commentedRelated issue is in.
Comment #11
jessehsComment #12
chx CreditAttribution: chx commentedComment #13
jessehsThe variables mentioned in the issue about menus available for each node type are for Drupal 7 only, I believe.
I've rerolled the patch to apply against HEAD.
Comment #14
chx CreditAttribution: chx commentedRolled into #2154955: Migrate variables to config