ICEfaces
  1. ICEfaces
  2. ICE-9974

ice:selectOneRadio causes error when attempting to submit form in IE7

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P01
    • Fix Version/s: EE-3.3.0.GA_P02
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Any
    • Assignee Priority:
      P2

      Description

      Whenever trying to submit a form that contains an ice:selectOneRadio component, a Javascript error appears in the console, and the from cannot be submitted. The error that appears is the following:

      SCRIPT5007: Unable to get property 'indexOf' of undefined or null reference
      File: jsf.js.jsf, Line: 1, Column: 14523

      This was initially reported in ICE-9731, as the Rich Text demo of the component-showcase app was failing. It was found that the problem was with ice:selectOneRadio. To reproduce, simply go to the Rich Text demo in component-showcase and click on the save button of the editor or try to select a different option in the radio control. This can also be reproduced in the Extended Components > Selection demo by trying to enter any value in any component. These issues don't occur when removing the ice:selectOneRadio components from the pages.

        Activity

        Hide
        Mircea Toma added a comment -

        The fix adds 'name' attribute to the submitting fieldset element to avoid JS error in Mojarra bridge. Mojarra is trying to access the 'name' attribute when serialising the form.

        Show
        Mircea Toma added a comment - The fix adds 'name' attribute to the submitting fieldset element to avoid JS error in Mojarra bridge. Mojarra is trying to access the 'name' attribute when serialising the form.
        Hide
        Arturo Zambrano added a comment -

        Re-opening issue.

        ICE-9970 was re-opened, reporting this same issue on IE9/10. Upon closer inspection on this JIRA and on subversion, it seems that there were no commits related to this fix.

        Show
        Arturo Zambrano added a comment - Re-opening issue. ICE-9970 was re-opened, reporting this same issue on IE9/10. Upon closer inspection on this JIRA and on subversion, it seems that there were no commits related to this fix.
        Hide
        Mircea Toma added a comment - - edited

        Commit #40822 which modifies SelectManyCheckboxListRenderer is the one that fixes this issue. The fix is for the maintenance branch only.

        Show
        Mircea Toma added a comment - - edited Commit #40822 which modifies SelectManyCheckboxListRenderer is the one that fixes this issue. The fix is for the maintenance branch only.
        Hide
        Liana Munroe added a comment -

        Reopening this JIRA because of this comment in http://jira.icesoft.org/browse/ICE-9970
        "2) IE9/10: JS error when entering text and click Save icon:
        Message: Unable to get value of the property 'indexOf': object is null or undefined
        Line: 43
        Char: 9
        Code: 0
        URI: http://10.18.39.149:8080/component-showcase/javax.faces.resource/ckeditor_ext.js.jsf?ln=inputrichtext
        This issue is not related to this JIRA. It's the same as ICE-9974. Marking this JIRA (ICE-9970) as fixed."

        Issue is present in Component showcase Icefaces ee-3.3.0 IE only
        RichText Demo
        IE9/10: JS error when entering text and click Save icon:
        Message: Unable to get value of the property 'indexOf': object is null or undefined
        Line: 43
        Char: 9
        Code: 0
        URI: http://10.18.39.149:8080/component-showcase/javax.faces.resource/ckeditor_ext.js.jsf?ln=inputrichtext

        Show
        Liana Munroe added a comment - Reopening this JIRA because of this comment in http://jira.icesoft.org/browse/ICE-9970 "2) IE9/10: JS error when entering text and click Save icon: Message: Unable to get value of the property 'indexOf': object is null or undefined Line: 43 Char: 9 Code: 0 URI: http://10.18.39.149:8080/component-showcase/javax.faces.resource/ckeditor_ext.js.jsf?ln=inputrichtext This issue is not related to this JIRA. It's the same as ICE-9974 . Marking this JIRA (ICE-9970) as fixed." Issue is present in Component showcase Icefaces ee-3.3.0 IE only RichText Demo IE9/10: JS error when entering text and click Save icon: Message: Unable to get value of the property 'indexOf': object is null or undefined Line: 43 Char: 9 Code: 0 URI: http://10.18.39.149:8080/component-showcase/javax.faces.resource/ckeditor_ext.js.jsf?ln=inputrichtext
        Hide
        Mircea Toma added a comment -

        Adding a 'name' attribute to the fieldset element fixed the issue just in HTML5 pages. In non-HTML5 pages the fieldset does not have the name attribute defined in IE since it's not in the spec, hence the error when trying to call indexOf function on an undefined object.

        Show
        Mircea Toma added a comment - Adding a 'name' attribute to the fieldset element fixed the issue just in HTML5 pages. In non-HTML5 pages the fieldset does not have the name attribute defined in IE since it's not in the spec, hence the error when trying to call indexOf function on an undefined object.
        Hide
        Mircea Toma added a comment -

        Modified RadioRenderer and SelectManyCheckboxListRenderer to stop using "fieldset" element for grouping the input elements, thus avoding triggering the JS error when Mojarra bridge tries to serialise the fieldset element.

        Show
        Mircea Toma added a comment - Modified RadioRenderer and SelectManyCheckboxListRenderer to stop using "fieldset" element for grouping the input elements, thus avoding triggering the JS error when Mojarra bridge tries to serialise the fieldset element.
        Hide
        Liana Munroe added a comment -

        Confirmed fixed Icefaces ee-3.3.0 maintenance branch r40876. IE, 7,8, 9, 10, 11. Tomcat 7.

        Show
        Liana Munroe added a comment - Confirmed fixed Icefaces ee-3.3.0 maintenance branch r40876. IE, 7,8, 9, 10, 11. Tomcat 7.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: