ICEfaces
  1. ICEfaces
  2. ICE-1435

Ajax Push sometimes fails to update page without concurrentDOMViews

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6DR#2
          • Fix Version/s: 1.6
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            all
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Obtain a reference to the PersistentFaceState in a bean getter/setter rather than the constructor as a getter is more likely to be called specifically on the user's current page.

                    persistentFacesState = PersistentFacesState.getInstance();
            Show
            Obtain a reference to the PersistentFaceState in a bean getter/setter rather than the constructor as a getter is more likely to be called specifically on the user's current page.         persistentFacesState = PersistentFacesState.getInstance();

            Description

            With concurrentDOMViews set to false, although a render is requested, the current page may not update in some cases. This may be due to the PersistentFacesState being associated with a previously viewed page.

             

              Activity

              Ascending order - Click to sort in descending order
              Ted Goddard created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Fix Version/s 1.6DR#4 [ 10060 ]
              Ken Fyten made changes -
              Assignee Priority P2
              Assignee Mircea Toma [ mircea.toma ]
              Hide
              Permalink
              Mircea Toma added a comment -

              Is there a way to reproduce this reliably?

              Show
              Mircea Toma added a comment - Is there a way to reproduce this reliably?
              Hide
              Permalink
              Ted Goddard added a comment -

              The best way would be to get Brad's security demo in its original form (the checked-in version might have a workaround in place).

              Show
              Ted Goddard added a comment - The best way would be to get Brad's security demo in its original form (the checked-in version might have a workaround in place).
              Ken Fyten made changes -
              Fix Version/s 1.6 [ 10031 ]
              Fix Version/s 1.6DR#4 [ 10060 ]
              Hide
              Permalink
              Ted Goddard added a comment -

              Brad has verified that obtaining the PersistentFacesState in the constructor does not work.

              Show
              Ted Goddard added a comment - Brad has verified that obtaining the PersistentFacesState in the constructor does not work.
              Ken Fyten made changes -
              Assignee Priority P2 P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #14086 Thu Jun 07 16:24:54 MDT 2007 mircea.toma Reuse PersistentFacesState instance on redirects -- ICE-1435
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/xmlhttp/PersistentFacesState.java
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/ViewBoundAdaptingServlet.java
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/UploadServlet.java
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/command.js
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.async.js
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/ServletSessionMap.java
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.js
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/MultiViewServlet.java
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/application.js
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/ServletView.java
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/SingleViewServlet.java
              Hide
              Permalink
              Mircea Toma added a comment -

              Reuse PersistentFacesState instance on redirects.

              Show
              Mircea Toma added a comment - Reuse PersistentFacesState instance on redirects.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Ted Goddard added a comment -

              Brad, could you test your security application with PersistentFacesState.getInstance() in the bean constructor?

              Show
              Ted Goddard added a comment - Brad, could you test your security application with PersistentFacesState.getInstance() in the bean constructor?
              Ted Goddard made changes -
              Assignee Mircea Toma [ mircea.toma ] Brad Kroeger [ brad.kroeger ]
              Hide
              Permalink
              Brad Kroeger added a comment -

              I tested and it now works with the call in the constructor.

              Show
              Brad Kroeger added a comment - I tested and it now works with the call in the constructor.
              Brad Kroeger made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Closed [ 6 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P1
              Assignee Brad Kroeger [ brad.kroeger ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Ted Goddard
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: