Station feeds with non-audio RSS feeds cause FC

arashja / npr-android-app

Automatically exported from code.google.com/p/npr-android-app

0 stars 0 forks source link

Station feeds with non-audio RSS feeds cause FC #129

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

What steps will reproduce the problem?
1. Go to locate stations
2. Find wxxi (the AM version)
3. Go under podcasts and tap WXXI News Stories

What is the expected output? What do you see instead?
One would expect to see a list of news stories like a traditional RSS. Instead, 
the app crashes. 

Please use labels and text to provide additional information.
I believe this is because they are putting in a different type of RSS than we 
would expect under a podcast field; one without any media in it.

Original issue reported on code.google.com by jpenn...@gmail.com on 13 Jan 2012 at 7:06

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 13 Jan 2012 at 7:06

Changed state: Backlog

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 13 Jan 2012 at 7:14

Added labels: Priority-Low
Removed labels: Priority-Medium

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 13 Feb 2012 at 8:48

Added labels: Milestone-2.3

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 13 Feb 2012 at 8:49

Changed state: Accepted

GoogleCodeExporter commented 8 years ago

Assigning these to you.

Original comment by jpenn...@gmail.com on 13 Feb 2012 at 8:51

GoogleCodeExporter commented 8 years ago

I'm unable to reproduce this with wxxi.  Not sure if they fixed their feed 
(more likely) or if I've addressed the bug due to other changes.  Do you know 
of any other stations I could check to see if I can reproduce this issue?

Original comment by justinfr...@gmail.com on 27 Mar 2012 at 5:22

GoogleCodeExporter commented 8 years ago

KUCB-FM is another. Does look like they fixed the feed on the other end.

Moving out of 2.3 release.

Original comment by jpenn...@gmail.com on 16 Apr 2012 at 7:31

Added labels: Milestone-2.4
Removed labels: Milestone-2.3

GoogleCodeExporter commented 8 years ago

To clarify, WXXI fixed their feed, KUCB still crashes.

Original comment by jpenn...@gmail.com on 23 Apr 2012 at 7:20

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 2 May 2012 at 6:15

GoogleCodeExporter commented 8 years ago

The app will now display a toast that it's unable to play the item and clear 
the player when no url is available.

Also, since KUCB doesn't include a url for the audio in their stories in their 
feed, should the application even display the podcast stories?  Note that if 
you say no, the user would see an empty list when they click on the podcast.  
If we then wanted to not show empty podcasts, it would entail loading and 
parsing all podcasts when the station page is shown.

Original comment by justinfr...@gmail.com on 16 May 2012 at 5:26

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

Original comment by justinfr...@gmail.com on 16 May 2012 at 5:26

GoogleCodeExporter commented 8 years ago

Original comment by jpenn...@gmail.com on 13 Nov 2012 at 10:28

Changed state: Live