When migrating from Drupal 6 to Drupal 7 using CCK module, but without migrate, fatal error happens due the missing migrate module:

+ drush -y content-migrate-fields
PHP Fatal error:  Class 'MigrateSimpleFieldHandler' not found in /var/www/dfr7/www7/sites/all/modules/contrib/phone/phone.migrate.inc on line 34

Migrate is not a hard dependency for this module, and must remain optional, the right way to solve this is to move the MigratePhoneFieldHandler class definition into its own file, and specify this file into the files[] info for core to be able autoload it. Proceding this way, this class will be loaded only when the migrate module will spawn it and avoid those fatal errors in all other cases.

I am writing a patch, wait a few minutes.

CommentFileSizeAuthor
#1 1867340-1-phone-migrate_handler_fatal_errors.patch1.36 KBpounard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pounard’s picture

pounard
Primary language French
CreditAttribution: pounard commented
FileSize
1867340-1-phone-migrate_handler_fatal_errors.patch1.36 KB
Artusamak’s picture

Artusamak
he/him
Primary language French
Location Bzh
CreditAttribution: Artusamak commented
Status: Active » Reviewed & tested by the community

I confirm that the code looks ok, we bumped into this issue and the patch solved it.

wavesailor’s picture

wavesailor CreditAttribution: wavesailor commented

FYI: I just downloaded and enabled the migrate module and this issue went away

pounard’s picture

pounard
Primary language French
CreditAttribution: pounard commented

Which is heretic, this dependency should not be forced.

crutch’s picture

crutch CreditAttribution: crutch commented

this patch works well only tested with dev version, phone migration complete, thank you

troybthompson’s picture

troybthompson CreditAttribution: troybthompson commented
Issue summary: View changes

This solved my upgrade problem as well.