commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 33228] - [beanutils] Can't access mapped property in some case
Date Tue, 25 Jan 2005 13:57:07 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33228>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33228





------- Additional Comments From nacho@visual-ma.com  2005-01-25 14:57 -------
Created an attachment (id=14095)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=14095&action=view)
Patch solving bug #33228

It seems that BeanUtils is bypassing its specifications (which is a good thing
in this case). You can find it here:
http://jakarta.apache.org/commons/beanutils/commons-beanutils-1.6.1/docs/api/org/apache/commons/beanutils/package-summary.html#overview.background


The good news is that the behaviour you want can be acheived with a couple of
minor changes which in turn would bring consistency:

* getAccessibleMethodFromInterfaceNest is checking for accesibility beginning
with interfaces of the superClass and not the class itself. 

* MappedPropertyDescriptor is not checking for its interfaces' accesibility, it
discards the method if it is not directly accessible.

So both of these can be acheived by changing a couple of lines of code. The
thing is that IMHO MappedPropertyDescriptor should have an init method to
perform the common things that are done in its constructors. This would make it
much easier to mantain. 

For the time being, the patch is attached with a slightly modified test case
(separate attachment).

Regards,

Nacho


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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


Mime
View raw message