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

        Cruz Miraback created issue -
        Cruz Miraback made changes -
        Field Original Value New Value
        Salesforce Case []
        Security Private [ 10001 ]
        Cruz Miraback made changes -
        Environment
        « Hide
        ICEfaces3.0.0.RC2 Bundle 9 / ICEfaces3 Trunk Revision# 27041
        Server: Tomcat7
        Browser: IE9
        ICEfaces3.0.0.RC2 Bundle 9 / ICEfaces3 Trunk Revision# 27041
        Server: Tomcat7
        Browser: IE9
        Salesforce Case []
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P2
        Security Private [ 10001 ]
        Assignee Mark Collette [ mark.collette ]
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P2 P1
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Mark Collette [ mark.collette ] yip.ng [ yip.ng ]
        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.
        yip.ng made changes -
        Attachment screenshot-1.png [ 13944 ]
        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.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27086 Wed Jan 04 10:55:41 MST 2012 yip.ng ICE-7612: Showcase App: Uploading a file with ace:fileEntry throws a JS error in IE9.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/templates/main-template.xhtml
        yip.ng made changes -
        Attachment screenshot-2.png [ 13945 ]
        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
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: