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

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27804 Mon Feb 13 08:36:00 MST 2012 philip.breau ICE-7760
              fix undefined reference
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/javascript/fixjsf.js
              Philip Breau created issue -
              Philip Breau made changes -
              Field Original Value New Value
              Assignee Philip.breau [ philip.breau ]
              Hide
              Permalink
              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
              Philip Breau made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Fix Version/s 3.0.1 [ 10282 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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
              Evgheni Sadovoi made changes -
              Attachment image 1.jpg [ 14059 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27924 Fri Feb 17 11:20:39 MST 2012 ken.fyten ICE-7760 - Fix undefined reference
              Files Changed
              Commit graph MODIFY /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/core/src/main/javascript/fixjsf.js
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27927 Fri Feb 17 13:42:45 MST 2012 esadovoi added ice:panelDivider > Dynamic Content example after testing the fix from ICE-7760
              Files Changed
              Commit graph MODIFY /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/divider/DividerContent.java
              Commit graph MODIFY /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/divider/DividerBean.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27928 Fri Feb 17 13:43:41 MST 2012 esadovoi added ice:panelDivider > Dynamic Content example after testing the fix from ICE-7760
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/divider/DividerBean.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/divider/DividerContent.java
              Hide
              Permalink
              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.
              Ken Fyten made changes -
              Fix Version/s EE-3.0.0.BETA [ 10324 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: