ICEfaces
  1. ICEfaces
  2. ICE-4317

selectManyCheckbox not clearable

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8RC2
          • Fix Version/s: 1.8
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            All

            Description

            If you have a selectManyCheckbox, with some checkboxes checked and other not, then you click on a commandButton whose actionListener clears all of the bean values for the selectManyCheckbox, you do not see that reflected in the browser.

              Issue Links

              blocks

              Bug - A problem which impairs or prevents the functions of the product. ICE-4392 selectManyCheckbox still not clearable?

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #18723 Wed Apr 01 18:08:59 MDT 2009 mark.collette ICE-4317 : selectManyCheckbox not clearable
                Files Changed
                Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/BridgeFacesContext.java
                Mark Collette created issue -
                Hide
                Permalink
                Mark Collette added a comment -

                Subversion 18723
                icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java

                Show
                Mark Collette added a comment - Subversion 18723 icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #18727 Wed Apr 01 20:25:58 MDT 2009 mark.collette ICE-4317 : selectManyCheckbox not clearable
                Files Changed
                Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/BridgeFacesContext.java
                Ken Fyten made changes -
                Field Original Value New Value
                Salesforce Case []
                Fix Version/s 1.8 [ 10161 ]
                Ken Fyten made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                Mark Collette added a comment -

                The issue was that selectManyCheckbox submits its form elements by name, not id, so our applyBrowserDOMChanges() wasn't accounting for it. The fix for ICE-3945, which was a follow-on to ICE-3190, for adding support for checkboxes, added better support for individual checkboxes, at the expense of multiple checkboxes. Now both types should work properly.

                Show
                Mark Collette added a comment - The issue was that selectManyCheckbox submits its form elements by name, not id, so our applyBrowserDOMChanges() wasn't accounting for it. The fix for ICE-3945 , which was a follow-on to ICE-3190 , for adding support for checkboxes, added better support for individual checkboxes, at the expense of multiple checkboxes. Now both types should work properly.
                Hide
                Permalink
                Mark Collette added a comment -

                While I was in there, added support for selectOneRadio. I think this was the issue for the following forum posting:
                http://www.icefaces.org/JForum/posts/list/12143.page#48503

                Subversion 18727
                icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java

                Show
                Mark Collette added a comment - While I was in there, added support for selectOneRadio. I think this was the issue for the following forum posting: http://www.icefaces.org/JForum/posts/list/12143.page#48503 Subversion 18727 icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #18745 Thu Apr 02 15:49:42 MDT 2009 mark.collette ICE-4317 : selectManyCheckbox not clearable
                Files Changed
                Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/core/src/com/icesoft/faces/context/BridgeFacesContext.java
                Hide
                Permalink
                Mark Collette added a comment -

                Backported to ICEfaces 1.7 branch
                Subversion 18745
                icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java

                Show
                Mark Collette added a comment - Backported to ICEfaces 1.7 branch Subversion 18745 icefaces\core\src\com\icesoft\faces\context\BridgeFacesContext.java
                Hide
                Permalink
                Joanne Bai added a comment -

                Tested successfully on 1.7.2-SP2 build 3

                Test environment: tomcat 6 + FF2 and IE6

                Show
                Joanne Bai added a comment - Tested successfully on 1.7.2-SP2 build 3 Test environment: tomcat 6 + FF2 and IE6
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]
                Hide
                Permalink
                Lari Tuominen added a comment -

                Is this really fixed in 1.8.0?

                See http://www.icefaces.org/JForum/posts/list/12185.page#48607 and http://jira.icefaces.org/browse/ICE-4392

                Show
                Lari Tuominen added a comment - Is this really fixed in 1.8.0? See http://www.icefaces.org/JForum/posts/list/12185.page#48607 and http://jira.icefaces.org/browse/ICE-4392
                Hide
                Permalink
                Mark Collette added a comment -

                Try trunk, since there was a regression with ICE-4066, which is fixed in ICE-4324 on trunk. That issue might be making it look like this issue wasn't resolved.

                Show
                Mark Collette added a comment - Try trunk, since there was a regression with ICE-4066 , which is fixed in ICE-4324 on trunk. That issue might be making it look like this issue wasn't resolved.
                Mark Collette made changes -
                Link This issue blocks ICE-4392 [ ICE-4392 ]

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    Mark Collette
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: