commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Yandell (JIRA)" <j...@apache.org>
Subject [jira] Commented: (BEANUTILS-92) PropertyUtilsBean.copyProperties does not catch NoSuchMethodException
Date Tue, 17 Jul 2007 18:32:04 GMT

    [ https://issues.apache.org/jira/browse/BEANUTILS-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12513317
] 

Henri Yandell commented on BEANUTILS-92:
----------------------------------------

I've committed a modified version of your patches as r557008, it felt easier than adding another
attachment etc:

http://svn.apache.org/viewvc?view=rev&revision=557008

How does that look?

> PropertyUtilsBean.copyProperties does not catch NoSuchMethodException
> ---------------------------------------------------------------------
>
>                 Key: BEANUTILS-92
>                 URL: https://issues.apache.org/jira/browse/BEANUTILS-92
>             Project: Commons BeanUtils
>          Issue Type: Bug
>          Components: Bean / Property Utils
>    Affects Versions: 1.7.0
>         Environment: Operating System: other
> Platform: Other
>            Reporter: Will Pugh
>            Assignee: Niall Pemberton
>             Fix For: 1.8.0
>
>         Attachments: Beanutils-92-updated.patch, fixCopyPropertyException, Jira92TestCase.java
>
>
> I ran into a problem where I had a bean that had an IndexedSetter but no simple
> setter.  This caused a NoSuchMethodException to get thrown in
> PropertyUtilsBean.copyProperties.  This is inconsistant with BeanUtilsBean which
> catches this case and continues copying the other properties.
> When I asked about this in on the mailing list, the answer seemed to come back
> that this is probabaly incorrect behaviour, but it is possible people depend on
> this behaviour so this might be too big a change for a point release.  I'm
> attaching the patch so it can be added to the next major release (if it is
> determined to be incorrect behaviour).
> The scenario I ran into this was one where I had a bean that I then used CGLib
> for enhancing.  After that, the bean failed to be clonable by BeanUtils.clone.
> This could potentially become a big deal since Hibernate used CGLib for adding
> proxies to beans for lazy loading.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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