[BEANUTILS-224] [beanutils] provide better error message for "argument type mismatch"

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.8.0
• Component/s: Bean / Property Utils

Environment

Operating System: All
Platform: Other

• Bugzilla Id: 38165
• Assignee: Unassigned
• Reporter: Ralf Hauser
• Created: 07/Jan/06 01:34
• Updated: 22/Apr/08 07:03
• Resolved: 07/Nov/06 05:32

Description

In <<Object org.apache.commons.beanutils.PropertyUtilsBean.invokeMethod(Method
method, Object bean, Object[] values) throws IllegalAccessException,
InvocationTargetException>>
the throw block should rather be:

throw new IllegalArgumentException(
"Cannot invoke " + method.getDeclaringClass().getName() + "."
+ method.getName() + " - " + e.getMessage() + " - found " +
bean.getClass().getName()+ " but expected "
+ (values[0]).getClass().getName());

see also In <<Object org.apache.commons.beanutils.PropertyUtilsBean.invokeMethod(Method
method, Object bean, Object[] values) throws IllegalAccessException,
InvocationTargetException>>
the throw block should rather be:

throw new IllegalArgumentException(
"Cannot invoke " + method.getDeclaringClass().getName() + "."
+ method.getName() + " - " + e.getMessage() + " - found " +
bean.getClass().getName()+ " but expected "
+ (values[0]).getClass().getName());

see also Bug 38163 for the context where this would have helped a lot. for the context where this would have helped a lot.