Details
-
Type:
Bug
-
Status: Closed
-
Priority:
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.
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.
-
Hide
-
-
META-INF/MANIFEST.MF
-
WEB-INF/lib/backport-util-concurrent.jar
-
WEB-INF/.../TableBean$QuestionItem.class
-
WEB-INF/lib/commons-digester.jar
-
WEB-INF/lib/FastInfoset.jar
-
pageA.jspx
-
WEB-INF/lib/icefaces-comps.jar
-
WEB-INF/lib/jxl.jar
-
WEB-INF/classes/.../sortable/TableBean.java
-
WEB-INF/classes/.../SortableList.class
-
WEB-INF/.../krysalis-jCharts-1.0.0-alpha-1.jar
-
WEB-INF/lib/commons-beanutils.jar
-
WEB-INF/faces-config.xml
-
WEB-INF/classes/.DS_Store
-
WEB-INF/lib/commons-fileupload.jar
-
WEB-INF/lib/jsf-impl.jar
-
index.jsp
-
WEB-INF/lib/jsf-api.jar
-
WEB-INF/lib/icefaces.jar
-
WEB-INF/web.xml
-
WEB-INF/lib/commons-logging.jar
-
WEB-INF/classes/.../sortable/TableBean.class
-
WEB-INF/lib/commons-collections.jar
-
WEB-INF/classes/.../SortableList.java
-
-
Activity
| Field | Original Value | New Value |
|---|---|---|
| Attachment | sc9079.war [ 12285 ] |
| Salesforce Case | [] | |
| 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 required attribute is still set to false for both selectOneRadio components. |
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. |
| Salesforce Case | [] | |
| Fix Version/s | 1.8.2-EE-GA_P02 [ 10226 ] | |
| Assignee Priority | P1 | |
| Assignee | Adnan Durrani [ adnan.durrani ] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #21177 | Wed Apr 07 17:18:08 MDT 2010 | adnan.durrani | Fix for |
| Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelseries/UISeries.java
|
| Status | Open [ 1 ] | In Progress [ 3 ] |
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
| Status | In Progress [ 3 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
| Assignee Priority | P1 | |
| Assignee | Adnan Durrani [ adnan.durrani ] |
| Resolution | Fixed [ 1 ] | |
| Status | Closed [ 6 ] | Reopened [ 4 ] |
| Assignee Priority | P1 |
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Assignee Priority | P1 | |
| Resolution | Fixed [ 1 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
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.