SK-Yang / torchat

Automatically exported from code.google.com/p/torchat
0 stars 0 forks source link

Settings menu causes an error with AddGrowableCol() #81

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Right click in the main menu, select settings

What is the expected output? What do you see instead?
Expected output would seem to be a settings Window. Instead I get this on stdout
(0) [_core,11002,PopupMenu] Traceback (most recent call last):
(0) [_core,11002,PopupMenu]   File 
"/Users/steve/Desktop/Downloads/src/tc_gui.py", line 444, in onSettings
(0) [_core,11002,PopupMenu] dialog = dlg_settings.Dialog(self.mw)
(0) [_core,11002,PopupMenu]   File 
"/Users/steve/Desktop/Downloads/src/dlg_settings.py", line 57, in __init__
(0) [_core,11002,PopupMenu] self.p1 = dlg.Panel(self.notebook)
(0) [_core,11002,PopupMenu]   File "/Users/steve/Desktop/Downloads/src/dlg.py", 
line 41, in __init__
(0) [_core,11002,PopupMenu] self.grid_sizer.AddGrowableCol(1)
(0) [_core,11002,PopupMenu]   File 
"/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-
packages/wx-2.9.1-osx_cocoa/wx/_core.py", line 14946, in AddGrowableCol
(0) [_core,11002,PopupMenu] return _core_.FlexGridSizer_AddGrowableCol(*args, 
**kwargs)
(0) [_core,11002,PopupMenu] wx._core
(0) [_core,11002,PopupMenu] .
(0) [_core,11002,PopupMenu] PyAssertionError
(0) [_core,11002,PopupMenu] :
(0) [_core,11002,PopupMenu] C++ assertion "!m_cols || idx < (size_t)m_cols" 
failed at ../src/common/sizer.cpp(1948) in AddGrowableCol(): invalid column 
index

What version of the product are you using? On what operating system?
Version 2011-10-13, Mac OS X, happens with both 10.5.8 and 10.6.6. Both are 
using wxWidgets 2.9.1 and wxPython 2.9.1.1.

Please provide any additional information below.
I think the line in dlg.py should be
        self.grid_sizer.AddGrowableCol(0)
At least, this works for me.

Original issue reported on code.google.com by schecko...@gmail.com on 14 Jan 2011 at 4:08

GoogleCodeExporter commented 9 years ago
can you test this with r475 please?

Original comment by prof7...@gmail.com on 16 Jan 2011 at 4:19

GoogleCodeExporter commented 9 years ago
Fixed.

Original comment by schecko...@gmail.com on 16 Jan 2011 at 4:43

GoogleCodeExporter commented 9 years ago

Original comment by prof7...@gmail.com on 16 Jan 2011 at 6:00