[mythtv-commits] mythtv/master commit: d77e4347c by Robert McNamara (rmcnamara)

[MythTV]

      Author:  Robert McNamara <rmcnamara at mythtv.org>
 Change Date:  2011-04-13T17:47:44-07:00
   Push Date:  2011/04/13 17:53:29 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  d77e4347c42504be92ac0d3aad178881cfb13326
   Changeset:  https://github.com/MythTV/mythtv/commit/d77e4347c

Log:

RSS Parsing: Check player tags to make sure they don't have child elements.

This works around some very odd RSS feeds produced by Blip.tv, which inline a bunch of of HTML instead of using attributes as the Media RSS specification specifies:

Reference: http://video.search.yahoo.com/mrss

"Allows the media object to be accessed through a web browser media player console. This element is required only if a direct media url attribute is not specified in the <media:content> element. It has 1 required attribute, and 2 optional attributes.

        \<media:player url=\"http://www.foo.com/player?id=1111\" height=\"200\" width="400" /\>

url is the url of the player console that plays the media. It is a required attribute.

height is the height of the browser window that the url should be opened in. It is an optional attribute.

width is the width of the browser window that the url should be opened in. It is an optional attribute."

So, basically, we work around the violation of the spec by checking for child elements.

Modified:

   mythtv/libs/libmyth/rssparse.cpp