[mythtv-commits] mythtv/fixes/0.24 commit: f6e282f4c by Robert McNamara (rmcnamara)

[MythTV]

      Author:  Robert McNamara <rmcnamara at mythtv.org>
 Change Date:  2011-04-13T17:53:47-07:00
   Push Date:  2011/04/13 17:54:23 -0700
  Repository:  mythtv
      Branch:  fixes/0.24
New Revision:  f6e282f4cd95de787448ee32281cc0c20caafc5d
   Changeset:  https://github.com/MythTV/mythtv/commit/f6e282f4c

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.

Fixes #9724.

Modified:

   mythtv/libs/libmyth/rssparse.cpp