ICEfaces
  1. ICEfaces
  2. ICE-7760

JS Error from Bridge when cleaning up iframes

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0
    • Fix Version/s: EE-3.0.0.BETA, 3.0.1
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      n/a

      Description

      Bridge tries to clean up iframes onBeforeUpdate (ICE-7129), but some iframe elements are undefined:

       namespace.onBeforeUpdate(function(updates) {
               each(updates.getElementsByTagName('update'), function(update) {
                   var id = update.getAttribute('id');
                   var updatedElement = lookupElementById(id);
                   if (updatedElement) {
                       each(updatedElement.getElementsByTagName('iframe'), function(iframe) {
       [b] iframe.parentNode.removeChild(iframe);[/b]
                       });
                   }
               });
           });

      Uncaught TypeError: Cannot read property 'parentNode' of undefined

      bridge.js.iface:2822

        Activity

        Hide
        Philip Breau added a comment -

        *27804 2/13/12 11:36 AM 1 philip.breau ICE-7760 fix undefined reference

        Show
        Philip Breau added a comment - *27804 2/13/12 11:36 AM 1 philip.breau ICE-7760 fix undefined reference
        Hide
        Evgheni Sadovoi added a comment -

        Same experience in our showcase. Our QA team found it and I removed demo temporary until this will be resolved.

        Details below:
        ice:panelDivider > Dynamic Content

        1) FF9: navigating to this page must display content for both top pane and bottom pane, according to default drop-downs selections; no content is rendered in the top pane.
        IE8: image 1.png attached. (does not load google and yahoo inside iframe)

        2) cannot navigate away from this page, clicking on another link on the left side of the page, makes the page to load forever.
        In FF9, the error console shows:

        Error: iframe is undefined
        Source File: http://localhost:8080/showcase/javax.faces.resource/bridge.js.jsf?v=1495488463
        Line: 2815

        In IE8: JS error:

        User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; MDDS; .NET4.0C)
        Timestamp: Thu, 26 Jan 2012 20:17:15 UTC
        Message: 'parentNode' is null or not an object
        Line: 2815
        Char: 21
        Code: 0
        URI: http://localhost:8080/showcase/javax.faces.resource/bridge.js.jsf?v=1495488463

        Show
        Evgheni Sadovoi added a comment - Same experience in our showcase. Our QA team found it and I removed demo temporary until this will be resolved. Details below: ice:panelDivider > Dynamic Content 1) FF9: navigating to this page must display content for both top pane and bottom pane, according to default drop-downs selections; no content is rendered in the top pane. IE8: image 1.png attached. (does not load google and yahoo inside iframe) 2) cannot navigate away from this page, clicking on another link on the left side of the page, makes the page to load forever. In FF9, the error console shows: Error: iframe is undefined Source File: http://localhost:8080/showcase/javax.faces.resource/bridge.js.jsf?v=1495488463 Line: 2815 In IE8: JS error: User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; MDDS; .NET4.0C) Timestamp: Thu, 26 Jan 2012 20:17:15 UTC Message: 'parentNode' is null or not an object Line: 2815 Char: 21 Code: 0 URI: http://localhost:8080/showcase/javax.faces.resource/bridge.js.jsf?v=1495488463
        Hide
        Ken Fyten added a comment -

        Added missing commit to the icefaces-3.0.x-maintenance branch. SVN rvn# 27924.

        Show
        Ken Fyten added a comment - Added missing commit to the icefaces-3.0.x-maintenance branch. SVN rvn# 27924.
        Hide
        Evgheni Sadovoi added a comment -

        Added back ice:panelDivider > Dynamic Content demo after testing it with code from rvn# 27924.

        Show
        Evgheni Sadovoi added a comment - Added back ice:panelDivider > Dynamic Content demo after testing it with code from rvn# 27924.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: