ICEfaces
  1. ICEfaces
  2. ICE-8527

DataTable, row/cell editing crashes when using LazyDataModel

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Cannot Reproduce
    • Affects Version/s: 3.1
    • Fix Version/s: 3.2
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Icefaces 3.1
      JSF 2.1.1
      Tomcat 7.0.28
      Java 7

      Description

      When using a LazyDataModel on da DataTable with row/cell editing enabled, and you click on a cell to edit, change the value and click on V, ALL the cells change into editable mode, and nothing works anymore. I have to either reload the page or change the value back and cancel the operation.

        Activity

        Hide
        Nils Lundquist added a comment - - edited

        It sounds to me like the real issue here is that all of the rows are sharing a row state or are otherwise being enabled by the editing of the one row.
        Are you implementing hashCode() and equals() for the row objects being lazily loaded?
        A test case would make the issue more clear.

        Show
        Nils Lundquist added a comment - - edited It sounds to me like the real issue here is that all of the rows are sharing a row state or are otherwise being enabled by the editing of the one row. Are you implementing hashCode() and equals() for the row objects being lazily loaded? A test case would make the issue more clear.
        Hide
        Leo Milhouse added a comment - - edited

        Hey Nil,
        It's been over 2 months since I actually had this problem, and since I couldn't make it work I threw away the code for this particular functionality. I usually implement hash and equals for my VOs, but I could've missed this particular one, no way to know now.
        What I pledge to do is, the second I have some free time, try to implement this again, double checking the hash and equals in the lazily loaded row objects, and get back to you. If it doesnt work again I'll post a sample of the code here.
        Again, thanks for the attention.

        Show
        Leo Milhouse added a comment - - edited Hey Nil, It's been over 2 months since I actually had this problem, and since I couldn't make it work I threw away the code for this particular functionality. I usually implement hash and equals for my VOs, but I could've missed this particular one, no way to know now. What I pledge to do is, the second I have some free time, try to implement this again, double checking the hash and equals in the lazily loaded row objects, and get back to you. If it doesnt work again I'll post a sample of the code here. Again, thanks for the attention.
        Hide
        Nils Lundquist added a comment - - edited

        Resolving pending test case/new details from customer.

        Show
        Nils Lundquist added a comment - - edited Resolving pending test case/new details from customer.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Leo Milhouse
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: