From: Stefan S. <ste...@tu...> - 2003-04-10 17:04:20
|
> I have a problem with FunFormKit 0.4.1. If I check some values in a > `MultiCheckboxField` display and the form is redisplayed after a > failed validation, the check marks are removed in the redisplayed > form. For comparison, `TextField`s and `RadioField`s work fine. I've found the bug and submitted a patch suggestion to Sourceforge. > Another problem I found in code for some fields; in several places the > format `"%s_%2i" % (name, id)` is used for ids. However, for ids lower > than 10 this results in names like "name_ 3" (note the space) which > may cause trouble with some browsers. Also there (and some other things :-) ). As I looked at the FunFormKit source code, I thought that it would simplify the code somewhat if - the first items in selections (for radio buttons and checkboxes) would be converted to strings as early in the processing as possible. So there's less need for code like if type(x) == types.StringType: ... else: ... In fact, such code was missing from MultiCheckboxMixin and that was the bug that bit me. - for multi-checkboxes, passed-in variables were always converted to a list, perhaps with only one element. This makes code simpler because you don't have to write if type(x) == types.ListType: ... else: ... to differentiate between lists and strings. What do you think? Stefan |