Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      IE6 only

      Description

      The error can be reproduce by using component-showcase, after interacting with the application just reload the page. The error message shown by IE6 is attached.
      1. screenshot-1.jpg
        42 kB
      2. screenshot-2.jpg
        42 kB
      3. screenshot-3.jpg
        42 kB
      4. screenshot-4.jpg
        42 kB

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23854 Mon Jan 31 05:57:02 MST 2011 mircea.toma ICE-6511 Removed blockUI configuration in compat JS code since the block UI code resides solely in core. Fixed pseudo URLs used by overlay iframe to render correctly the markup.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/application/ExtrasSetup.java
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/javascript/status.js
        Mircea Toma created issue -
        Mircea Toma made changes -
        Field Original Value New Value
        Salesforce Case []
        Fix Version/s 2.0.1 [ 10255 ]
        Assignee Mircea Toma [ mircea.toma ]
        Mircea Toma made changes -
        Attachment screenshot-1.jpg [ 12833 ]
        Hide
        Mircea Toma added a comment - - edited

        Removed blockUI configuration in compat JS code since the block UI code resides solely in core.
        This change uncovered another problem with the overlay used to render the busy indicator. The JS code found in the pseudo URLs of the iframe used as overlay didn't render correctly. The fix changed the JS code to pass in the markup into document.write method as it should have been in the first place.

        Show
        Mircea Toma added a comment - - edited Removed blockUI configuration in compat JS code since the block UI code resides solely in core. This change uncovered another problem with the overlay used to render the busy indicator. The JS code found in the pseudo URLs of the iframe used as overlay didn't render correctly. The fix changed the JS code to pass in the markup into document.write method as it should have been in the first place.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23855 Mon Jan 31 14:55:58 MST 2011 mircea.toma ICE-6511 Use trick to dynamically add elements to the document to avoid 'Operation aborted' in IE.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
        Mircea Toma made changes -
        Attachment screenshot-2.jpg [ 12837 ]
        Mircea Toma made changes -
        Attachment screenshot-3.jpg [ 12838 ]
        Mircea Toma made changes -
        Attachment screenshot-4.jpg [ 12839 ]
        Hide
        Mircea Toma added a comment -

        Once the blockUI configuration parameter was removed another three errors popup while reloading the page. Screenshots 2,3, and 4 show the type of errors.
        All these errors seem to indicate that compat.js and icefaces-compat.js are not loaded properly. Looking at the markup I could see that GoogleMaps JS reference would be rendered just before compat.js and icefaces-compat.js. After modifying GMap component to not render its JS reference all the mentioned errors went away. It seems that the popup rendered by GoogleMaps mentioning that the key is not valid for the site interferes with the loading of the subsequent referenced JS code.
        So, really this errors would not occur if the GoogleMaps key would be configured correctly during deployment. Also the errors would show up only if IE's Reload button is pressed, hitting Enter or Go on the URL will always work without any issues.

        Show
        Mircea Toma added a comment - Once the blockUI configuration parameter was removed another three errors popup while reloading the page. Screenshots 2,3, and 4 show the type of errors. All these errors seem to indicate that compat.js and icefaces-compat.js are not loaded properly. Looking at the markup I could see that GoogleMaps JS reference would be rendered just before compat.js and icefaces-compat.js. After modifying GMap component to not render its JS reference all the mentioned errors went away. It seems that the popup rendered by GoogleMaps mentioning that the key is not valid for the site interferes with the loading of the subsequent referenced JS code. So, really this errors would not occur if the GoogleMaps key would be configured correctly during deployment. Also the errors would show up only if IE's Reload button is pressed, hitting Enter or Go on the URL will always work without any issues.
        Hide
        Mircea Toma added a comment - - edited

        There still an issue remaining during reload. When selecting the 'Popup Panel' panel in component-showcase and then page is reloaded an 'Operation aborted' popup is shown. This type of error is described here: http://support.microsoft.com/kb/927917 .
        Looking closer at what popup related code is evaluated during page load I manged to find that Ice.iFrameFix.start() function is indeed invoked and it has document.appendChild() method invoked in its body.
        The fix applied was to change slightly the code to use element.inserBefore function. This trick was already used in other places in extras.js.

        Show
        Mircea Toma added a comment - - edited There still an issue remaining during reload. When selecting the 'Popup Panel' panel in component-showcase and then page is reloaded an 'Operation aborted' popup is shown. This type of error is described here: http://support.microsoft.com/kb/927917 . Looking closer at what popup related code is evaluated during page load I manged to find that Ice.iFrameFix.start() function is indeed invoked and it has document.appendChild() method invoked in its body. The fix applied was to change slightly the code to use element.inserBefore function. This trick was already used in other places in extras.js.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: