ICEfaces
  1. ICEfaces
  2. ICE-7918

blockUIOnSubmit causes a JavaScript error when calling window.close() on a modal window in IE8

    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: Framework
    • Labels:
      None
    • Environment:
      IE8 version 8.0.76xx, blockUIOnSubmit=true, ICEfaces EE 1.8.2 P03, trunk

      Description

      An "Access is Denied" JavaScript error is shown when closing a modal window dialog in IE8. This dialog is opened by calling the following JavaScript: "window.showModalDialog(....);"

      The call that closes the dialog is the following: "window.close();"

      This is only an issue when the blockUIOnSubmit=true and only in the more recent versions of IE8.

      8.0.7600.16385 - Shows the error

      8.0.6001.18702 - Doesn't show the error

      My guess is that the UI blocker code is interfering with the modal window dialog or vice versa.

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case [5007000000LEufh]
        Hide
        Arran Mccullough added a comment -

        Attaching test case and screen shot that shows JS error.

        Steps to reproduce:

        • Load welcomeICEfaces.iface
        • Click on the open button
        • Click on the close button for the modal window.
        Show
        Arran Mccullough added a comment - Attaching test case and screen shot that shows JS error. Steps to reproduce: Load welcomeICEfaces.iface Click on the open button Click on the close button for the modal window.
        Arran Mccullough made changes -
        Attachment IE8-JavaScript-Error.jpg [ 14194 ]
        Attachment Case11058Example.zip [ 14195 ]
        Attachment Case11058Example.war [ 14196 ]
        Hide
        Ken Fyten added a comment -

        Note that this issue only occurs on the more recent IE8 8.0.7.x, which appears to only be available on Windows 7.

        Show
        Ken Fyten added a comment - Note that this issue only occurs on the more recent IE8 8.0.7.x, which appears to only be available on Windows 7.
        Ken Fyten made changes -
        Fix Version/s EE-1.8.2.GA_P04 [ 10280 ]
        Assignee Priority P2
        Assignee Mircea Toma [ mircea.toma ]
        Hide
        Arran Mccullough added a comment -

        Link to the MS Virtual PC IE App Compatibility Images: http://www.microsoft.com/download/en/details.aspx?id=11575

        Show
        Arran Mccullough added a comment - Link to the MS Virtual PC IE App Compatibility Images: http://www.microsoft.com/download/en/details.aspx?id=11575
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28688 Thu Apr 12 11:03:34 MDT 2012 mircea.toma ICE-7918 Capture exception thrown when closing the popup window.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/status.js
        Hide
        Mircea Toma added a comment -

        It seems that IE sees the dynamically created iframe (our status indicator overlay) as being part of a different domain . Modifying the iframe's URL would create regressions for issues we already fixed by generating the frame's content using a pseudo-URL.

        The solution applied was to capture and ignore the exception thrown by the popup window when closed. Since the error has no functional impact (the popup window is closed anyway) this can be considered a reasonable fix.

        Show
        Mircea Toma added a comment - It seems that IE sees the dynamically created iframe (our status indicator overlay) as being part of a different domain . Modifying the iframe's URL would create regressions for issues we already fixed by generating the frame's content using a pseudo-URL. The solution applied was to capture and ignore the exception thrown by the popup window when closed. Since the error has no functional impact (the popup window is closed anyway) this can be considered a reasonable fix.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

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

            Dates

            • Created:
              Updated:
              Resolved: