Bug 2492 - Podcast title not shown correct
: Podcast title not shown correct
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: 6.2.0
: All Debian Linux
: P2 normal (vote)
: ---
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-11-04 12:35 UTC by Manuel Salzunger
Modified: 2008-09-15 14:39 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Salzunger 2005-11-04 12:35:34 UTC
When adding
http://www.dradio.de/rss/podcast/sendungen/hintergrundpolitik/?mode=text to the
podcasts, the <title> information isnt shown correct.
Comment 1 KDF 2005-11-04 13:07:26 UTC
This looks like a possible recurrence of bug 2185.

d_plugins shows:
2005-11-04 13:06:55.6809 RssNews failed to parse feed
<http://www.dradio.de/rss/podcast/sendungen/hintergrundpolitik/?mode=text > because:

undefined entity at line 6, column 26, byte 265 at
D:\slim\server\CPAN/XML/Parser.pm line 187
Comment 2 Blackketter Dean 2005-11-04 14:05:25 UTC
our entity handling for rss/podcasts is a little broken, we have a number of
hard coded entities, but they can be defined in the xml.
Comment 3 KDF 2005-11-04 16:34:34 UTC
argh...seems the problem is line 351 of Podcast/Plugin.pm:

HTML::Entities::encode_entities($content, "\x80-\xff");

commenting this out, avoids the problem, but it was added as part of change 4362
by Dan to fix bug 1392.  However, it seems to work ok without (using sample
feeds from that bug).  Sample feeds given there fail to parse in player UI
because one more instance of the encoding problem crops up. copying line 349
from the plugin to line 564 of the Button mode should do it.  will do more tests
and post a patch/commit later.
Comment 4 KDF 2005-11-04 20:18:13 UTC
patch committed to trunk at change 5055
supplied feed above should work with Nov 5 nightly build of 6.5b1

please re-open if any issues still exist