Re: [FunFormKit-discuss] RadioField bug

[Luke O. <lu...@me...>]

Quoting Edmund Lian <el...@in...>:

>
> On 04/09/2003 10:11:06 AM funformkit-discuss-admin wrote:
>
> >Not sure if this is in CVS? Just a "+" instead of "%".
> >
> >- Luke
> >
> >class RadioField(SelectField):
> >
> >    def htInputRender(....):
> >        ...
> >            out.write(html.label(
> >>>              for_='%s_%i' + (nameMap(self.name()),
> id),
> ><<              for_='%s_%i' % (nameMap(self.name()),
> id),
> >                c=htmlEncode(value)))
> >            out.write(html.br())
>
> What's the effect of having a "+" there? I've not noticed
> any problems, and
> looking at the code, it looks like a % is correct since
> there is a string
> substitution going on.
>
> ...Edmund.

Um, I guess my email was ambiguous. There IS a + there, and
it SHOULD be a %. Just checked CVS, it's a + there as well.
To sum up, the fix is to change it to a %, because (like
you  say) there is string substitution going on.

The error you'll get is "cannot add string to tuple" with
the current code.

- Luke