ICEfaces
  1. ICEfaces
  2. ICE-3977

Preselection of radio or checkbox with layout=spread not working

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2, 1.7.2 SP1, 1.8DR#1, 1.8DR#2
    • Fix Version/s: 1.7.2-SP2, 1.8RC1, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When pre-selecting a value for ice:radio or ice:checkbox is not working with 1.7.2 SP1. It was working with ICEfaces 1.7.1

        Activity

        Hide
        Mark Collette added a comment -

        The checkbox rendering code was incorrectly getting the component selection value from the checkbox, when it should have gotten it from the UISelectMany referenced by the for attribute.

        Subversion 18262
        icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java

        Subversion 18263
        icefaces17\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java

        Show
        Mark Collette added a comment - The checkbox rendering code was incorrectly getting the component selection value from the checkbox, when it should have gotten it from the UISelectMany referenced by the for attribute. Subversion 18262 icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java Subversion 18263 icefaces17\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java
        Hide
        Sam Xiao added a comment -

        checkbox now works but problem still exists from radio button

        Show
        Sam Xiao added a comment - checkbox now works but problem still exists from radio button
        Hide
        Mark Collette added a comment -

        Included the radio components in the fix, and noticed an oversight in the checkbox fix as well.

        TRUNK
        Subversion 18275
        icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\RadioRenderer.java
        icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java

        ICEfaces 1.7 Branch
        Subversion 18276
        icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\RadioRenderer.java
        icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java

        Show
        Mark Collette added a comment - Included the radio components in the fix, and noticed an oversight in the checkbox fix as well. TRUNK Subversion 18275 icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\RadioRenderer.java icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java ICEfaces 1.7 Branch Subversion 18276 icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\RadioRenderer.java icefaces\core\src\com\icesoft\faces\renderkit\dom_html_basic\SelectManyCheckboxListRenderer.java
        Hide
        Joanne Bai added a comment -

        verified with success on 1.7.2-SP2 build 3 (tested on tomcat6 + FF2 and IE6)

        Show
        Joanne Bai added a comment - verified with success on 1.7.2-SP2 build 3 (tested on tomcat6 + FF2 and IE6)

          People

          • Assignee:
            Unassigned
            Reporter:
            Arran Mccullough
          • Votes:
            4 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: