ICEfaces
  1. ICEfaces
  2. ICE-1711

Facelet View Handler throws nested forms FacesException when using a flow-through template

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Facelets & JSF (not necessarily just with Seam)
    • Affects:
      Compatibility/Configuration
    • Workaround Exists:
      Yes
    • Workaround Description:
      you have to create a <ui:insert name="value" for the possible values that will come along. Since the above seems to be a standard construct with facelets and the standard view handlers, we should make ours as easy to use as well.

      Description

      with the facelet page :-
      <ui:composition xmlns="http://www.w3.org/1999/xhtml"
                      xmlns:s="http://jboss.com/products/seam/taglib"
                      xmlns:ui="http://java.sun.com/jsf/facelets"
                      xmlns:f="http://java.sun.com/jsf/core"
                      xmlns:h="http://java.sun.com/jsf/html"
      xmlns:ice="http://www.icesoft.com/icefaces/component"
                      template="layout/template.xhtml">
                             
      <ui:define name="body">
          
          <h:messages globalOnly="true" styleClass="message" id="globalMessages"/>
          
          <ice:form id="testEditTemplate" styleClass="edit">
              
              <ice:panelGroup styleClass="edit">
                  
                  
                <s:decorate template="layout/display.xhtml">
                      <ui:define name="label">firstName</ui:define>
                      <h:inputText id="firstName" value="first testValue using s:decorate"/>
                  </s:decorate>
                  
                  <ui:decorate template="layout/display.xhtml">
                      <ui:define name="label">firstName</ui:define>
                      <h:inputText id="firstName" value="second testValue using ui:decorate"/>
                  </ui:decorate>
                  
              </ice:panelGroup>
          </ice:form>
         </ui:define>
      </ui:composition>

      and the following template:-
      <ui:composition xmlns="http://www.w3.org/1999/xhtml"
                       xmlns:ui="http://java.sun.com/jsf/facelets"
                       xmlns:h="http://java.sun.com/jsf/html"
                       xmlns:f="http://java.sun.com/jsf/core">
                       
          <div class="prop">
              <span class="name">
                  <ui:insert name="label"/>
              </span>
              <span class="value">
                  <ui:insert/>
              </span>
          </div>
          
      </ui:composition>

      either instance (the s:decorate and the ui:decorate) do not understand the flowthrough of the class="value" of the template and get the following error:-

      18:07:43,803 ERROR [D2DFaceletViewHandler] Problem in renderResponse: Nested form found on the page. The form action element can not be nested
      javax.faces.FacesException: Nested form found on the page. The form action element can not be nested
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.validateNestingForm(FormRenderer.java:362)
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.validateNestingForm(FormRenderer.java:365)
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.validateNestingForm(FormRenderer.java:365)
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.validateNestingForm(FormRenderer.java:365)
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.validateNestingForm(FormRenderer.java:365)
      at com.icesoft.faces.renderkit.dom_html_basic.FormRenderer.encodeBegin(FormRenderer.java:83)
      at javax.faces.component.UIComponentBase.encodeBegin(UIComponentBase.java:512)

        Activity

        Judy Guglielmin created issue -
        Hide
        Ted Goddard added a comment -

        Mark, please comment on the root cause.

        Show
        Ted Goddard added a comment - Mark, please comment on the root cause.
        Ted Goddard made changes -
        Field Original Value New Value
        Fix Version/s 1.6.1 [ 10070 ]
        Affects Version/s 1.6 [ 10031 ]
        Affects Version/s 1.6DR#5 [ 10071 ]
        Assignee Mark Collette [ mark.collette ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Fix Version/s 1.6.1 [ 10070 ]
        Ken Fyten made changes -
        Fix Version/s 1.6.1 [ 10070 ]
        Fix Version/s 1.7 [ 10080 ]
        Affects [Compatibility/Configuration]
        Assignee Priority P1
        Hide
        Mark Collette added a comment -

        It turns out that JBoss was shipping its own custom Facelets JAR with Seam 1.2.1 GA and Seam 1.3.0 which was derived from Facelets 1.1.12 plus their custom bugfix for this issue. Then for Seam 2.0.0 they started shipping stock Facelets 1.1.13a, which incorporates their bugfix. All that's documented here: http://fisheye.jboss.com/browse/JBoss/jboss-seam/lib/jsf-facelets.jar

        So I looked through the Facelets CVS logs and found these:

        /cvs/facelets/src/java/com/sun/facelets/tag/ui/DefineHandler.java
        revision 1.4
        date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +8 -2
        Added changes from Gavin King

        /cvs/facelets/src/java/com/sun/facelets/tag/ui/DecorateHandler.java
        revision 1.14
        date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +4 -4
        Added changes from Gavin King

        /cvs/facelets/src/java/com/sun/facelets/tag/ui/CompositionHandler.java
        revision 1.11
        date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +3 -3
        Added changes from Gavin King

        I've backported these Facelets 1.1.13a changes back into our own custom icefaces-facelets.jar which is derived from Facelets 1.1.11, plus other backported bugfixes. The rationale being that we don't have time to fully upgrade to Facelets 1.1.13a in time for our ICEfaces 1.6.1 release. We will do that for ICEfaces 1.7.0 though.

        Subversion 14730
        3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\DefineHandler.java
        3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\DecorateHandler.java
        3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\CompositionHandler.java

        Show
        Mark Collette added a comment - It turns out that JBoss was shipping its own custom Facelets JAR with Seam 1.2.1 GA and Seam 1.3.0 which was derived from Facelets 1.1.12 plus their custom bugfix for this issue. Then for Seam 2.0.0 they started shipping stock Facelets 1.1.13a, which incorporates their bugfix. All that's documented here: http://fisheye.jboss.com/browse/JBoss/jboss-seam/lib/jsf-facelets.jar So I looked through the Facelets CVS logs and found these: /cvs/facelets/src/java/com/sun/facelets/tag/ui/DefineHandler.java revision 1.4 date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +8 -2 Added changes from Gavin King /cvs/facelets/src/java/com/sun/facelets/tag/ui/DecorateHandler.java revision 1.14 date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +4 -4 Added changes from Gavin King /cvs/facelets/src/java/com/sun/facelets/tag/ui/CompositionHandler.java revision 1.11 date: 2007/05/06 17:35:04; author: jhook; state: Exp; lines: +3 -3 Added changes from Gavin King I've backported these Facelets 1.1.13a changes back into our own custom icefaces-facelets.jar which is derived from Facelets 1.1.11, plus other backported bugfixes. The rationale being that we don't have time to fully upgrade to Facelets 1.1.13a in time for our ICEfaces 1.6.1 release. We will do that for ICEfaces 1.7.0 though. Subversion 14730 3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\DefineHandler.java 3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\DecorateHandler.java 3rd\facelets-1.1\src\java\com\sun\facelets\tag\ui\CompositionHandler.java
        Mark Collette made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14678 Thu Aug 16 14:46:34 MDT 2007 mark.collette ICE-1711 : Facelet View Handler throws nexted forms FacesException when using a flow-through template
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/lib/icefaces-facelets.jar
        Hide
        Mark Collette added a comment -

        Forgot to commit the built JAR.

        Subversion 14678
        icefaces\lib\icefaces-facelets.jar

        Show
        Mark Collette added a comment - Forgot to commit the built JAR. Subversion 14678 icefaces\lib\icefaces-facelets.jar
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14702 Wed Aug 22 10:14:00 MDT 2007 judy.guglielmin icefaces-seam-gen templates for Seam 2 and ICEfaces 1.6.1 release --update for changes to jsf-facelets ICE-1710, ICE-1711 and changes to css file to reflect those changes
        Files Changed
        Commit graph MODIFY /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.6.1/icefaces/view/view.xhtml.ftl
        Commit graph MODIFY /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.6.1/icefaces/view/stylesheet/theme.css
        Commit graph MODIFY /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.6.1/icefaces/view/viewproperty.xhtml.ftl
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14717 Fri Aug 24 17:46:44 MDT 2007 judy.guglielmin icefaces-seam-gen templates for Seam 2 and ICEfaces 1.6.1 release --update for changes to jsf-facelets ICE-1710, ICE-1711 and changes to css file to reflect those changes
        Files Changed
        Commit graph MODIFY /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.6.1/icefaces/view/editproperty.xhtml.ftl
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14729 Fri Aug 24 18:40:59 MDT 2007 judy.guglielmin icefaces-seam-gen templates for Seam 1.2.1 and ICEfaces 1.6.1 release --update for changes to jsf-facelets ICE-1710, ICE-1711 and changes to css file to reflect those changes and to make the templates identical to the Seam2.0 templates so same selenium test script can be used on both
        Files Changed
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/cal-prev.png
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/web.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/display.xhtml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/orange-toggle-true.gif
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/hibernatetools/.settings/org.hibernate.eclipse.console.prefs
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/arrowDown.gif
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/template.xhtml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-test.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/hibernatetools/.settings
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/dtpick.gif
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/stylesheet
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/components.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/nbproject
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/cal-next.png
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/nbproject/project.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/nbproject/ide-file-targets.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/jboss-beans.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-dev-war.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/arrowUp.gif
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/components-war.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-test-war.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/jboss-app.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/menu.xhtml.ftl
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-prod-war.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/loginout.xhtml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/ejb-jar.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/faces-config.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/pages.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/pojo
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/orange-toggle-false.gif
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/menu.xhtml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/pojo/GetPropertyAnnotation.ftl
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/WEB-INF/jboss-web.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/nbproject/debug-jboss.properties
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/application.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/error.png
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/layout/edit.xhtml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-dev.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/resources/META-INF/persistence-prod.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/stylesheet/theme.css
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/view/img/ICEfaces-logo.gif
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14730 Fri Aug 24 18:44:55 MDT 2007 judy.guglielmin icefaces-seam-gen templates for Seam 1.2.1 and ICEfaces 1.6.1 release --update for changes to jsf-facelets ICE-1710, ICE-1711 and changes to css file to reflect those changes and to make the templates identical to the Seam2.0 templates so same selenium test script can be used on both
        Files Changed
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/build.xml
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/build.properties
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/USAGE
        Commit graph ADD /projects/icefaces-seam-gen/branches/icefaces-seam-gen-1.2ForIce1.6.1/README
        Ken Fyten made changes -
        Summary Facelet View Handler throws nexted forms FacesException when using a flow-through template Facelet View Handler throws nested forms FacesException when using a flow-through template
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P1
        Assignee Mark Collette [ mark.collette ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Judy Guglielmin
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: