This patch adds support for Migrate API/package to the Email field module.

CommentFileSizeAuthor
#11 email-migrate_support-1232028-11.patch1.49 KBvictoriachan
#5 migrate_support_for_email_field-1232028-5.patch1.38 KBdas-peter
migrate_support_for_email_field.patch1.91 KBBevan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bevan’s picture

Bevan CreditAttribution: Bevan commented

This patch was based on a related issue: #943140: Support migration into phone fields

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented

Subscribe

mh86’s picture

mh86 CreditAttribution: mh86 commented

I 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.

invisibleink’s picture

invisibleink CreditAttribution: invisibleink commented

This 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

das-peter’s picture

das-peter CreditAttribution: das-peter commented
Status: Needs review » Reviewed & tested by the community
FileSize
migrate_support_for_email_field-1232028-5.patch1.38 KB

Thanks Bevan.
I've re-rolled the patch to use the path a/ instead of a/app/sites/all/modules/contrib/email/ in the patch and changed Implementation of hook_migrate_api() to the new standard Implements hook_migrate_api()..
And of course I gave it a try - it works like a charm.
Thus marking as RTBC.

Bevan’s picture

Bevan CreditAttribution: Bevan commented

Thanks 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 ...)

mh86’s picture

mh86 CreditAttribution: mh86 commented
Status: Reviewed & tested by the community » Fixed

Committed patch from #5. Thanks for the patch and reviews!

wusel’s picture

wusel CreditAttribution: wusel commented
Status: Fixed » Active

It is necessary, to disable, then to delete and then to enable my migration-import-module.
Then all is ok.

Thank you very much.

wusel’s picture

wusel CreditAttribution: wusel commented

Status: Active » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

victoriachan’s picture

victoriachan CreditAttribution: victoriachan commented
FileSize
email-migrate_support-1232028-11.patch1.49 KB

Hi,

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

wusel’s picture

wusel CreditAttribution: wusel commented
Status: Closed (fixed) » Active
Bevan’s picture

Bevan CreditAttribution: Bevan commented
Status: Active » Closed (fixed)

wusel, why did you reopen?