I needed to get the enclosure working with Media so I created this patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davidseth’s picture

Patch attached

davidseth’s picture

Status: Active » Needs review
maciej.zgadzaj’s picture

Status: Needs review » Fixed

Thanks David! Patch applied (with 2 small changes) and pushed to d.o.

Out of sheer curiosity - you're using 1.x version of Media module I guess? I'm asking as I'm developing Views RSS: Media (MRSS) Elements module (and testing everything) based on 2.x branch of Media module, where there is no media field type anymore, and that's also why it was not included in the field types here.

Quoting after Comparison between Media 1.x and 2.x:

The 'Multimedia asset' field has been deprecated and moved to its own sub-module. Site builders are recommended to use the core Image and File fields for handling file and media (and use the 'Media file selector widget for those fields) rather than using 'Multimedia asset' fields.

davidseth’s picture

Hello @maciej.zgadzaj,

Yes, I am using the deprecated Media field type. But it is still used in Media 1.x and some early adopters of 2.x (that is me!). So there would be a fair amount of users who have it in place.

Thanks for taking the time to apply the patch!

Cheers,

David

Status: Fixed » Closed (fixed)

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