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.
This patch adds support for Migrate API/package to the Email field module.
Comment | File | Size | Author |
---|---|---|---|
#11 | email-migrate_support-1232028-11.patch | 1.49 KB | victoriachan |
#5 | migrate_support_for_email_field-1232028-5.patch | 1.38 KB | das-peter |
migrate_support_for_email_field.patch | 1.91 KB | Bevan |
Comments
Comment #1
Bevan CreditAttribution: Bevan commentedThis patch was based on a related issue: #943140: Support migration into phone fields
Comment #2
mikeryanSubscribe
Comment #3
mh86 CreditAttribution: mh86 commentedI haven't been using Migrate yet, so I can't review your patch. As soon as someone tests and reviews it, I'm willing to commit it.
Comment #4
invisibleink CreditAttribution: invisibleink commentedThis patch worked beautifully for me. Solved my migration hang-up both for migrating and rolling back.
Brilliant! Thanks.
Using: Migrate 7.x-2.2-rc1 and Email 7.x-1.0-beta1
Comment #5
das-peter CreditAttribution: das-peter commentedThanks Bevan.
I've re-rolled the patch to use the path
a/
instead ofa/app/sites/all/modules/contrib/email/
in the patch and changedImplementation of hook_migrate_api()
to the new standardImplements hook_migrate_api().
.And of course I gave it a try - it works like a charm.
Thus marking as RTBC.
Comment #6
Bevan CreditAttribution: Bevan commentedThanks das-peter! :)
(I have since realised my patches were non-standard and also learned how to generate drupal standard patches from non-drupal git repos:
git diff --relative=app/sites/all/modules/contrib/email ...
)Comment #7
mh86 CreditAttribution: mh86 commentedCommitted patch from #5. Thanks for the patch and reviews!
Comment #8
wusel CreditAttribution: wusel commentedIt is necessary, to disable, then to delete and then to enable my migration-import-module.
Then all is ok.
Thank you very much.
Comment #9
wusel CreditAttribution: wusel commentedComment #11
victoriachan CreditAttribution: victoriachan commentedHi,
Thank you Bevan, for creating the patch, and thank you, mh86 for committing it to dev.
I'm just re-rolling the patch to work with version 7.x-1.0 for people who might need to use it with 1.0 now.
Cheers,
Victoria
Comment #12
wusel CreditAttribution: wusel commentedComment #13
Bevan CreditAttribution: Bevan commentedwusel, why did you reopen?