SelectFieldRenderer with two character option

14 views
Skip to first unread message

Peter

unread,
Jun 12, 2012, 10:19:59 AM6/12/12
to forma...@googlegroups.com
I ran across a minor bug in the SelectFieldRenderer class of fields module.

In the render method, there is a test len(L[0]) == 2. I think this is testing for a two element tuple but it also erroneously tests True for a string of length 2.

This was my case and although I got round the problem by avoiding selects which include two character choices, it may be worth changing the test to avoid this problem.

Keep up the good work.
Thanks,
Peter

Gaël Pasgrimaud

unread,
Jun 15, 2012, 10:43:00 AM6/15/12
to forma...@googlegroups.com
Right. Ca you open an issue on github ?

> Keep up the good work.
> Thanks,
> Peter
>
> --
> You received this message because you are subscribed to the Google Groups
> "FormAlchemy" group.
> To view this discussion on the web visit
> https://groups.google.com/d/msg/formalchemy/-/1Frk1BOR5q8J.
> To post to this group, send email to forma...@googlegroups.com.
> To unsubscribe from this group, send email to
> formalchemy...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/formalchemy?hl=en.
Reply all
Reply to author
Forward
0 new messages