I'd personally rather have it throw an exception - it makes it harder to
catch errors if nulls are ignored in general.
-- Adam
On 8/28/07, Simon Lessard <[EMAIL PROTECTED]> wrote:
> Hello all,
>
> Currenty, FacesBean.getProperty(null) throws a NullPointerException from the
> checkNotListKey method call. However, I feel it should rather return null
> and not throw an exception. That way, we would no longer have to use the
> following code snippet in our renderer to know if the property is supported
> on any given component:
> if (someKey == null)
> {
> return null;
> }
> else
> {
> return ComponentUtils.resolveSomeType(bean.getProperty(someKey),
> someDefaultValue);
> }
>
> If getProperty were to return null instead of throwing an exception, only
> return line would be needed, reducing some boilerplate code in the various
> renderers.
>
> Any objection to make that change to FacesBeanImpl and reflect it in the
> FacesBean's javadoc?
>
>
> Regards,
>
> ~ Simon
>
