ICEfaces
  1. ICEfaces
  2. ICE-7561

MyFaces: showcase app ace:contextMenu demo "reset all" button doesn't work the first time it's pressed

          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: Sample Apps
          • Labels:
            None
          • Environment:
            ICEfaces 3, Showcase sample app, context menu demo, MyFaces 2.1.3
          • Assignee Priority:
            P1

            Description

            ace:contextMenu: demo in showcase sample app.:

            In a Table:
            All browsers: It takes two clicks of 'Undo All' to see the cars removed after adding cars to the table.

            This works fine with Mojarra but not with MyFaces.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26883 Wed Dec 14 16:14:45 MST 2011 nils.lundquist ICE-7561 / ICE-7563 - Fixed MyFaces table + other component out of sync interaction. MyFaces appears to generate the DataModel for the DataTable lifecycle before the features being used in either issue has a chance to alter the backing List of the DataTable. Since the DataModel is cached for the life of the component (as was derived from the Mojarra behaviour for getDataModel) we must regenerate the dataModel prerender to fix the issue. This is required in Mojarra when the DataTable has altered its own backing List during the lifecycle, like in filtering. The autofiltering the table attempts during its first render is responsible for the state saving change Deryk originally made fixing the issue by continually attempting a filter, and thus regenerating the model.
              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/component/src/org/icefaces/ace/component/datatable/DataTableRenderer.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26842 Tue Dec 13 08:36:15 MST 2011 mark.collette ICE-7561 : MyFaces: showcase app ace:contextMenu demo "reset all" button doesn't work the first time it's pressed
              (Reverted)
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #26838 Mon Dec 12 15:27:55 MST 2011 deryk.sinotte ICE-7561: remove the old-style state save/restore methods and rely on the superclass.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: