Entertainer Media Center

Attribute error when no feeds set

Bug #261005 reported by Matt Layman on 2008-08-25

Affects		Status	Importance	Assigned to	Milestone
	Entertainer Media Center	Fix Released	High	Jamie Bennett	Entertainer Media Center entertainer-0.2 "Madagascar"

Bug Description

I had no feeds in my library (I removed the default feeds) and I got the following traceback:

matt@eden:~/branches/entertainer$ ./entertainer

** (./entertainer:13337): WARNING **: failed to bind GLXGetProcAddress or GLXGetProcAddressARB

Entertainer backend starting...
Exception in thread FeedFetcher:
Traceback (most recent call last):
  File "/usr/lib/python2.5/threading.py", line 486, in __bootstrap_inner
    self.run()
  File "/home/matt/branches/entertainer/entertainerlib/backend/components/feeds/feed_fetcher.py", line 61, in run
    data.feed.title)
  File "/var/lib/python-support/python2.5/feedparser.py", line 236, in __getattr__
    raise AttributeError, "object has no attribute '%s'" % key
AttributeError: object has no attribute 'title'

Related branches

lp:~joshuascotton/entertainer/entertainer-feed-fixes

Merged into lp:entertainer at revision 281

Matt Layman: Pending requested 2008-09-23

Paul Hummer: Pending requested 2008-09-23

lp:~jamiebennett/entertainer/Bug-261005

Merged into lp:entertainer at revision 288

Joshua Scotton: Approve on 2008-10-13

Entertainer Release Team: Pending requested 2008-10-13

Revision history for this message

Paul Hummer (rockstar) wrote on 2008-09-01:

This bug is a duplicate of bug 258449

Changed in entertainer:
status:	New → Invalid

Revision history for this message

Matt Layman (mblayman) wrote on 2008-09-01:

I'd like to recirculate this bug. The traceback is different from bug 258449 so I think it could be a different issue that happens to have a different AttributeError. I'd at least like to discuss why this was marked as a dup.

Changed in entertainer:
status:	Invalid → New

Revision history for this message

Paul Hummer (rockstar) wrote on 2008-09-19:

This is a duplicate of bug 258449

Changed in entertainer:
assignee:	nobody → rockstar
status:	New → Invalid

Revision history for this message

Paul Hummer (rockstar) wrote on 2008-09-22:

Oh bother, I didn't see that I had already tried to mark this as a duplicate. I think the other bug has an incorrect title, and that's what threw me off. It MAY be the same issue, but I can't be sure now. Maybe we'll just have to close two bugs with the same fix.

Changed in entertainer:
assignee:	rockstar → nobody
importance:	Undecided → High
milestone:	none → entertainer-0.2
status:	Invalid → Triaged

Revision history for this message

Joshua Scotton (joshuascotton) wrote on 2008-09-22:

I'll take a look into this

Changed in entertainer:
assignee:	nobody → joshuascotton

Joshua Scotton (joshuascotton) on 2008-09-23

Changed in entertainer:
status:	Triaged → In Progress

Paul Hummer (rockstar) on 2008-10-04

Changed in entertainer:
status:	In Progress → Fix Committed

Revision history for this message

Matt Layman (mblayman) wrote on 2008-10-06:

This was not actually fixed in the branch that Josh committed. A fresh checkout of the most up to date trunk can reproduce this problem with the following steps: Go into the content manager and remove the default feeds. Start Entertainer. This should be enough to trigger the traceback that I initially listed.

Changed in entertainer:
status:	Fix Committed → Triaged

Revision history for this message

Joshua Scotton (joshuascotton) wrote on 2008-10-06:

Ok, I'll fix this during the sprint next weekend

Revision history for this message

Jamie Bennett (jamiebennett) wrote on 2008-10-10:

I have a fix for this here (just test for empty feed list) but even when the list is empty, the old feeds still show under the 'Headlines' menu. These should be removed?

Revision history for this message

Jamie Bennett (jamiebennett) wrote on 2008-10-10:

I've pushed my trivial fix for this bug at:

https://code.edge.launchpad.net/~jamiebennett/entertainer/Bug-261005

I'm still thinking that if the rss feed is removed from the content.conf file and entertainer restarted, the old feeds should be removed (how else can you remove them?).

Revision history for this message

Joshua Scotton (joshuascotton) wrote on 2008-10-10:

#10

>>I'm still thinking that if the rss feed is removed from the content.conf file and entertainer restarted, the old feeds should be removed (how else can you remove them?).

yep, that is correct and hopefully we'll get this side of it working when Paul's new database backend is merged into the system. We should open a separate bug for that.

The reason the feeds are still there is because they're still in the cache. Delete the cache and remove the rss feed from content.conf then restart entertainer using your fix branch. That'll confirm your fix either way.

Joshua Scotton (joshuascotton) on 2008-10-13