[BEANUTILS-333] Avoid calling setAccessible() if not needed

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

Environment

Windows XP Professional SP3, Xwork 2.1,

• Assignee: Niall Pemberton
• Reporter: Disabled - L-u-k-a-s-z Lenart - OpenSymphony
• Created: 16/Dec/08 23:10
• Updated: 25/Sep/16 22:16
• Resolved: 04/Aug/09 21:08

Description

I already solved the bug in Xwork (see http://jira.opensymphony.com/browse/XW-661), but the same problem exists with Common BeanUtils in org.apache.commons.beanutils.MethodUtils.getMatchingAccessibleMethod():558, add additional check if method is already accessible, like this

if (!method.isAccessible()) {
method.setAccessible(true);
}