ICEfaces
  1. ICEfaces
  2. ICE-2747

just-ice.jar and navigation handler

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7DR#3, 1.7Beta1
    • Fix Version/s: 1.7.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ANY

      Description

      When navigate from non-icefaces to icefaces page using just-ice.jar, it seems that navigation handler always used the suffix of the originating page and ignore that of the destination page. The problem doesn't present in icefaces 1.6.2 though.

      Use repo/qa/trunk/Regression/ICE-1678 to reproduce the problem

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Mark Collette [ mark.collette ]
        Ken Fyten made changes -
        Security Private [ 10001 ]
        Mark Collette made changes -
        Fix Version/s 1.7.1 [ 10122 ]
        Mark Collette made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mark Collette added a comment -

        Put in a check (ala ICEfaces 1.6 code) for the ".iface" suffix in getActionURL, only delegating if there's no ".iface" suffix.

        Changed the URL guessing algorithm in renderResponse, to update the UIViewRoot.viewId from the guessed file suffix.

        TRUNK
        Subversion 16664
        icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java

        ICEfaces 1.7 branch
        Subversion 16665
        icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java

        Show
        Mark Collette added a comment - Put in a check (ala ICEfaces 1.6 code) for the ".iface" suffix in getActionURL , only delegating if there's no ".iface" suffix. Changed the URL guessing algorithm in renderResponse , to update the UIViewRoot.viewId from the guessed file suffix. TRUNK Subversion 16664 icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java ICEfaces 1.7 branch Subversion 16665 icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java
        Hide
        Mark Collette added a comment -

        I had to change the build.xml, so that it would use just-ice.jar, and then changed the navigation rules to do away with the old style global from-view-id, to use specific from-view-id.

        Subversion 17199
        repo/qa/trunk/Regression/ICE-1678/web/WEB-INF/faces-config.xml
        repo/qa/trunk/Regression/ICE-1678/build.xml

        Show
        Mark Collette added a comment - I had to change the build.xml, so that it would use just-ice.jar, and then changed the navigation rules to do away with the old style global from-view-id, to use specific from-view-id. Subversion 17199 repo/qa/trunk/Regression/ ICE-1678 /web/WEB-INF/faces-config.xml repo/qa/trunk/Regression/ ICE-1678 /build.xml
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #16665 Wed May 21 13:28:09 MDT 2008 mark.collette ICE-2747 : just-ice.jar and navigation handler
        Files Changed
        Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/core/src/com/icesoft/faces/application/D2DViewHandler.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #16664 Wed May 21 13:27:46 MDT 2008 mark.collette ICE-2747 : just-ice.jar and navigation handler
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/application/D2DViewHandler.java
        Mark Collette made changes -
        Assignee Ted Goddard [ ted.goddard ] Mark Collette [ mark.collette ]
        Sam Xiao made changes -
        Assignee Sam Xiao [ sam.xiao ] Ted Goddard [ ted.goddard ]
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Sam Xiao [ sam.xiao ]
        Hide
        Ted Goddard added a comment -

        Might this be a general navigation problem and not specific to just-ice.jar?

        Show
        Ted Goddard added a comment - Might this be a general navigation problem and not specific to just-ice.jar?
        Sam Xiao made changes -
        Field Original Value New Value
        Assignee Ted Goddard [ ted.goddard ]
        Sam Xiao created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Sam Xiao
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: