[SANDBOX-371] [BeanUtils2] Make sure that a property is readable in DefaultBeanAccessor.getProperty( String name )

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Nightly Builds
• Component/s: BeanUtils2
• Assignee: Simone Tripodi
• Reporter: Benedikt Ritter
• Created: 30/Jan/12 11:55
• Updated: 30/Jan/12 15:10
• Resolved: 30/Jan/12 15:10

Description

Problem: The following statement in line 50 in DefaultBeanAccessor may cause a NullPointerException, because getReadMethod() will return null, if no getter for the property is present:

Object newBean = propertyDescriptor.getReadMethod().invoke( bean ); 

Solution: throw a NoSuchMethodException, if the property is write only.

if ( propertyDescriptor.getReadMethod() == null )
{
    throw new NoSuchMethodException( String.format( "Bean of type %s does not provide a getter for property '%s'!",
                                                 bean.getClass().getName(), name ) );
}