ICEfaces
  1. ICEfaces
  2. ICE-5705

DOM is null during navigation

          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: Framework
          • Labels:
            None
          • Environment:
            ICEfaces 2 JSF 2

            Description

            JSF 2 navigation always results in a new view root over the course of a JSF lifecycle, even if the view id is the same (ie postback). Unsure whether or not this is correct behaviour but we need to take this into account in various scenarios, especially involving partial page renders. The current detection that we do for navigation needs to be modified and simplified.

              Activity

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21394 Tue May 11 13:03:14 MDT 2010 deryk.sinotte ICE-5705: remove navigation detection and simply test for rendering when old DOM is null
              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/DOMPartialViewContext.java
              Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/DetectNavigationPhaseListener.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21396 Tue May 11 14:16:42 MDT 2010 ted.goddard removed null DOM logging (ICE-5705)
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/context/DOMPartialViewContext.java
              Deryk Sinotte created issue -
              Deryk Sinotte made changes -
              Field Original Value New Value
              Salesforce Case []
              Assignee Priority P2
              Assignee Deryk Sinotte [ deryk.sinotte ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21409 Thu May 13 14:19:57 MDT 2010 ted.goddard removed logging since null DOM is expected during navigation (ICE-5705)
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/context/DOMPartialViewContext.java
              Hide
              Permalink
              Deryk Sinotte added a comment -

              The null DOM logging has been removed/toned down because it's perfectly valid for the old DOM to be null during navigation. The old DOM reference is stored in the view map which is wiped out during navigation - including postbacks).

              Show
              Deryk Sinotte added a comment - The null DOM logging has been removed/toned down because it's perfectly valid for the old DOM to be null during navigation. The old DOM reference is stored in the view map which is wiped out during navigation - including postbacks).
              Deryk Sinotte made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Fix Version/s 2.0-Alpha3 [ 10032 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21420 Fri May 14 10:20:46 MDT 2010 ted.goddard removed unused import (ICE-5705)
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/java/com/icesoft/faces/context/CompatDOMPartialViewContext.java
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10230 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P2

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: