ICEfaces
  1. ICEfaces
  2. ICE-3810

selectOneRadio/checkbox not keeping its value when layout=spread

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7.2, 1.8DR#1
          • Fix Version/s: None
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All
          • Workaround Exists:
            Yes
          • Workaround Description:
            Use ICEfaces version 1.7.1

            Description

            In an expandable data table, if the selectOneRadio or selectOneCheckbox component is used in conjunction with the ice:radio or ice:checkbox component and a radio/checkbox is selected when the table is expanded the radio/checkbox loses its value. It stays set in the backing bean but is removed from the screen. This issue does not occur in 1.7.1.
            1. Zip Archive
              Case5356Example2.zip
              19 kB
              Arran Mccullough
            2. Java Source File
              RadioRenderer.java
              10 kB
              Krashan Brahmanjara
            3. Java Source File
              RadioRenderer.java
              9 kB
              Renzo Zanelli
            1. ScreenHunter_01 Apr. 03 17.39.jpg
              89 kB

              Issue Links

              is duplicated by

              Bug - A problem which impairs or prevents the functions of the product. ICE-3646 ice:radio no longer work

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Renzo Zanelli added a comment -

                The attached source file fixes the issues with selectOneRadio using layout="spread".
                This fixes two issues:
                1. Incorrect ice:radio ids.
                2. Radio button loses its selected value.

                Show
                Renzo Zanelli added a comment - The attached source file fixes the issues with selectOneRadio using layout="spread". This fixes two issues: 1. Incorrect ice:radio ids. 2. Radio button loses its selected value.
                Hide
                Permalink
                Renzo Zanelli added a comment -

                This is the same fix provided for Jira issue ICE-3646.

                Show
                Renzo Zanelli added a comment - This is the same fix provided for Jira issue ICE-3646 .
                Hide
                Permalink
                Krashan Brahmanjara added a comment -

                Source attached by Renzo is not especially usable for now. It is very old revision o RadioRenderer.

                In second attachment I add RadioRenderer for current revision 17939 of icefaces.
                My patch resolves similar issues with incorrect interpetation of current selected values and disabled attribute.

                it works very well.

                Show
                Krashan Brahmanjara added a comment - Source attached by Renzo is not especially usable for now. It is very old revision o RadioRenderer. In second attachment I add RadioRenderer for current revision 17939 of icefaces. My patch resolves similar issues with incorrect interpetation of current selected values and disabled attribute. it works very well.
                Hide
                Permalink
                Krashan Brahmanjara added a comment - - edited

                Patch of core/ ../RadioRenderer for current revision 17939 of icefaces.
                Resolves issues with incorrect interpetation of current selected values and disabled attribute.

                Changed sections are signed by tags //byrt start (...) //byrt end

                Show
                Krashan Brahmanjara added a comment - - edited Patch of core/ ../RadioRenderer for current revision 17939 of icefaces. Resolves issues with incorrect interpetation of current selected values and disabled attribute. Changed sections are signed by tags //byrt start (...) //byrt end
                Hide
                Permalink
                Renzo Zanelli added a comment -

                Sorry, I should have specified that my patch fixes 1.7.2 SP1 only.

                Show
                Renzo Zanelli added a comment - Sorry, I should have specified that my patch fixes 1.7.2 SP1 only.
                Hide
                Permalink
                Eric Tiongson added a comment -

                Thanks Renzo for the patch. Confirmed that it works on 1.7.2 SP1. Krashan's patch is not usable for us at the moment.

                Show
                Eric Tiongson added a comment - Thanks Renzo for the patch. Confirmed that it works on 1.7.2 SP1. Krashan's patch is not usable for us at the moment.
                Hide
                Permalink
                Arkadiusz Król added a comment -

                Hello
                We have burning problem with that. Radio after patching works fine but the same problem occurs with selectManyCheckbox
                Have You got some patches for that?

                Show
                Arkadiusz Król added a comment - Hello We have burning problem with that. Radio after patching works fine but the same problem occurs with selectManyCheckbox Have You got some patches for that?
                Hide
                Permalink
                Eric Tiongson added a comment - - edited

                The patch for selectManyCheckbox (1.7.2 SP1) is attached in ICE-3706

                Show
                Eric Tiongson added a comment - - edited The patch for selectManyCheckbox (1.7.2 SP1) is attached in ICE-3706
                Hide
                Permalink
                Krashan Brahmanjara added a comment -

                Almost fixed in ICE-3977.

                For correct results initializers need sequence 0,1,2,3...

                Show
                Krashan Brahmanjara added a comment - Almost fixed in ICE-3977 . For correct results initializers need sequence 0,1,2,3...
                Hide
                Permalink
                yip.ng added a comment -

                Fixes already applied in ICE-3977 and ICE-4142. Problem doesn't occur anymore.

                Show
                yip.ng added a comment - Fixes already applied in ICE-3977 and ICE-4142 . Problem doesn't occur anymore.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: