[
https://issues.apache.org/jira/browse/BEANUTILS-92?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Niall Pemberton updated BEANUTILS-92:
-------------------------------------
Attachment: Beanutils-92-updated.patch
Attaching updated patch for PropertyUtilsBean's copyProperties() to catch
NoSuchMethodException. Because Maps and DynaBeans can be used as a facade to
wrap a "regular" bean I think NoSuchMethodException should be caught in all
three cases.
> 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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
