Tkinter rowconfigure and columnconfigure functions crash if minsize, pad, or weight is not None

[aoileslie]

BPO	13519
Nosy	@birkenfeld, @terryjreedy, @serhiy-storchaka
Files	Reproduce.py: A python file to reproduce the problem

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2013-10-13.18:08:03.512>
created_at = <Date 2011-12-02.09:11:10.754>
labels = ['type-bug', 'expert-tkinter']
title = 'Tkinter rowconfigure and columnconfigure functions crash if minsize, pad, or weight is not None'
updated_at = <Date 2013-10-13.18:08:03.511>
user = 'https://bugs.python.org/aoileslie'

bugs.python.org fields:

activity = <Date 2013-10-13.18:08:03.511>
actor = 'georg.brandl'
assignee = 'none'
closed = True
closed_date = <Date 2013-10-13.18:08:03.512>
closer = 'georg.brandl'
components = ['Tkinter']
creation = <Date 2011-12-02.09:11:10.754>
creator = 'aoi.leslie'
dependencies = []
files = ['23835']
hgrepos = []
issue_num = 13519
keywords = []
message_count = 5.0
messages = ['148752', '148779', '148785', '197763', '199741']
nosy_count = 5.0
nosy_names = ['georg.brandl', 'terry.reedy', 'gpolo', 'aoi.leslie', 'serhiy.storchaka']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue13519'
versions = ['Python 2.7', 'Python 3.3', 'Python 3.4']

[aoileslie]

Symptom:
When use tkinter Widget class's rowconfigure or columnconfigure function (The two functions are defined in baseclass Misc.) to get the setting for a row or column (The setting is a dict containing fields 'minsize', 'pad', 'weight', and 'uniform'.), if field value of 'minsize', 'pad', or 'weight' is a positive integer instead of None, then error |TypeError: argument of type 'int' is not iterable| is raised. Field value of 'uniform' does not matter.

Speculation:
File |tkinter.__init__|, function |_grid_configure|, line 1279, code |elif '.' in value| caused this error. The code assumes the value is a str, but the value can be int.

Suggested Fix:
Change the code block around line 1279 to handle int value as well.

[terryjreedy]

Running on Win 7, 3.2.2, IDLE, I get
''' 
Traceback (most recent call last):
  File "F:\Python\mypy\tem.py", line 19, in <module>
    the_rowconfigure_info = the_frame.rowconfigure(the_row_index)
  File "C:\Programs\Python32\lib\tkinter\__init__.py", line 1326, in grid_rowconfigure
    return self._grid_configure('rowconfigure', index, cnf, kw)
  File "C:\Programs\Python32\lib\tkinter\__init__.py", line 1279, in _grid_configure
    elif '.' in value:
TypeError: argument of type 'int' is not iterable
'''
(For tracker purposes, this is a graceful exit, not a crash - as in *nix segfault or equivalent Windows error box.)
'''
>>> help(Frame.rowconfigure)
Help on function grid_rowconfigure in module tkinter:

grid_rowconfigure(self, index, cnf={}, **kw)
    Configure row INDEX of a grid.
    
    Valid resources are minsize (minimum size of the row),
    weight (how much does additional space propagate to this row)
    and pad (how much space to let additionally).
'''
The above implies that setting uniform=1 (in your code) works because it is ignored. From docs on the web, it appears that 'uniform' is valid and is just missing from our doc string. It is different from the other three, though, in jperhaps not being restricted to int values.

You are right that (line 1259)
def _grid_configure(self, command, index, cnf, kw):
expects strings (which is what tcl uses). I do not know enough, though, to know where the bug is. I do notice, however, that setting with a Python int matches online Python examples and that the code runs without the attempt to read the config.

[aoileslie]

Setting the config has no problem. Only reading has.

The config is read from Tk at line 1270 by code

            res = self.tk.call('grid',
                       command, self._w, index)

. If each of the four options (minsize, pad, uniform, and weight) has been set to value 1, |res|'s value after the tk call would be a tuple |('-minsize', 1, '-pad', 1, '-uniform', '1', '-weight', 1)|. This explains why |uniform|'s value does not cause problem, because it is a str, while the other three's are int. Also int 0 does not cause problem because it is handled at line 1277 by code

                if not value:
                    value = None

so the resulting option value appears as None in rowconfigure or columnconfigure's resulting dict.

In my speculation, the bug is that, when converting from the tuple returned by tk call into the resulting dict to be returned by rowconfigure or columnconfigure, the converting code assumes that the option values in the tuple returned by tk call are all str. But somehow only |uniform|'s value is str, while other three's are int.

[serhiy-storchaka]

I think this was fixed in bpo-16809.

[birkenfeld]

I agree.