commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robert burrell donkin <robertdon...@mac.com>
Subject Re: [PATCH][BeanUtils] NoSuchMethod for read only properties and NullPointer when populating primitives
Date Mon, 27 May 2002 21:25:01 GMT
hi Tomas

i've taken a look at the patches and - at first glance - i like what i see.

but...

i'm a bit reluctant to change anything in this part of beanutils without 
the ok from craig - but he's pretty busy at the moment.

so...

i'll leave it a couple of days before i take any action. that way, anyone 
who wants to say something has the chance to do so. (this will also give 
me the chance to make sure i haven't missed anything.)

- robert

On Sunday, May 26, 2002, at 06:17 PM, Tomas Viberg wrote:

> Comments, anyone?
>
> Regards
> / Tomas
>
>
> ----- Original Message -----
> From: Tomas Viberg
> To: commons-dev@jakarta.apache.org
> Sent: Sunday, May 19, 2002 7:29 PM
> Subject: [PATCH] BeanUtils - read only properties and null value 
> conversion
>
>
> Hello,
>
> I humbly submit a patch for BeanUtils.java, correcting the following 
> errors:
>
> 1. When you try to populate a read-only property, the populate method 
> throws a NoSuchMethodException (wrapped in an InvocationTargetException), 
> since there is no check that the property has a setter method.
>
> 2. When you try to populate a primitive property with a null value, you 
> get a NullPointerException, since no data type conversion is applied to 
> null values.
>
> I have also patched BeanUtilsTestCase.java to include setting a primitive 
> property to null and attempting to set a read-only (simple) property.
>
> Regards,
> Tomas Viberg, tomas@viberg.nu
>
>
>
> --
> To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.
> org>
> For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.
> org>
>   --
> To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.
> org>
> For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.
> org>


--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message