ICEfaces
  1. ICEfaces
  2. ICE-5593

Validation firing on non-required input components

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2-EE-GA
          • Fix Version/s: 1.8.2-EE-GA_P02
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            -

            Description

            The attached test case has a dataTable with 3 selectOneRadio components. The row level object of the table has a boolean attribute bound to the required attribute of the selectOnRadio component. In this case, only the first selectOneRadio is required. Here are the steps to reproduce the issue:

            1. Leave all fields unchecked and click submit. The first selectOneRadio will fire a validation error indicating the value is required, which is correct.
            2. If you now check the first selectOneRadio and click the submit button, the two non-required selectOneRadio components will fire validation errors indicating they are required fields. This is incorrect as the required attribute of each is still set to false.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Mark Collette added a comment -

              Each table row renders an asterisk * if that row is required, and it's only rendered for the first row, so it looked like the bottom two rows should know they're not required. That's what makes it so puzzling.

              Show
              Mark Collette added a comment - Each table row renders an asterisk * if that row is required, and it's only rendered for the first row, so it looked like the bottom two rows should know they're not required. That's what makes it so puzzling.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Stale code removed.
              Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
              Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
              Completed: At revision: 21177

              Show
              Adnan Durrani added a comment - Stale code removed. Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java Completed: At revision: 21177

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Tyler Johnson
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: