After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 600685 - Fails to subscribe to RSS podcast whose <title> is empty
Fails to subscribe to RSS podcast whose <title> is empty
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: Podcasting
1.5.1
Other Linux
: Normal normal
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-04 14:28 UTC by Pedro Villavicencio
Modified: 2010-03-05 20:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Ignores empty title in Podcasts (1.49 KB, patch)
2009-11-22 15:02 UTC, Austriaco
needs-work Details | Review

Description Pedro Villavicencio 2009-11-04 14:28:35 UTC
this report has been filed here:

https://bugs.edge.launchpad.net/ubuntu/+source/banshee/+bug/468323

"Banshee fails to subscribe to an RSS podcast, whose <title> tag is empty. Test case

http://kingworldnews.com/kingworldnews/Broadcast_Gold+/rss.xml

Banshee will not subscribe to this feed

I have reworked that podcast at the following address:

http://www.315-web.com/kingworldnews.php

The only difference between the original and the copy is that the latter has a proper <title> tag. In this case Banshee subscribes, sees the new episodes and downloads the latest one."
Comment 1 Austriaco 2009-11-22 15:02:09 UTC
Created attachment 148273 [details] [review]
Ignores empty title in Podcasts

This patch fixes the issue. It avoids testing for the empty title in CheckRss() and when the title is empty, it substitutes the feed URL for the name in UpdateFeed()
Comment 2 Gabriel Burt 2009-11-23 20:59:47 UTC
Review of attachment 148273 [details] [review]:

::: src/Libraries/Migo/Migo.Syndication/RssParser.cs
@@ +93,3 @@
             try {
                 feed.Title            = StringUtil.RemoveNewlines (GetXmlNodeText (doc, "/rss/channel/title"));
+                if(String.IsNullOrEmpty(feed.Title)) {

please follow the HACKING code style guide
if( => if (
Empty( => Empty ( etc
Comment 3 Bertrand Lorentz 2010-03-05 20:03:23 UTC
Comment on attachment 148273 [details] [review]
Ignores empty title in Podcasts

I committed a different fix : still check for the presence of the title tag, and display "Unknown Podcast" if it's empty.
Comment 4 Bertrand Lorentz 2010-03-05 20:03:43 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.