Closed (won't fix)
Project:
Drupal core
Version:
8.0.x-dev
Component:
migration system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Oct 2014 at 21:14 UTC
Updated:
27 Aug 2015 at 13:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
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
miguelc303 commentedAdded organization support to Anexus IT
Comment #3
jcost commentedWill need to be submitted again to Core since moving from sandbox.
Comment #4
phenaproximaComment #5
phenaproximaUpdated and wrote a test.
Comment #6
quietone commentedreroll
Comment #7
quietone commentedOh, and moved to the user module
Comment #8
mikeryanAttached are the diffs from the D6 version (always very useful in reviewing D7 migrations where the D6 and D7 source didn't change much).
Explicit dependencies are not needed.
s/migrate_drupal/user/
These use statements are unused (grey in PhpStorm).
D6 version is MigrateUserConfigsTest
D6 version was named testUserSettings().
testUserMail() from D6 is not present at all here. I don't think much if anything changed in this area between D6 and D7?
Comment #9
quietone commented1. fixed
2. fixed
3. fixed
4. D6 and D7 class names are the same
5. method name changed to match D6 version of test
Using the interdiff led me to find a few things. thx mikeryan
1, Add user_mail_status_deleted_notify to the D6 test, and the equivalent user_mail_status_canceled_notify to the D7 test.
2. Remove the duplicate 'user_mail-' from the following names in the D6 side of things
- user_mail_user_mail_register_pending_approval_subject
- user_mail_user_mail_register_pending_approval_body
Comment #11
quietone commentedThe name changes in Variable.php got missed in that patch. Here they are.
Comment #12
quietone commentedComment #13
phenaproximaThe migration in #11 has been rolled into #2414651: Migration Files for Drupal 7 Users, so we can close this one out.