ICEfaces-EE
  1. ICEfaces-EE
  2. IPCK-394

ICEfaces1.8 composite-comps-showcase - JS error after adjusting Polling demo poll interval

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P03
          • Fix Version/s: EE-1.8.2.GA_P04
          • Component/s: Facelet Components
          • Labels:
            None
          • Environment:
            ICEfaces trunk rev.28617, ICEpack rev.31054
            Server: Tomcat6
            Browsers: Chrome17
          • Affects:
            Sample App./Tutorial

            Description

            A JS error was seen in console (Chrome) while clicking on the left-side links to navigate to various components, I cannot find out how to reproduce it consistently.

            Error message:
            Uncaught TypeError: Cannot read property 'parentNode' of null

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Mircea Toma added a comment -

              Tested with Chrome 18. I could not reproduce the error.

              Show
              Mircea Toma added a comment - Tested with Chrome 18. I could not reproduce the error.
              Hide
              Permalink
              Ken Fyten added a comment -

              Okay, to reproduce you view the Polling demo, change the poll interval, then navigate to another demo. The JS error console will start throwing errors for each poll interval. Seems like maybe it's not shutting down the polling after you adjust it the first time. Happens on all browsers.

              Show
              Ken Fyten added a comment - Okay, to reproduce you view the Polling demo, change the poll interval, then navigate to another demo. The JS error console will start throwing errors for each poll interval. Seems like maybe it's not shutting down the polling after you adjust it the first time. Happens on all browsers.
              Hide
              Permalink
              Philip Breau added a comment -

              Revision: 31168
              Author: philip.breau
              Date: 4:15:12 PM, Friday, April 13, 2012
              Message:
              IPCK-394 JS error after adjusting Polling demo poll interval

              • re-implemented component (backport from 2.0). the simple setInterval() was not accounting for multiple instances, navigation, colliding interval calls, etc., which the re-implemented version does

                Modified : /icepack/trunk/icepack/composite-components/web/polling/polling.xhtml
                Modified : /icepack/trunk/icepack/samples/composite-comps-showcase/web/WEB-INF/includes/examples/composition-components/polling.jspx

              Show
              Philip Breau added a comment - Revision: 31168 Author: philip.breau Date: 4:15:12 PM, Friday, April 13, 2012 Message: IPCK-394 JS error after adjusting Polling demo poll interval re-implemented component (backport from 2.0). the simple setInterval() was not accounting for multiple instances, navigation, colliding interval calls, etc., which the re-implemented version does Modified : /icepack/trunk/icepack/composite-components/web/polling/polling.xhtml Modified : /icepack/trunk/icepack/samples/composite-comps-showcase/web/WEB-INF/includes/examples/composition-components/polling.jspx
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified fix using EE-ICEfaces 1.8.2.GA Bundle2 in Chrome18 and IE8, on Tomcat7.0.27.

              Show
              Carmen Cristurean added a comment - Verified fix using EE-ICEfaces 1.8.2.GA Bundle2 in Chrome18 and IE8, on Tomcat7.0.27.

                People

                • Assignee:
                  Philip Breau
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: