commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rey Francois <Francois....@capco.com>
Subject RE: [beanutils][PATCH] PropertyUtils and indexedProperties
Date Wed, 07 Nov 2001 09:37:55 GMT
Well, my understanding of the JavaBean spec is that the only type allowed
for non-indexed getter/setter of an indexed property is an array. Reading
the spec 1.01, section 7.2, it does not specify any other type. This
conforms to the JDK 1.3.1 code i've looked at...
Fr.

-----Original Message-----
From: Tom Klaasen (TeleRelay) [mailto:tom.klaasen@telerelay.com]
Sent: 07 November 2001 10:37
To: Jakarta Commons Developers List
Subject: RE: [beanutils][PATCH] PropertyUtils and indexedProperties


Of course I tried it, but only on Struts (not on a generic javabeans
interpreter like an IDE). And it works for a non indexed getter based on
List.

Maybe I got the whole JavaBeans story wrong. However, if what you
describe is the case, then the JavaBeans mechanism seems even less
powerful than I thought. Seems a pity to me.

Is anybody sure that (s)he understands the JavaBeans spec, and whether
this patch is applicable?

Thanks,
tomK



> -----Original Message-----
> From: Rey Francois [mailto:Francois.Rey@capco.com] 
> Sent: woensdag 7 november 2001 10:22
> To: 'Jakarta Commons Developers List'
> Subject: RE: [beanutils][PATCH] PropertyUtils and indexedProperties
> 
> 
> Have you actually tried this code?
> I've only *briefly* looked into the JDK and BeanUtils code, 
> so I'm not 100%
> sure of this, but it seems to me that your patch may work 
> when your property
> only has non indexed getter/setter based on List, but should 
> fail if your
> property also has indexed getter/setter. In the latter case, 
> I would expect
> the JDK IndexedPropertyDescriptor class to throw an
> IntrospectionException("type mismatch between indexed and non-indexed
> methods"). Check the source of
> IndexedPropertyDescriptor.findIndexedPropertyType().
> 
> Again, I'm not 100% sure, I would need to test this but I 
> don't have time at
> present...
> 
> Fr.
> 
> -----Original Message-----
> From: Tom Klaasen (TeleRelay) [mailto:tom.klaasen@telerelay.com]
> Sent: 07 November 2001 09:55
> To: Jakarta Commons Developers List
> Subject: [beanutils][PATCH] PropertyUtils and indexedProperties
> 
> 
> Hi all,
> 
> I found a glitch in the PropertyUtils: you can only get an
> indexedProperty if its container is an array. But a java.util.List is
> also indexed. So it would be logical to be able to get an element of a
> List also as an indexedProperty.
> 
> Hence the patch attached.
> 
> Hope you enjoy it,
> 
> tomK
> **************************************************************
> **********
> The information in this email is confidential and is intended solely
> for the addressee(s).
> Access to this email by anyone else is unauthorised. If you are not
> an intended recipient, please notify the sender of this email 
> immediately. You should not copy, use or disseminate the 
> information contained in the email.
> Any views expressed in this message are those of the individual
> sender, except where the sender specifically states them to be
> the views of Capco.
> 
http://www.capco.com
***********************************************************************


--
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>
************************************************************************
The information in this email is confidential and is intended solely
for the addressee(s).
Access to this email by anyone else is unauthorised. If you are not
an intended recipient, please notify the sender of this email 
immediately. You should not copy, use or disseminate the 
information contained in the email.
Any views expressed in this message are those of the individual
sender, except where the sender specifically states them to be
the views of Capco.

http://www.capco.com
***********************************************************************


--
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