Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#3460 closed defect (fixed)

traceback in Browse for 'set_invoker'

Reported by: erikos Owned by: erikos
Priority: Unspecified by Maintainer Milestone:
Component: Sugar 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 (5)

comment:1 Changed 8 years 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.

comment:2 Changed 8 years ago by erikos

  • Keywords patch added

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.

comment:3 Changed 8 years ago by erikos

  • Resolution set to fixed
  • Status changed from new to closed

Pushed as d57212a040434140482f5371494bc116cbc9360b

comment:4 Changed 7 years ago by dnarvaez

  • Component changed from sugar-toolkit-gtk3 to Sugar

comment:5 Changed 7 years ago by dnarvaez

  • Milestone 0.96 deleted

Milestone 0.96 deleted

Note: See TracTickets for help on using tickets.