[BEANUTILS-306] LocaleConvertUtilsBean.convert throws NPE on null Locale when debug logging is enabled

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.7.0, 1.8.0-BETA
• Fix Version/s: 1.8.0
• Component/s: Locale BeanUtils / Converters
• Assignee: Niall Pemberton
• Reporter: Lucian Chirita
• Created: 15/Feb/08 15:57
• Updated: 25/Mar/08 08:08
• Resolved: 15/Feb/08 20:16

Description

LocaleConvertUtilsBean.convert(String value, Class clazz, Locale locale, String pattern) generally works with a null Locale argument, as the lookup(Locale locale) method checks whether the argument is null and uses the default locale in that case.

However, if debug logging is enabled for LocaleConvertUtils, the method throws NullPointerException on null locale because the debug message does locale.toString().

The same thing happens with the convert(String values[], Class clazz, Locale locale, String pattern) method.