[Zope3-dev] Re: [Zope3-checkins] CVS: Zope3/src/zope/app/browser/form - configure.zcml:1.24 widget.py:1.58

[Martijn Faassen]

sree wrote:
> Update of /cvs-repository/Zope3/src/zope/app/browser/form
> In directory cvs.zope.org:/tmp/cvs-serv2121/browser/form
> 
> Modified Files:
> 	configure.zcml widget.py 
> Log Message:
> add views for file, image, dtml document, template file
> Also have added ASCII widget in replace ment for Bytes field..
> ASCII for text areas
> Bytes should now on be used only for File/Binary data...

I'm worried that the ASCII widget is actually accepting any 8 bit
encoded textual data. This means it isn't an ASCII widget but a 
'text in unknown encoding widget', which is bad and should be
avoided, unless we want to get obscure unicode errors pop up in
unexpected places.

If there's to be an ASCII widget there'd better be an ASCII field as
well which verifies at least whether every character is in the ASCII
range (7 bits). I imagine it'd be safe for this to be a subclass of
the BytesField, as it contains bytes but restricts what bytes are
acceptable.

Regards,

Martijn