AttributeError: 'PhotoAlbumsScreen' object has no attribute 'menu'

Bug #315242 reported by Marco Rodrigues
2
Affects Status Importance Assigned to Milestone
Entertainer Media Center
Fix Released
Critical
Matt Layman

Bug Description

I use current version 0.3

TESTCASE:
1- Run entertainer
2- Go to menu Photographs
3- Use keys UP/Down

Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/entertainerlib/frontend/gui/user_interface.py", line 458, in handle_keyboard_event
    self.handle_user_event(UserEvent(UserEvent.NAVIGATE_SELECT))
  File "/usr/lib/python2.5/site-packages/entertainerlib/frontend/gui/user_interface.py", line 564, in handle_user_event
    self.current.handle_user_event(event)
  File "/usr/lib/python2.5/site-packages/entertainerlib/frontend/gui/screens/photoalbums_screen.py", line 328, in handle_user_event
    album = self.menu.get_current_menuitem().get_userdata()
AttributeError: 'PhotoAlbumsScreen' object has no attribute 'menu'

Thanks go to Piotr from Debian PAPT Team for finding this out.

Related branches

Revision history for this message
Samuel Buffet (samuel-buffet) wrote : Re: [Bug 315242] [NEW] AttributeError: 'PhotoAlbumsScreen' object has no attribute 'menu'

Marco,

Thank you very much for your report.

In that screen I see

        if self.library.get_number_of_images() == 0:
            self._create_no_photos_information()
        else:
            # Album preview group
            self.preview = clutter.Group()
            self.preview.set_position(self.ui.x(0.07), self.ui.y(0.1953))
            ...
            >> self.menu = None <<
            (variables set to None)
            >> self._create_album_menu() <<

so my question is :

Do you have the
            "There are no indexed photographs in the Entertainer media "
            "library. To add photographs, start the Content management tool "
            "and open the 'Images' tab. Now click on the 'Add' button and "
            "select some folders which contain image files."

message or not ?

If it's the case then the fix is trivial. Otherwise ??

Thanks again (and to Piotr also ;-))

Revision history for this message
Paul Hummer (rockstar) wrote :

This warrants a point release, as it is blocking our package in Debian. Please put this at the highest priority.

Changed in entertainer:
importance: Undecided → Critical
milestone: none → entertainer-0.3.1
status: New → Triaged
Revision history for this message
Matt Layman (mblayman) wrote :

I may have hosed this on my tab refactoring. I know the fix immediately. I can work this out now.

Changed in entertainer:
assignee: nobody → laymansterms
Revision history for this message
Marco Rodrigues (gothicx) wrote :

Hi!

Yes Samuel, that's the message that appears me, so go ahead with your fix.

Thanks

Matt Layman (mblayman)
Changed in entertainer:
status: Triaged → Fix Committed
Matt Layman (mblayman)
Changed in entertainer:
milestone: entertainer-0.3.1 → entertainer-0.4
Paul Hummer (rockstar)
Changed in entertainer:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.