ICEfaces
  1. ICEfaces
  2. ICE-1385

Seam: PageFlow navigation not applied correctly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6DR#3, 1.6
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      JBoss 4.0.5

      Description


      In the dvdstore application, entering an invalid customer record causes the view to return to the home page.

      Preliminary instrumentation shows the following sequence. In particular, home.xhtml should not be restored when the user is only interacting with account.xhtml. It may be due to fallthrough of an error case.

      16:35:23,373 INFO [STDOUT] D2DViewHandler.restoreView /home.xhtml
      16:35:23,408 INFO [STDOUT] Pageflow.navigate Page(account)
      16:35:23,408 INFO [STDOUT] Pageflow.redirect Page(account)
      16:35:23,408 INFO [STDOUT] Unfiltered URI: /seam-dvd/newuser/account.seam?conversationId=10&conversationIsLongRunning=true
      16:35:23,417 INFO [STDOUT] D2DViewHandler.restoreView /newuser/account.xhtml
      16:35:23,417 INFO [STDOUT] D2DViewHandler.createView /newuser/account.xhtml
      16:35:23,428 INFO [STDOUT] D2DViewHandler.renderView /newuser/account.xhtml
      16:35:23,638 INFO [STDOUT] D2DViewHandler.restoreView /home.xhtml
      16:35:23,638 INFO [STDOUT] D2DViewHandler.createView /home.xhtml
      16:35:23,641 INFO [STDOUT] D2DViewHandler.restoreView /home.xhtml
      16:35:23,642 INFO [STDOUT] D2DViewHandler.createView /home.xhtml
      16:35:23,654 INFO [STDOUT] D2DViewHandler.renderView /home.xhtml
      16:35:23,656 INFO [STDOUT] D2DViewHandler.renderView /home.xhtml

        Issue Links

          Activity

          Ted Goddard created issue -
          Hide
          Ted Goddard added a comment -

          Requests for images (.gif and .jpg) are passing through to our Servlets with URLs indicating that they are for the containing page. I will now investigate why this occurs with Seam 1.2.1.

          Show
          Ted Goddard added a comment - Requests for images (.gif and .jpg) are passing through to our Servlets with URLs indicating that they are for the containing page. I will now investigate why this occurs with Seam 1.2.1.
          Hide
          Ted Goddard added a comment -

          This is not explicitly a problem with images, the problem is that attempting to fetch resources that do not exist causes home.seam to be rendered.

          Show
          Ted Goddard added a comment - This is not explicitly a problem with images, the problem is that attempting to fetch resources that do not exist causes home.seam to be rendered.
          Hide
          Ted Goddard added a comment -

          What appeared to be a pageflow problem is instead a problem with the error page causing the current DOM to be replaced.

          Commenting out the error page section in web.xml is a workaround for this:

          <!--
          <error-page>
          <error-code>404</error-code>
          <location>/home.seam</location>
          </error-page>
          -->

          concurrentDOMViews may help to resolve the problem of the error page clobbering the current DOM.

          Show
          Ted Goddard added a comment - What appeared to be a pageflow problem is instead a problem with the error page causing the current DOM to be replaced. Commenting out the error page section in web.xml is a workaround for this: <!-- <error-page> <error-code>404</error-code> <location>/home.seam</location> </error-page> --> concurrentDOMViews may help to resolve the problem of the error page clobbering the current DOM.
          Ken Fyten made changes -
          Field Original Value New Value
          Component/s Framework [ 10013 ]
          Hide
          Ted Goddard added a comment -


          This bug is actually a feature request for the error page capability (which may work with concurrentDOMViews; this requires testing).

          Show
          Ted Goddard added a comment - This bug is actually a feature request for the error page capability (which may work with concurrentDOMViews; this requires testing).
          Ted Goddard made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 1.6DR#3 [ 10050 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 1.6 [ 10031 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-1810 [ ICE-1810 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: