ICEfaces
  1. ICEfaces
  2. ICE-10384

icecore:push double submit issue with IE 11.

    Details

      Description

      Showcase icecore:push
      icecore:push demo is not running properly in IE 11. The demo appears to submit twice when doing any updates, no matter which push strategy is chosen. When choosing update "Page every second" the page updates twice per second. This is not an issue in other browsers UNLESS they are launched after the page is used in IE 11. Meaning that when icecore:push is deployed on a clean server only using FF, Chrome or IE 10 the issue is not seen. However, launch IE 11, observe the issue, then launch other browsers and the issue will be seen as well.

        Activity

        Hide
        Mircea Toma added a comment -

        I cannot reproduce the issue.

        Show
        Mircea Toma added a comment - I cannot reproduce the issue.
        Hide
        Liana Munroe added a comment -

        This issue can be caused by starting the demo in a browser, then closing the browser, then opening a new browser and running the demo again. The second browser will double update the charts.

        Show
        Liana Munroe added a comment - This issue can be caused by starting the demo in a browser, then closing the browser, then opening a new browser and running the demo again. The second browser will double update the charts.
        Hide
        Mircea Toma added a comment - - edited

        Modified icecore:push demo to generate a notification group per window. This avoids having the notifications generated by one window to be broadcasted to the rest of windows or browsers connected to the demo.

        Show
        Mircea Toma added a comment - - edited Modified icecore:push demo to generate a notification group per window. This avoids having the notifications generated by one window to be broadcasted to the rest of windows or browsers connected to the demo.
        Hide
        Liana Munroe added a comment - - edited

        Tested with ICEfaces 4 tag r43311
        Reopening for these issues:
        1.) The page updates too often when only used in IE 11.
        This can be reproduced by installing showcase on Tomcat 7 and launching IE 11.
        Navigate directly to icecore:push and choose "update page every second". After the first update the page starts to update 2 times per second.

        2.)The issue is still present where one page will update another page in the group.
        This can be reproduced by installing showcase on Tomcat 7 and launching FF.
        Navigate directly to icecore:push and choose "update page every second". The demo runs as expected.
        Next open another tab in FF and navigate directly to icecore:push.
        The demo in the first tab will stop updating when the demo in the 2nd tab starts.
        The demo in the second tab updates approx. twice per second.

        Show
        Liana Munroe added a comment - - edited Tested with ICEfaces 4 tag r43311 Reopening for these issues: 1.) The page updates too often when only used in IE 11. This can be reproduced by installing showcase on Tomcat 7 and launching IE 11. Navigate directly to icecore:push and choose "update page every second". After the first update the page starts to update 2 times per second. 2.)The issue is still present where one page will update another page in the group. This can be reproduced by installing showcase on Tomcat 7 and launching FF. Navigate directly to icecore:push and choose "update page every second". The demo runs as expected. Next open another tab in FF and navigate directly to icecore:push. The demo in the first tab will stop updating when the demo in the 2nd tab starts. The demo in the second tab updates approx. twice per second.
        Hide
        Mircea Toma added a comment -

        Modified pushID expiry mechanism to reset pushID responsiveness counter once a confirmation is received (instead of just decrementing the corresponding un-responsiveness counter). Increased un-responsiveness threshold to 10 since the browsers are dropping 'storage' events during high local storage activity. Modified 'storage' event callbacks to filter on name to avoid redundant callback invocations. Also modified PushWindowScopeBean to shutdown timer on window disposal.

        Show
        Mircea Toma added a comment - Modified pushID expiry mechanism to reset pushID responsiveness counter once a confirmation is received (instead of just decrementing the corresponding un-responsiveness counter). Increased un-responsiveness threshold to 10 since the browsers are dropping 'storage' events during high local storage activity. Modified 'storage' event callbacks to filter on name to avoid redundant callback invocations. Also modified PushWindowScopeBean to shutdown timer on window disposal.
        Hide
        Carmen Cristurean added a comment -

        ICEfaces-4.0.0 tag Jenkins Build#4 (r.43337) testing results:
        The initial issue from description still exists: in IE11, when opening the icecore:push demo in first browser window, and selecting "Update page every second", the push updates appear to be more frequent than once/second. When opening this demo in a second separate IE11 window and selecting "Update page every second", the frequent updates are visible on the second window, while on the first window the push updates stop. The push updates remain stopped on the first window also when closing the second window.

        Testing using the same step in other browsers (Chrome38, FF33, IE10):

        • the more frequent push updates cannot be reproduced in any of these browsers.
        • in all these browsers, after the demo is loaded in 2 separate windows and push updates are selected for "every second", the push updates work fine in both opened windows for about 10-15 seconds, afterwards they stop updating on both windows.
        Show
        Carmen Cristurean added a comment - ICEfaces-4.0.0 tag Jenkins Build#4 (r.43337) testing results: The initial issue from description still exists: in IE11, when opening the icecore:push demo in first browser window, and selecting "Update page every second", the push updates appear to be more frequent than once/second. When opening this demo in a second separate IE11 window and selecting "Update page every second", the frequent updates are visible on the second window, while on the first window the push updates stop. The push updates remain stopped on the first window also when closing the second window. Testing using the same step in other browsers (Chrome38, FF33, IE10): the more frequent push updates cannot be reproduced in any of these browsers. in all these browsers, after the demo is loaded in 2 separate windows and push updates are selected for "every second", the push updates work fine in both opened windows for about 10-15 seconds, afterwards they stop updating on both windows.
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r43375. IE 11 issue still exists
        To reproduce issue:
        Launch showcase in IE 11.
        Navigate to icecore:push > Overview demo.
        For Select push strategy choose "Update page every second".
        Observe that the update appears to happen twice per second instead of once per second.
        Open a new IE 11 browser.
        Observe that the initial demo now updates once per second, then eventually stops.

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r43375. IE 11 issue still exists To reproduce issue: Launch showcase in IE 11. Navigate to icecore:push > Overview demo. For Select push strategy choose "Update page every second". Observe that the update appears to happen twice per second instead of once per second. Open a new IE 11 browser. Observe that the initial demo now updates once per second, then eventually stops.
        Hide
        Ken Fyten added a comment -

        Also note a forum report with a similar issue (added to the Forum link field).

        Show
        Ken Fyten added a comment - Also note a forum report with a similar issue (added to the Forum link field).
        Hide
        Mircea Toma added a comment -

        Created new JIRA case for the push issue: PUSH-343 .

        Show
        Mircea Toma added a comment - Created new JIRA case for the push issue: PUSH-343 .
        Hide
        Mircea Toma added a comment -

        It seems that IE11 is not detected properly by the inter-window communication mechanism. Because of this the 'storage' event callback for the window that triggers the events is invoked twice, by the framework and the browser.

        Show
        Mircea Toma added a comment - It seems that IE11 is not detected properly by the inter-window communication mechanism. Because of this the 'storage' event callback for the window that triggers the events is invoked twice, by the framework and the browser.
        Hide
        Mircea Toma added a comment -

        Detect IE11 also to make sure the 'storage' event callback is not invoked in the window triggering the events.

        Show
        Mircea Toma added a comment - Detect IE11 also to make sure the 'storage' event callback is not invoked in the window triggering the events.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r43633. Tomcat 7, IE 11, 10, FF 33, Chrome 39.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r43633. Tomcat 7, IE 11, 10, FF 33, Chrome 39.
        Hide
        Wolfgang Lumetsberger added a comment -
        Show
        Wolfgang Lumetsberger added a comment - Issue still exists when using IE9: http://www.icesoft.org/JForum/posts/list/22731.page#79652

          People

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

            Dates

            • Created:
              Updated:
              Resolved: