[MainMenu]Main Menu applet fail trying to load .gtk-bookmarks

Bug #270647 reported by Julien Lavergne
2
Affects Status Importance Assigned to Milestone
Individual Screenlets
Fix Released
Medium
Unassigned
Universal Applets
Fix Committed
Undecided
Unassigned

Bug Description

Main Menu crash if there is no gtk-bookmarks. Using Debian Lenny up-to-date UA or Screenlets (same behavior for both).
Traceback :
  File "/usr/share/screenlets/MainMenu/MainMenuScreenlet.py", line 553, in <module>
    screenlets.session.create_session(MainMenuScreenlet)
  File "/usr/lib/python2.5/site-packages/screenlets/session.py", line 472, in create_session
    session.start()
  File "/usr/lib/python2.5/site-packages/screenlets/session.py", line 246, in start
    sl = self.screenlet(session=self, id=self.__get_next_id())
  File "/usr/share/screenlets/MainMenu/MainMenuScreenlet.py", line 193, in __init__
    model,self.objlist3 = menus.get_places(self.theme1)
  File "/usr/share/screenlets/MainMenu/menus.py", line 22, in get_places
    bookmark_list = open(book_loc,"r")
IOError: [Errno 2] No such file or directory: '/home/gilir/.gtk-bookmarks'

IMO, should use utils.LoodBookmark(), but don't know if it'll be enought.

Revision history for this message
Julien Lavergne (gilir) wrote :
Revision history for this message
Natan Yellin (aantny) wrote :

Thanks for the patch.

MainMenu's code is a nightmare, but I'll look into using utils.LoadBookmarks when I get a chance. Thanks.

Revision history for this message
Natan Yellin (aantny) wrote :

Committed in revision 115.

Changed in universal-applets:
status: New → Fix Committed
Revision history for this message
Julien Lavergne (gilir) wrote :

Fix commited in screenlets trunk revision 467

Changed in screenlets:
status: New → Fix Committed
importance: Undecided → Medium
Papp Bence (sclegnrbs)
affects: screenlets → indiv-screenlets
summary: - Main Menu applet fail trying to load .gtk-bookmarks
+ [MainMenu]Main Menu applet fail trying to load .gtk-bookmarks
Märt Põder (boamaod)
Changed in indiv-screenlets:
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.