Bevan’s picture

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

mikeryan’s picture


mh86’s picture

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

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

Status:Needs review» Reviewed & tested by the community
new1.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

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

Status:Reviewed & tested by the community» Fixed

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

wusel’s picture

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

Status:Active» Closed (fixed)

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

victoriachan’s picture

new1.49 KB


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.


wusel’s picture

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

Status:Active» Closed (fixed)

wusel, why did you reopen?