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.

Files: 

Comments

geodaniel’s picture

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

Status: Active » Needs review
FileSize
2.57 KB
2.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

Status: Needs review » Fixed

good work. committed to cvs. thanks.

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

Anonymous’s picture

Status: Fixed » Closed (fixed)

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