Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.8.2.GA_P02
-
Fix Version/s: EE-2.0.0
-
Component/s: Samples
-
Labels:None
-
Environment:All
-
Affects:Sample App./Tutorial
-
Workaround Exists:Yes
-
Workaround Description:Set the editable value associated with the RichDataBooleanRowProperty to false to allow the row to be editable. True to allow the row to be non-editable.
Description
Currently the RichDataBooleanRowProperty object uses a "false" value to allow rows to be editable. This should be "true" as this can cause confusion for developers.
Upon review this seems like a logic error in the Composite Showcase app. bean itself (Employee class). The "editable" property on that bean is using "false" to make things editable, and "true" to disable editing. This logic should be reversed in the showcase app code.
Note that it is important not to change the component logic itself as to avoid breaking backwards compatibility with existing apps.
Upon review this seems like a logic error in the Composite Showcase app. bean itself (Employee class). The "editable" property on that bean is using "false" to make things editable, and "true" to disable editing. This logic should be reversed in the showcase app code.
Note that it is important not to change the component logic itself as to avoid breaking backwards compatibility with existing apps.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
The more I think about it the more I think this isn't an issue with the RichDataBooleanRowProperty but with the RichDataGridBean object. Possibly there is an issue in the row selection logic. No matter what the row object editable value is called setting it to true should allow it to be editable and false should be non-editable.
Show
Arran Mccullough
added a comment - The more I think about it the more I think this isn't an issue with the RichDataBooleanRowProperty but with the RichDataGridBean object. Possibly there is an issue in the row selection logic. No matter what the row object editable value is called setting it to true should allow it to be editable and false should be non-editable.
This seems to be resolved in the ICEfaces EE 2.0 beta release.