On Tue, 2003-12-02 at 15:49, Vadim Gritsenko wrote:
> [EMAIL PROTECTED] wrote:
> 
> >   Index: PlainBooleanConvertor.java
> >  ===================================================================
> >  RCS file: 
> > /home/cvs/cocoon-2.1/src/blocks/woody/java/org/apache/cocoon/woody/datatype/convertor/PlainBooleanConvertor.java,v
> >  retrieving revision 1.1
> >  retrieving revision 1.2
> >  diff -u -r1.1 -r1.2
> >  --- PlainBooleanConvertor.java     12 Aug 2003 12:54:45 -0000      1.1
> >  +++ PlainBooleanConvertor.java     2 Dec 2003 14:44:56 -0000       1.2
> >  @@ -7,6 +7,9 @@
> >    */
> >   public class PlainBooleanConvertor implements Convertor {
> >       public Object convertFromString(String value, Locale locale, 
> > Convertor.FormatCache formatCache) {
> >  +        if (value == null || value.length() == 0) {
> >  +            return "";
> >  +        }
> >           return Boolean.valueOf(value);
> >       }
> >  
> >
> 
> Can somebody else confirm that this is ok? Works for me...

My intuition tells me that a BooleanConvertor should return a Boolean
object.

What's the purpose of this, IOW what is affected by this?

-- 
Bruno Dumon                             http://outerthought.org/
Outerthought - Open Source, Java & XML Competence Support Center
[EMAIL PROTECTED]                          [EMAIL PROTECTED]

Reply via email to