When a url in Drupal 7 has a 'video/youtube' filemime, but the uri is formatted as 'youtube://', the DrupalFile7Migration class sets the file class to be MigrateExtrasFileYoutube class. That should be MigrateFileUriAsIs, because the Youtube handler is not designed to handle youtube:// formatted uris.

CommentFileSizeAuthor
#1 migrate_d2d-d7_youtube_fileclass-2320857-1.patch798 bytescthos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cthos’s picture

cthos CreditAttribution: cthos commented
Status: Active » Needs review
FileSize
migrate_d2d-d7_youtube_fileclass-2320857-1.patch798 bytes

Attached patch should take care of that.

  • mikeryan committed 7c73d07 on 7.x-2.x authored by cthos 
    Issue #2320857 by cthos: Properly handle Youtube URIs in D7 migration
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Assigned: Unassigned » cthos
Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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