Home » Download » Modules » Video » Issues

Media RSS support for Video module

geodaniel - February 16, 2007 - 10:33
Project:Video
Version:HEAD
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

» Login or register to post comments

#1

geodaniel - February 16, 2007 - 10:55

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?

Login or register to post comments

#2

ekes - March 16, 2008 - 11:01
Status:active» needs review

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

AttachmentSize
MRSS only patch 2.55 KB
Both: MRSS and Enclosures patch combined 2.57 KB
Login or register to post comments

#3

fax8 - March 16, 2008 - 11:44
Status:needs review» fixed

good work. committed to cvs. thanks.

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

Login or register to post comments

#4

Anonymous (not verified) - March 30, 2008 - 11:51
Status:fixed» closed

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

Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.