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

        Liana Munroe created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Fix Version/s 4.0 [ 11382 ]
        Affects Version/s 4.0 [ 11382 ]
        Assignee Priority P1 [ 10010 ]
        Liana Munroe made changes -
        Summary icecore:push double submit issue with IE 10, 11. icecore:push double submit issue with IE 11.
        Liana Munroe made changes -
        Description Showcase icecore:push
        icecore:push demo is not running properly in IE 10 or 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.
        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.
        Hide
        Mircea Toma added a comment -

        I cannot reproduce the issue.

        Show
        Mircea Toma added a comment - I cannot reproduce the issue.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Cannot Reproduce [ 5 ]
        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.
        Mircea Toma made changes -
        Comment [ It seems this was a regression caused by http://jira.icesoft.org/browse/ICE-10339?focusedCommentId=55588&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-55588 fix. That fix was rolled back so this issue is not valid anymore. ]
        Mircea Toma made changes -
        Resolution Cannot Reproduce [ 5 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43308 Thu Oct 30 17:42:31 MDT 2014 mircea.toma ICE-10384 Modified icecore:push demo to generate a notification group per window to avoid the notifications generated by one window to be broadcasted to the rest of windows or browsers connected to the demo.
        Files Changed
        Commit graph ADD /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushWindowScopeBean.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushBean.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/core/push.xhtml
        Commit graph DEL /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushViewScopeBean.java
        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.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43309 Fri Oct 31 10:04:17 MDT 2014 ken.fyten ICE-10384 Modified icecore:push demo to generate a notification group per window to avoid the notifications generated by one window to be broadcasted to the rest of windows or browsers connected to the demo.
        Files Changed
        Commit graph DEL /icefaces4/tags/icefaces-4.0.0/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushViewScopeBean.java
        Commit graph MODIFY /icefaces4/tags/icefaces-4.0.0/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushBean.java
        Commit graph ADD /icefaces4/tags/icefaces-4.0.0/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushWindowScopeBean.java
        Commit graph MODIFY /icefaces4/tags/icefaces-4.0.0/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/core/push.xhtml
        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.
        Liana Munroe made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43328 Tue Nov 04 06:48:06 MST 2014 mircea.toma ICE-10384 Modified pushID expiry mechanism to reset pushID responsiveness counter once a confirmation is received (instead of just decrementing the corresponding responsiveness counter). Increased un-responsiveness threshold to 10 since the browsers are dropping 'storage' events during high storage activity. Modified 'storage' event callbacks to filter on name to avoid redundant callback invocations. Also modified PushWindowScopeBean to shutdown timer on window disposal.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushWindowScopeBean.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/pushid.expiry.js
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/inter.window.notification.js
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
        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.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43337 Tue Nov 04 09:03:19 MST 2014 ken.fyten ICE-10384 Modified PushWindowScopeBean to shutdown timer on window disposal.
        Files Changed
        Commit graph MODIFY /icefaces4/tags/icefaces-4.0.0/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/core/push/PushWindowScopeBean.java
        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.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Priority P1 [ 10010 ] P2 [ 10011 ]
        Ken Fyten made changes -
        Fix Version/s EE-4.0.0.GA [ 11171 ]
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Component/s ICECORE-Components [ 10370 ]
        Component/s Sample Apps [ 10010 ]
        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).
        Ken Fyten made changes -
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43594 Tue Nov 25 07:12:21 MST 2014 mircea.toma ICE-10384 Spread out broadcasting of notifications to windows in order to avoid having 'storage' events dropped by the browser.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/pushid.expiry.js
        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 .
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43631 Tue Dec 02 04:52:00 MST 2014 mircea.toma ICE-10384 Detect IE11 also to make sure the 'storage' event callback is not invoked in the window triggering the events.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/inter.window.notification.js
        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.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mircea Toma made changes -
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        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
        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: