media_youtube was just set to update on an existing site .. the update failed due to undefined file_type_save() function. In tracking that down, it appears to be form file_entity in media 7.x-2.x, while my site has 7.x-1.2. That probably just needs the version set in the module dependencies.

CommentFileSizeAuthor
#1 1927290-media_youtube-dependancy-1.patch393 bytesrickmanelius
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rickmanelius’s picture

rickmanelius CreditAttribution: rickmanelius commented
Status: Active » Needs review
FileSize
1927290-media_youtube-dependancy-1.patch393 bytes

Patch attached

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

file_entity is included with Media 1.x, so I'm not sure why this would be an improvement or require 2.x. Note that this is also fixed with #1949520: Remove media_youtube_update_7202() and media_youtube_file_default_types_alter()

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Closed (works as designed)

I agree with #2; requiring File entity would be redundant.

The issue outlined in the OP is caused by using File entity 2.x with Media 1.x. You should always use the same version of Media and File entity.