Line 34 of migrate_extras.module registers a class by name called 'MigrateEmvideoHandler'. However, the real class name is different:

class MigrateEmvideoFieldHandler extends MigrateFieldHandler {

leading to a fatal error.

Given the API, it looks like the class was correctly named, but the registry entry is a typo. It looks like somebody else noticed the same thing in comment 6 of issue #1428800: Support for Emvideo CCK field (Embedded Media Field)

CommentFileSizeAuthor
#3 migrate_extras-class_MigrateEmvideoHandler_not_found-1931952-2.patch761 bytesapotek
#2 migrate_extras-class_MigrateEmvideoHandler_not_found-#1931952-2.patch761 bytesapotek
#1 migrate_extras-class_MigrateEmvideoHandler_not_found-#1931952-2.patch761 bytesapotek

Comments

apotek’s picture

apotek commented
Issue summary: View changes

Adding additional confirmation of the issue from where it was mentioned in another issue.

apotek’s picture

apotek commented
Status: Active » Needs review
StatusFileSize
new migrate_extras-class_MigrateEmvideoHandler_not_found-#1931952-2.patch761 bytes

here's the patch.

apotek’s picture

apotek commented
StatusFileSize
new migrate_extras-class_MigrateEmvideoHandler_not_found-#1931952-2.patch761 bytes

For some reason, that patch didn't make it up to drupal, or was deleted. Trying again.

apotek’s picture

apotek commented
StatusFileSize
new migrate_extras-class_MigrateEmvideoHandler_not_found-1931952-2.patch761 bytes

Ok. Lesson learned. Don't put an octothorpe (#) in the file name. It seems to upload without error, but the link created to the file breaks. Hope this time it works.

sclapp’s picture

sclapp commented

Thank you so much for noticing what the issue was and creating a patch - saved my day!

apotek’s picture

apotek commented

*crickets*

VirtualNorman’s picture

VirtualNorman commented
Status: Needs review » Reviewed & tested by the community

I applied the patch and it resolved the error immediately, no cache clearing even required.

This is on Drupal 6.28, using Migrate Extras 6.x-2.5.

VirtualNorman’s picture

VirtualNorman commented
Issue summary: View changes

Minor reference typo fixed.

crazysix’s picture

crazysix commented

This was very helpful. I burned a few hours trying to figure out what was going on here. Thank you. Wish we could get this merged in!

hargobind’s picture

hargobind
he/him
Location Austin, Texas
commented
Issue summary: View changes

Also confirming that this fixes the issue. Too bad this project is listed as "no further development".

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Priority: Normal » Major

This issue breaks migrate_ui on 'admin/content/migrate' resulting in a blank screen e.g. Fatal error.
The pacth works beautifully!