ICEfaces
  1. ICEfaces
  2. ICE-5671

Logic to detect navigation does not work if the viewID does not change.

    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

      We currently have a mechanism for determining whether or not navigation has occurred during the JSF lifecycle which relies on the view id changing. However, some navigation use cases are of a "loopback" style where navigation has occurred back to the original page and the view id does not change.

        Activity

        Hide
        Deryk Sinotte added a comment -

        I modified the logic so that it now looks for a marker in the view map rather than detecting if the view ID has changed. If the view map does not contain the marker, then we assume that the view has changed and navigation has occurred. We also now add the marker after the RESTORE_VIEW phase so that the marker is present in cases where INVOKE_APPLICATION is not invoked.

        Show
        Deryk Sinotte added a comment - I modified the logic so that it now looks for a marker in the view map rather than detecting if the view ID has changed. If the view map does not contain the marker, then we assume that the view has changed and navigation has occurred. We also now add the marker after the RESTORE_VIEW phase so that the marker is present in cases where INVOKE_APPLICATION is not invoked.
        Hide
        Deryk Sinotte added a comment -

        Re-opening because postbacks should not be considered navigation during Ajax requests.

        Show
        Deryk Sinotte added a comment - Re-opening because postbacks should not be considered navigation during Ajax requests.
        Hide
        Deryk Sinotte added a comment -

        Added a check for postback on Ajax requests to avoid doing full page updates in those cases.

        Show
        Deryk Sinotte added a comment - Added a check for postback on Ajax requests to avoid doing full page updates in those cases.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: