Pidgeot / python-lnp

Cross-platform re-implementation of the Lazy Newb Pack launcher.
ISC License
65 stars 10 forks source link

population cap fails to update #128

Closed Pidgeot closed 4 years ago

Pidgeot commented 4 years ago

Changing the population cap results in an error trying to coerce the int value to unicode.

#!text

Exception in Tkinter callback
Traceback (most recent call last):
  File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 1489, in __call__
    return self.func(*args)
  File "/.../tkgui.py", line 1313, in set_pop_cap
    self.update_displays()
  File "/.../tkgui.py", line 1304, in update_displays
    value)
TypeError: coercing to Unicode: need string or buffer, int found

Attached patch (against tip) converts value with str(value)

[Issue created by beltorak: 2014-09-01]

[Comment created by Pidgeot: 2014-09-01] Fix error when changing population cap (fixes #20)

→ <<cset 122209304b08>>