ICEfaces
  1. ICEfaces
  2. ICE-5728

ViewState is not applied to all forms

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Alpha3, 2.0.0
    • Component/s: Bridge, Framework
    • Labels:
      None
    • Environment:
      ICEfaces 2 JSF 2

      Description

      Based on the navigation test (samples/test/nav) It looks like there may be a problem during certain navigation scenarios.

      If you click on a button that causes a full page replacement (ie a ViewRoot update), then the ViewState does not appear to be re-applied to all the forms in the page. This leads to the next submission likely not working properly.

      In the navigation test identified above, if you click a button that posts back to the same page and:

      - with stock JSF 2, has f:ajax render='all' or
      - is enabled with ICEfaces

      the resulting full page update is applied but the ViewState update is not applied to all the forms.

      If you then click a button that attempts to navigate to page 2, it will not work because the ViewState is not included. Instead, you'll get another ViewRoot update of the full page, which will then properly apply the ViewState to all the forms and the next click will work as designed and navigate you to page 2.

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21461 Sun May 23 03:22:29 MDT 2010 mircea.toma ICE-5728 Apply fix only for ICEfaces enabled views.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/JSFViewStateNavigationFix.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21444 Tue May 18 14:01:00 MDT 2010 mircea.toma ICE-5728 Simplify code by using UIOutputWriter component.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/WindowAndViewIDSetup.java
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/JSFViewStateNavigationFix.java
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/BridgeSetup.java
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/UIOutputWriter.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21443 Tue May 18 13:47:25 MDT 2010 mircea.toma ICE-5728 Enclose script element with 'span' so that when DOM update needs to re-evaluate the script can lookup the enclosing element.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/BridgeSetup.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21440 Tue May 18 10:43:46 MDT 2010 deryk.sinotte ICE-5728: setting explicit id to prevent duplicate id problems
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/JSFViewStateNavigationFix.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21438 Tue May 18 10:03:50 MDT 2010 deryk.sinotte ICE-5728: disable listening for now
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/JSFViewStateNavigationFix.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21437 Tue May 18 08:03:31 MDT 2010 mircea.toma ICE-5728 Introduce system event listener that renders javax.faces.ViewState field into the forms when old DOM is missing due to navigation.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/resources/META-INF/faces-config.xml
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/context/DOMResponseWriter.java
          Commit graph ADD /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/JSFViewStateNavigationFix.java

            People

            • Assignee:
              Mircea Toma
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: