ICEfaces-EE
  1. ICEfaces-EE
  2. IPCK-256

RichDataBooleanRowProperty needs to be set to false in Row Object to allow the row to be editable

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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.
      1. Employee.java
        2 kB
        Arran Mccullough
      2. TestBean.java
        4 kB
        Arran Mccullough

        Activity

        Hide
        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.

        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.
        Hide
        Arran Mccullough added a comment -

        This seems to be resolved in the ICEfaces EE 2.0 beta release.

        Show
        Arran Mccullough added a comment - This seems to be resolved in the ICEfaces EE 2.0 beta release.

          People

          • Assignee:
            Philip Breau
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: