ICEfaces
  1. ICEfaces
  2. ICE-10413

Navigation issue using themeSelect demo then showcase search field

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: EE-4.0.0.GA, 4.1
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 4 trunk r43592. Tomcat 7, all browsers.
    • Assignee Priority:
      P3

      Description

      To reproduce:
      Use showcase ace:themeSelect demo to change the theme.
      In the showcase demo search field type tabSet to display the ace:tabSet demo selections.
      Choose the ace:tabSet > Server Side demo.
      Click on the 2nd tab in the Server Side demo.
      The page reloads with the themeSelect Demo.
      Change the theme again and you are redirected back to the ace:tabSet > Server Side demo.
      Note**
      This issue does not occur when changing the theme, then clicking on the sidebar link to navigate to the ace:tabSet demos. It only occurs when using the Showcase search field. This issue occurs with all of the tabSet demos except ace:tabSet Client Side.
      A few attempts were made at reproducing this issue with demos other than those in ace:tabSet. No further occurrences were found but testing was limited to only a few samples and was not comprehensive.

        Activity

        Liana Munroe created issue -
        Hide
        Carmen Cristurean added a comment - - edited

        Issue reproduced with IF4 trunk rev. 43997 (FF34, Chrome40, IE11).

        Show
        Carmen Cristurean added a comment - - edited Issue reproduced with IF4 trunk rev. 43997 (FF34, Chrome40, IE11).
        Hide
        Ken Fyten added a comment -

        Note that you need to follow the steps precisely to see this issue. Very strange.

        Show
        Ken Fyten added a comment - Note that you need to follow the steps precisely to see this issue. Very strange.
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Fix Version/s EE-4.0.0.GA [ 11171 ]
        Assignee Priority P3 [ 10012 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #44147 Mon Mar 02 17:38:30 MST 2015 mircea.toma ICE-10413 Use ice.sef (execute=@form, render=@all) function to submit the form when switching tabs.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/tabset/tabset.js
        Hide
        Mircea Toma added a comment -

        The tabset JS code submits the form with execute=@all, render=@all attributes when switching tabs. Because of that the form used for navigation is also used.

        The fix modifies tabset.js to use ice.sef (execute=@form, render=@all) function when switching tabs.

        Show
        Mircea Toma added a comment - The tabset JS code submits the form with execute=@all , render=@all attributes when switching tabs. Because of that the form used for navigation is also used. The fix modifies tabset.js to use ice.sef ( execute=@form, render=@all ) function when switching tabs.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Carmen Cristurean added a comment -

        Testing with EE-4.0.0 r41049/ IF4 trunk r44151/ browsers: IE11, FF34, Chrome40: the problem from description can no longer be reproduced.

        Reopening because on ace:tabSet > Proxy demo, the tabs cannot be changed anymore, that may be a regression of the above fix (all browsers).
        No errors are visible in browser console or in the server logs.

        Show
        Carmen Cristurean added a comment - Testing with EE-4.0.0 r41049/ IF4 trunk r44151/ browsers: IE11, FF34, Chrome40: the problem from description can no longer be reproduced. Reopening because on ace:tabSet > Proxy demo, the tabs cannot be changed anymore, that may be a regression of the above fix (all browsers). No errors are visible in browser console or in the server logs.
        Carmen Cristurean made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #44178 Thu Mar 05 12:43:44 MST 2015 mircea.toma ICE-10413, ICE-10498 Avoid navigating if the request is a postback with no 'exp' parameter defined.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/view/navigation/NavigationController.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #44179 Thu Mar 05 12:44:23 MST 2015 mircea.toma ICE-10413 Use full submit once again when switching tabs.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/tabset/tabset.js
        Hide
        Mircea Toma added a comment -

        Modified NavigationController so that navigation is avoided when the request is a post-back with no 'exp' parameter defined.
        Rolled back tabset.js change to use once again full submit when switching tabs.

        Show
        Mircea Toma added a comment - Modified NavigationController so that navigation is avoided when the request is a post-back with no 'exp' parameter defined. Rolled back tabset.js change to use once again full submit when switching tabs.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r44179. Tomcat 7, FF 34, Chrome 40, IE 11, 10, 9, 8.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r44179. Tomcat 7, FF 34, Chrome 40, IE 11, 10, 9, 8.
        Hide
        Jack Van Ooststroom added a comment -

        Verified with Poll as well.

        Show
        Jack Van Ooststroom added a comment - Verified with Poll as well.
        Jack Van Ooststroom made changes -
        Link This issue blocks IPCK-520 [ IPCK-520 ]
        Ken Fyten made changes -
        Fix Version/s 4.1 [ 11375 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Liana Munroe
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: