ICEfaces
  1. ICEfaces
  2. ICE-1187

NullPointerException with Seam and JSF 1.2

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.1
    • Fix Version/s: 1.5.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Operating System: Mac OS X 10.0
      Platform: Macintosh

      Description

      When the Seam ICEfaces example is configured to use JSF 1.2, the following NullPointerException is
      thrown when the first page is viewed:

      Caused by: java.lang.NullPointerException
              at com.icesoft.faces.application.D2DViewHandler.calculateRenderKitId(D2DViewHandler.java:696
      )
              at com.sun.faces.lifecycle.RestoreViewPhase.isPostback(RestoreViewPhase.java:241)
              at com.sun.faces.lifecycle.RestoreViewPhase.execute(RestoreViewPhase.java:170)
              at com.sun.faces.lifecycle.LifecycleImpl.phase(LifecycleImpl.java:244)

        Activity

        Hide
        Ken Fyten added a comment -

        Fixed in 1.5.2.

        Show
        Ken Fyten added a comment - Fixed in 1.5.2.
        Hide
        Ted Goddard added a comment -

        svn commit core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java -m "calling superclass
        constructor with ViewHandler for decorator pattern (ICE-1187)"
        Sending core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java
        Transmitting file data .
        Committed revision 12911.

        Reassigning to Greg to confirm that this change does not adversely affect Seam functionality.

        Show
        Ted Goddard added a comment - svn commit core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java -m "calling superclass constructor with ViewHandler for decorator pattern ( ICE-1187 )" Sending core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java Transmitting file data . Committed revision 12911. Reassigning to Greg to confirm that this change does not adversely affect Seam functionality.
        Hide
        Ted Goddard added a comment -

        A post on the JBoss forum

        http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4002322

        indicated that our D2DSeamFaceletViewHandler was missing a constructor required for
        ViewHandler decorating. Initial testing shows ICEfaces working with Seam and JSF 1.2 with
        the following patch:

        — core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java (revision 12910)
        +++ core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java (working copy)
        @@ -1,5 +1,6 @@
        package com.icesoft.faces.facelets;

        +import javax.faces.application.ViewHandler;

        import com.sun.facelets.compiler.Compiler;
        import com.sun.facelets.compiler.SAXCompiler;
        @@ -17,6 +18,10 @@
        // Log instance for this class
        private static Log log = LogFactory.getLog(D2DSeamFaceletViewHandler.class);

        + public D2DSeamFaceletViewHandler(ViewHandler delegate)

        { + super(delegate); + }

        +
        protected void faceletInitialize() {

        Show
        Ted Goddard added a comment - A post on the JBoss forum http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4002322 indicated that our D2DSeamFaceletViewHandler was missing a constructor required for ViewHandler decorating. Initial testing shows ICEfaces working with Seam and JSF 1.2 with the following patch: — core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java (revision 12910) +++ core/src/com/icesoft/faces/facelets/D2DSeamFaceletViewHandler.java (working copy) @@ -1,5 +1,6 @@ package com.icesoft.faces.facelets; +import javax.faces.application.ViewHandler; import com.sun.facelets.compiler.Compiler; import com.sun.facelets.compiler.SAXCompiler; @@ -17,6 +18,10 @@ // Log instance for this class private static Log log = LogFactory.getLog(D2DSeamFaceletViewHandler.class); + public D2DSeamFaceletViewHandler(ViewHandler delegate) { + super(delegate); + } + protected void faceletInitialize() {

          People

          • Assignee:
            Greg Dick
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: