ICEfaces
  1. ICEfaces
  2. ICE-4581

Add id to conversation Id Parameter with Seam so entire form is not included in DomDiff when pushed

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Seam, ajax-push

      Description

      From http://www.seamframework.org/Community/ConversationIdHiddenFieldAndICEFaces investigation showed that the conversation id was included on the form in a hidden field but that no id was associated with it (use <s:link> for redirection/navigation to show this hidden form field).

      If the cid is required to be a hidden field on the form, then the id should also be the conversation Id parameter and class com.icesoft.faces.renderkit.dom_html_basic.FormRenderer should be modified for this.

       


        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #19226 Fri Sep 04 11:47:44 MDT 2009 greg.dick ICE-4581 packaging change
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/core/LifecycleExecutor.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #19225 Fri Sep 04 11:43:30 MDT 2009 greg.dick ICE-4581 packaging change
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/env/SpringWebFlowInstantiationServlet.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #19222 Thu Sep 03 16:41:46 MDT 2009 greg.dick ICE-4581 Added an id attribute to the hidden input field containing the conversation id so that if it changes the dom diff update will be minimal
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/FormRenderer.java

          People

          • Assignee:
            Greg Dick
            Reporter:
            Judy Guglielmin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: