ICEfaces
  1. ICEfaces
  2. ICE-7612

Showcase App: Uploading a file with ace:fileEntry throws a JS error in IE9

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC2
    • Fix Version/s: 3.0
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces3.0.0.RC2 Bundle 9 / ICEfaces3 Trunk Revision# 27041
      Server: Tomcat7
      Browser: IE9
    • Assignee Priority:
      P1

      Description

      This occurs on the showcase sample application:

      Trying to upload a file on any of the ace:fileEntry pages (Overview/Listener/Validation) causes a JS error to be thrown:

      Message: 'XMLSerializer' is undefined
      Line: 1
      Char: 45940
      Code: 0
      URI: http://localhost:8080/showcase/javax.faces.resource/util/ace-components.js.jsf?ln=icefaces.ace

      The file does successfully upload, however, there is no confirmation on the page showing that it did.

        Activity

        Hide
        Mark Collette added a comment -

        Make sure you first load the showcase main page, and then navigate to the fileEntry page, to test things, because there's a known issue with IE9 of uploading from the first page GET after starting a new session.

        Also, with all the javascript file combining, we had previously not combined fileentry.js with anything else, so that there would be no dependencies on ACE, for when apps want to just use compat but need ACE just for fileEntry.

        Show
        Mark Collette added a comment - Make sure you first load the showcase main page, and then navigate to the fileEntry page, to test things, because there's a known issue with IE9 of uploading from the first page GET after starting a new session. Also, with all the javascript file combining, we had previously not combined fileentry.js with anything else, so that there would be no dependencies on ACE, for when apps want to just use compat but need ACE just for fileEntry.
        Hide
        yip.ng added a comment -

        See screenshot-1.png for the error.

        Show
        yip.ng added a comment - See screenshot-1.png for the error.
        Hide
        yip.ng added a comment -

        The error is caused by the "IE=EmulateIE7" HTML header. See screenshot-2.png. XMLSerializer is an IE9 invention. Emulated IE7 mode most probably doesn't have that function object.

        Show
        yip.ng added a comment - The error is caused by the "IE=EmulateIE7" HTML header. See screenshot-2.png. XMLSerializer is an IE9 invention. Emulated IE7 mode most probably doesn't have that function object.
        Hide
        yip.ng added a comment -

        Ken: Remove this line and retest on IE8.

        Show
        yip.ng added a comment - Ken: Remove this line and retest on IE8.
        Hide
        yip.ng added a comment -

        After removing line, works in IE7, IE8 and IE9.

        Show
        yip.ng added a comment - After removing line, works in IE7, IE8 and IE9.
        Hide
        yip.ng added a comment - - edited

        Ken: Mark this resolved (once you commit the fix).

        Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\samples\showcase\showcase\src\main\webapp\resources\templates\main-template.xhtml
        Completed: At revision: 27086

        Show
        yip.ng added a comment - - edited Ken: Mark this resolved (once you commit the fix). Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\samples\showcase\showcase\src\main\webapp\resources\templates\main-template.xhtml Completed: At revision: 27086
        Hide
        Carmen Cristurean added a comment -

        Fix has been verified with code revision 27114 on IE9 and IE8.

        Show
        Carmen Cristurean added a comment - Fix has been verified with code revision 27114 on IE9 and IE8.

          People

          • Assignee:
            yip.ng
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: