ICEfaces
  1. ICEfaces
  2. ICE-4164

Double clicking tab that has long processing causes components to be reset

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2 SP1, 1.8RC1
    • Fix Version/s: 1.8RC2, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      In a panelTabSet a tab's contents takes some time to load/process. If the user double clicks this tab some input components are reset to their default values/null. The components that seem to be affected are selectOneMenu, selectBooleanCheckbox, and selectManyMenu. The inputText is not affected by this behavior.

        Issue Links

          Activity

          Arran Mccullough created issue -
          Arran Mccullough made changes -
          Field Original Value New Value
          Salesforce Case [50070000007zpS7]
          Hide
          Arran Mccullough added a comment -

          Main files for test-case.

          Show
          Arran Mccullough added a comment - Main files for test-case.
          Arran Mccullough made changes -
          Attachment main.jspx [ 11552 ]
          Attachment TestBean.java [ 11553 ]
          Ken Fyten made changes -
          Assignee Deryk Sinotte [ deryk.sinotte ]
          Hide
          Deryk Sinotte added a comment -

          Mircea, please take a look and provide an analysis of what is happening.

          Show
          Deryk Sinotte added a comment - Mircea, please take a look and provide an analysis of what is happening.
          Deryk Sinotte made changes -
          Assignee Deryk Sinotte [ deryk.sinotte ] Mircea Toma [ mircea.toma ]
          Hide
          Arran Mccullough added a comment -

          Built test case, developed in NetBeans IDE, built for Tomcat 6. may need to manually navigate to http://localhost:8080/Case8045Example/main.iface

          Show
          Arran Mccullough added a comment - Built test case, developed in NetBeans IDE, built for Tomcat 6. may need to manually navigate to http://localhost:8080/Case8045Example/main.iface
          Arran Mccullough made changes -
          Attachment Case8045Example.war [ 11557 ]
          Hide
          Mircea Toma added a comment -

          Because of the delay, the update corresponding to the first request (when changing tabs) is not applied before the second request is sent out. Thus the second request is missing the parameters that correspond to selectOneMenu or selectManyMenu options and the components/renderers interpret this request as a deselection of their options.

          Show
          Mircea Toma added a comment - Because of the delay, the update corresponding to the first request (when changing tabs) is not applied before the second request is sent out. Thus the second request is missing the parameters that correspond to selectOneMenu or selectManyMenu options and the components/renderers interpret this request as a deselection of their options.
          Hide
          Mircea Toma added a comment -

          In order to fix this issue properly we'll need to implement the client-side queue to serially send and apply the received updates.

          Show
          Mircea Toma added a comment - In order to fix this issue properly we'll need to implement the client-side queue to serially send and apply the received updates.
          Ken Fyten made changes -
          Fix Version/s 1.8 [ 10161 ]
          Assignee Priority P2
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #18546 Wed Mar 11 17:18:08 MDT 2009 mircea.toma ICE-4197, ICE-4164 Implement client-side request queue. Delay query serialization until request is sent.
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.async.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/parameters.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/submit.js
          Mircea Toma made changes -
          Link This issue depends on ICE-4197 [ ICE-4197 ]
          Hide
          Mircea Toma added a comment -

          Solved by the fixes for ICE-4197.

          Besides the need for the queue, in order for this issue to be fixed the bridge was changed so that it builds the query string for the submitted form just before sending it. Before the fix the query strings would be calculated when iceSubmit/iceSubmitPartial would be invoked. By delaying building the query string the form elements have their state updated properly by the previously applied updates.

          Show
          Mircea Toma added a comment - Solved by the fixes for ICE-4197 . Besides the need for the queue, in order for this issue to be fixed the bridge was changed so that it builds the query string for the submitted form just before sending it. Before the fix the query strings would be calculated when iceSubmit/iceSubmitPartial would be invoked. By delaying building the query string the form elements have their state updated properly by the previously applied updates.
          Mircea Toma made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #18554 Thu Mar 12 17:43:26 MDT 2009 mircea.toma ICE-4197, ICE-4164 Rollback changes. Feature posteponed for 2.0.
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.async.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/connection.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/parameters.js
          Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/submit.js
          Ken Fyten made changes -
          Link This issue depends on ICE-4203 [ ICE-4203 ]
          Ken Fyten made changes -
          Fix Version/s 1.8RC2 [ 10163 ]
          Fix Version/s 1.8 [ 10161 ]
          Ken Fyten made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-4197 [ ICE-4197 ]
          Hide
          Ken Fyten added a comment -

          Due to additional complexities around the client-side request queue, it has been backed out for this release.

          An alternative solution to this issue is described in ICE-4203.

          Show
          Ken Fyten added a comment - Due to additional complexities around the client-side request queue, it has been backed out for this release. An alternative solution to this issue is described in ICE-4203 .
          Hide
          Joanne Bai added a comment -

          QA tested it with success on 1.8.0-RC2 build 3 + tomcat6 + FF2 and IE6

          Show
          Joanne Bai added a comment - QA tested it with success on 1.8.0-RC2 build 3 + tomcat6 + FF2 and IE6
          Ken Fyten made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Assignee Priority P2
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 1.8 [ 10161 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Mircea Toma [ mircea.toma ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Arran Mccullough
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: