ICEfaces
  1. ICEfaces
  2. ICE-11046

IE 8 - js error triggered when using ace:clientValidateRequired with ace:fileEntry

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-4.1.0.RC1
    • Fix Version/s: EE-4.1.0.GA, 4.2.BETA, 4.2
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 4 trunk r48822, Tomcat 8, IE 8
    • Assignee Priority:
      P1
    • Support Case References:

      Description

      If you upload a file after triggering required validation a popup js error is seen. This is not an issue with 4.1.1 libs.
      Message: Invalid argument.
      Line: 326
      Char: 13
      Code: 0
      URI: http://10.18.39.134:8080/fileEntry/javax.faces.resource/fileentry/fileEntry.js.jsf?ln=icefaces.ace&v=4_2_0_160518.

      To reproduce:
      1.) With IE 8 use the /fileEntryClientValidateRequired.jsf page found at:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/fileEntry
      2.) Press the Upload button (do not browse for file first). A required validation message will appear.
      3.) Press the Browse button and choose a file.
      4.) Press the Upload button. The js error popup appears.

        Activity

        Liana Munroe created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Fix Version/s EE-4.1.0.GA [ 12171 ]
        Affects Version/s EE-4.1.0.RC1 [ 13072 ]
        Affects Version/s 4.2 [ 12870 ]
        Affects Version/s EE-4.1.1.BETA [ 13079 ]
        Assignee Priority P1 [ 10010 ]
        Hide
        Mircea Toma added a comment -

        Modified ice.ace.fileentry.addHiddenInput to edit the hidden input fields if element is already present. Lookup the input elements before removing them from the form just in case they were changed by an update (and thus orphaned).

        Show
        Mircea Toma added a comment - Modified ice.ace.fileentry.addHiddenInput to edit the hidden input fields if element is already present. Lookup the input elements before removing them from the form just in case they were changed by an update (and thus orphaned).
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48838 Mon May 23 15:38:49 MDT 2016 mircea.toma ICE-11046 Modified ice.ace.fileentry.addHiddenInput to edit the hidden input fields if element is already present. Lookup the input elements before removeing them from the form just in case they were changed by an update (anf thus orphaned).
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r48844, Tomcat 8/7 using IE 8 on 2 different test machines.
        When uploading a file after the clientValidateRequired is rendered the following js popup error is seen. File is uploaded as expected. This is not an issue when uploading a file before forcing the clientValidateRequired to be shown.
        To reproduce, follow steps as written in description.

        Message: 'XMLSerializer' is undefined
        Line: 195
        Char: 17
        Code: 0
        URI: http://10.18.39.134:8080/fileEntry/javax.faces.resource/fileentry/fileEntry.js.jsf?ln=icefaces.ace&v=4_2_0_160524

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r48844, Tomcat 8/7 using IE 8 on 2 different test machines. When uploading a file after the clientValidateRequired is rendered the following js popup error is seen. File is uploaded as expected. This is not an issue when uploading a file before forcing the clientValidateRequired to be shown. To reproduce, follow steps as written in description. Message: 'XMLSerializer' is undefined Line: 195 Char: 17 Code: 0 URI: http://10.18.39.134:8080/fileEntry/javax.faces.resource/fileentry/fileEntry.js.jsf?ln=icefaces.ace&v=4_2_0_160524
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Mircea Toma added a comment -

        Test for the existence of XMLSerializer variable before using it. IE8 does not provide any XML serialization functionality.

        Show
        Mircea Toma added a comment - Test for the existence of XMLSerializer variable before using it. IE8 does not provide any XML serialization functionality.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48851 Tue May 24 16:36:35 MDT 2016 mircea.toma ICE-11046 Test for the existence of XMLSerializer variable before using it.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Hide
        Carmen Cristurean added a comment - - edited

        ICEfaces EE 4 trunk r.48851/IE8: verified with QA fileEntry test app, using the steps from JIRA description.

        Show
        Carmen Cristurean added a comment - - edited ICEfaces EE 4 trunk r.48851/IE8: verified with QA fileEntry test app, using the steps from JIRA description.
        Carmen Cristurean made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Carmen Cristurean made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 4.2.BETA [ 13091 ]
        Fix Version/s 4.2 [ 12870 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: