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.
update_variables_to_config('menu.settings', array(
'menu_main_links_source' => 'main_links',
'menu_secondary_links_source' => 'secondary_links',
));
See parent issue #2181257: [meta] Variables to config migration [d7] for instructions.
Comment | File | Size | Author |
---|---|---|---|
#21 | interdiff-2353763-17-21.txt | 3.96 KB | phenaproxima |
#21 | 2353763-21.patch | 4.17 KB | phenaproxima |
#19 | menu_settings_diff.txt | 2.73 KB | mikeryan |
#17 | 2353763-17.patch | 2.46 KB | phenaproxima |
#16 | 2353763-16.patch | 2.88 KB | phenaproxima |
Comments
Comment #1
oadaeh CreditAttribution: oadaeh commentedAttached is the YAML for this issue, from the patch in #2382117: Migration Files for Drupal 7 Variables.
Test(s) (and maybe a dump file) still need to be written.
Comment #2
oadaeh CreditAttribution: oadaeh commentedComment #3
miguelc303 CreditAttribution: miguelc303 at Anexus commentedAdded organization support to Anexus IT
Comment #4
jcost CreditAttribution: jcost commentedWill need to be submitted again to Core since moving from sandbox.
Comment #5
phenaproximaComment #6
phenaproximaUpdated, with a test.
Comment #7
benjy CreditAttribution: benjy at CodeDrop commentedLooks fine to me, just a missing comment on testMigration if someone wants to fix that.
Comment #9
alexpottThe migrate needs to move to a template - see #2463909: Migrations should support non-installed default configurations (templates) and I think the test group is wrong see #2495271: Split the migrate_drupal test suite in to 6.x and 7.x groups
Comment #10
phenaproximaRe-rolled.
Comment #14
phenaproximaChanged test group.
Comment #15
phenaproximaRe-rolling. I can't create an interdiff because of conflicts between the two patches (due to the unmergeability of the {variable} table dump).
Comment #16
phenaproximaRe-rolled again. No interdiff for the same reason I mentioned in #15.
Comment #17
phenaproximaAnd yet again.
Comment #18
phenaproximaEDIT: Um. Clicked Save on an empty comment. That was durmb.
Comment #19
mikeryanHere's a diff between the D6 and D7 menu_settings migrations.
Comment #20
mikeryanI assume these variables existed in D6 but not D7?
Migrate <=> d7
Why the different class name?
Why is this necessary for D7 but not D6?
Why the test function name change (testMenuSettings -> testMigration)?
The D6 test is looking for a FALSE value, but the D7 test is looking for a TRUE value?
Comment #21
phenaproximaThe
menu_primary_links_source
andmenu_secondary_links_source
variables existed in both D6 and D7. However, they have no counterpart in D8. The D6 migration was mistaken. Once I removed those variables from the D6 migration, though, I realized that the d6_menu_settings and d7_menu_settings migrations were identical. So this patch condenses them into a single migration tagged for both Drupal 6 and Drupal 7.Comment #22
chx CreditAttribution: chx commenteddiscussed with phenaproxima and he added those two to the known issues. if there's nothing to migrate them to, well.
Comment #24
webchickHm, right. Those are just straight-up blocks now. Yay for less one-off crap!
Committed and pushed to 8.0.x. Thanks!
Comment #26
-enzo- CreditAttribution: -enzo- at Anexus commented@webchick I dont if is possible to amend the attribution in this commit, because the original patch created by @miguelc303 and myself but splited by @oadaeh from issue #2382117: Migration Files for Drupal 7 Variables