I was just looking through the RSS feeds produced by Video module and noticed that it did not appear to support the widely used Media RSS standard properly. It supports the <enclosure> tag but then duplicates that for a <media> tag which is not correct as far as I'm aware.

To support MRSS, we need to add the namespace for it (xmlns:media="http://search.yahoo.com/mrss") and use the tags (e.g., media:content) as specified in the specs above.

CommentFileSizeAuthor
#2 MRSS only patch2.55 KBekes
#2 Both: MRSS and Enclosures patch combined2.57 KBekes
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geodaniel’s picture

geodaniel CreditAttribution: geodaniel commented

Oops, that should be xmlns:media="http://search.yahoo.com/mrss/" - I'd copied the one from YouTube but it looks like they're not following the spec. Argh, how many other big sites aren't following the specs for extending their RSS?

ekes’s picture

ekes CreditAttribution: ekes commented
Status: Active » Needs review
FileSize
Both: MRSS and Enclosures patch combined2.57 KB
MRSS only patch2.55 KB

Patch attached:

  • Adds the xmlns for MRSS
  • Puts the correct content element into the media namespace
  • Adds other metadata as defined by MRSS from the video node if it is available - this makes it work nicely with #234832: Mapper for Video Module to share video node content between Drupal sites

The -both patch includes the fix for #234858: RSS enclosure created breaks feed

fax8’s picture

fax8 CreditAttribution: fax8 commented
Status: Needs review » Fixed

good work. committed to cvs. thanks.

http://drupal.org/cvs?commit=106103

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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