[BEANUTILS] LocaleConvertUtilsBean convert method with null locale

2008-02-15 Thread Lucian Chirita 
Is the LocaleConvertUtilsBean.convert(String value, Class clazz, Locale 
locale, String pattern) method supposed to accept null Locale arguments?


This generally works, as the locale is used to resolve a converter, and 
the lookup(Locale locale) method checks whether the argument is null and 
uses the default locale in this case.


However, when debug statements are enabled for LocaleConvertUtilsBean, 
the convert methods fails with NullPointerException on null Locale as 
the debug statement does locale.toString().


Could this inconsistency be considered a bug?

Thanks,
Lucian

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: [BEANUTILS] LocaleConvertUtilsBean convert method with null locale

2008-02-15 Thread Niall Pemberton 
On Fri, Feb 15, 2008 at 1:39 PM, Lucian Chirita [EMAIL PROTECTED] wrote:
 Is the LocaleConvertUtilsBean.convert(String value, Class clazz, Locale
  locale, String pattern) method supposed to accept null Locale arguments?

  This generally works, as the locale is used to resolve a converter, and
  the lookup(Locale locale) method checks whether the argument is null and
  uses the default locale in this case.

  However, when debug statements are enabled for LocaleConvertUtilsBean,
  the convert methods fails with NullPointerException on null Locale as
  the debug statement does locale.toString().

  Could this inconsistency be considered a bug?

Yes please file a bug report in JIRA:

http://commons.apache.org/beanutils/issue-tracking.html

Thanks

Niall

  Thanks,
  Lucian

  -
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: [BEANUTILS] LocaleConvertUtilsBean convert method with null locale

2008-02-15 Thread Lucian Chirita 

Niall Pemberton wrote:

Yes please file a bug report in JIRA:

http://commons.apache.org/beanutils/issue-tracking.html

Thanks

Niall
  


I've logged BEANUTILS-306: 
http://issues.apache.org/jira/browse/BEANUTILS-306


Thank you,
Lucian

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]