On Sat, Aug 15, 2009 at 2:57 AM, Jeanne
Waldman<[email protected]> wrote:
> If I change Trinidad's demo faces-config.xml file to use a bogus
> default-render-kit-id, I get a NPE.
>
> <!-- Use the Trinidad RenderKit -->
> <default-render-kit-id>
> org.apache.myfaces.trinidad.coreBAD
> </default-render-kit-id>
>
> I get this:
>
> java.lang.NullPointerException
> at
> com.sun.faces.renderkit.RenderKitUtils.getResponseStateManager(RenderKitUtils.java:246)
> at
> com.sun.faces.lifecycle.RestoreViewPhase.isPostback(RestoreViewPhase.java:267)
> at
> com.sun.faces.lifecycle.RestoreViewPhase.execute(RestoreViewPhase.java:172)
> at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:100)
> at
> com.sun.faces.lifecycle.RestoreViewPhase.doPhase(RestoreViewPhase.java:104)
> Truncated. see log file for complete stacktrace
>>
>
> This seems to me to be a bug in
> com.sun.faces.renderkit.RenderKitUtils.getResponseStateManager:
>
> 3 renderKit = factory.getRenderKit(context, renderKitId);
> 244 }
> 245 }
> 246 return renderKit.getResponseStateManager();
>
> Has anyone seen this or have an opinion about this? I would have liked to
> have had a log message telling me why I got a NPE at least so I didn't have
> to track it down.
Question: Is the myfaces jsf-impl better here?
I don't know as I haven't done something like the above.
Sure, on first thought the bug is kinda stupid, but heck - typos can happen :-)
So a better "warning" like a FAcesException ("there is no damn
'org.apache.myfaces.trinidad.coreBAD' renderkit")
would be way better.
-M
>
> I suppose I could write out a warning message in Trinidad's
> FacesContextFactoryImpl.java if it returns null.
>
> Thanks,
> Jeanne
>
>
--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
