ICEfaces
  1. ICEfaces
  2. ICE-7574

MyFaces 2: DataTable state saving not working

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC1
    • Fix Version/s: 3.0.RC2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      MyFaces 2 ACE
    • Assignee Priority:
      P1

      Description

      I'm creating this case as a general, parent case for a small group of related issues that already have JIRAs:

      ICE-7560: ace:dataTable filter doesn't recover from "no-results" state without reset
      ICE-7561: showcase app ace:contextMenu demo "reset all" button doesn't work the first time it's pressed
      ICE-7563: ace:draggable/ace:droppable Table demo, Reset doesn't reset source dataTable, just destination one

      The current org.icefaces.ace.component.datatable.DataTable implementations of saveState/restoreState cause the above issues when running with MyFaces and it's implementation of partial state saving. Removing the saveState/restoreState method logic in the DataTable and deferring to the parent class gets rid of the problems but then the state of certain fields is not saved properly.

        Issue Links

          Activity

          Deryk Sinotte created issue -
          Deryk Sinotte made changes -
          Field Original Value New Value
          Link This issue is duplicated by ICE-7560 [ ICE-7560 ]
          Deryk Sinotte made changes -
          Link This issue is duplicated by ICE-7561 [ ICE-7561 ]
          Deryk Sinotte made changes -
          Link This issue is duplicated by ICE-7563 [ ICE-7563 ]
          Deryk Sinotte made changes -
          Salesforce Case []
          Assignee Priority P1
          Assignee Ken Fyten [ ken.fyten ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #26879 Wed Dec 14 14:02:31 MST 2011 nils.lundquist ICE-7574 - Converted DataTable state saving from Mojarra implementation to generic state handler. Required ComponentArtifact change to Field setters so that null Fields are not saved.
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/generator/src/org/icefaces/ace/generator/artifacts/ComponentArtifact.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableRenderer.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableMeta.java
          Nils Lundquist made changes -
          Link This issue is duplicated by ICE-7561 [ ICE-7561 ]
          Nils Lundquist made changes -
          Link This issue is duplicated by ICE-7563 [ ICE-7563 ]
          Hide
          Nils Lundquist added a comment -

          State saving resolved in r26879
          ICE-7561 / ICE-7563 look related to data table model regeneration when the underlying dataset changes, removed from this issue and assigned to myself for further analysis.

          Show
          Nils Lundquist added a comment - State saving resolved in r26879 ICE-7561 / ICE-7563 look related to data table model regeneration when the underlying dataset changes, removed from this issue and assigned to myself for further analysis.
          Nils Lundquist made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 3.0.RC2 [ 10313 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Ken Fyten
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: