Ticket #3460 (closed defect: fixed)

Opened 14 months ago

Last modified 13 months ago

traceback in Browse for 'set_invoker'

Reported by: erikos Owned by: erikos
Priority: Unspecified by Maintainer Milestone: 0.96
Component: sugar-toolkit-gtk3 Version: Git as of bugdate
Severity: Minor Keywords: 12.1.0, patch
Cc: Distribution/OS: OLPC
Bug Status: New

Description

palettewindow.py line 499 'NoneType' object has no attribute 'set_invoker'

Change History

Changed 14 months ago by erikos

I did a more detailed trace of this, and this points to the _ToolbarPalette(PaletteWindow) in toolbarbox. Some refactoring might be needed here to make that code clearer.

Changed 13 months ago by erikos

  • keywords 12.1.0, patch added; 12.1.0 removed

The _ToolbarPalette does get passed the invoker on initialisation. But we do create the PaletteWindowWidget later. We do attach the invoker to the widget when calling _setup_widget that is why it was still working without before.

I sent a patch that prevents the traceback that we had because of not having a widget at this point.

Changed 13 months ago by erikos

  • status changed from new to closed
  • resolution set to fixed

Pushed as d57212a040434140482f5371494bc116cbc9360b

Note: See TracTickets for help on using tickets.