[prev in list] [next in list] [prev in thread] [next in thread] 

List:       zope-dev
Subject:    Re: [Zope-dev] z3c.form bug with radio widget label
From:       Marius Gedminas <marius () gedmin ! as>
Date:       2008-12-12 16:10:59
Message-ID: 20081212161059.GB23960 () platonas
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Fri, Dec 12, 2008 at 02:33:53PM +0000, Johannes Raggam wrote:
> hey,
> 
> i guess i found a bug in z3c.form.browser.radio.
> 
> in z3c.form.browser.radio, line 49 (class RadioWidget, def update), the 
> line should read
> 
> label = term.value

You can't assume ``value`` will be a string.  People routinely use
all sorts of objects there.

> instead of label = term.token
> 
> we found this, while trying to display some unicode characters (umlaute) 
> in radio widget labels.

You should really be explicitly specifying the titles for your terms.

> this issue is still valid in current trunk (file revision 78513).
> 
> i report it here, because i could not find a bug tracker for z3c.form. is 
> there any?

https://bugs.launchpad.net/zope3

(Since https://bugs.launchpad.net/z3c.form doesn't exist.)

-- 
http://pov.lt/ -- Zope 3 consulting and development

["signature.asc" (application/pgp-signature)]

_______________________________________________
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )


[prev in list] [next in list] [prev in thread] [next in thread]