ICEfaces
  1. ICEfaces
  2. ICE-9209

REGRESSION: ace:list > Submitting selection inside iterative container causes de-select (MyFaces)

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.3
          • Fix Version/s: EE-3.3.0.GA_P01, 4.0.BETA, 4.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces3 trunk revision# 34488
            All browsers
            MyFaces specific
          • Assignee Priority:
            P2

            Description

            Selecting items from an ace:list component inside a iterative container and then submitting causes the selected items to become de-selected.

            To reproduce:
            1) Build / deploy test app located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/list
            2) Navigate to any dataTable / ui:repeat test.
            3) Select some items from the list
            4) Click submit
            Items will become de-selected.

              Activity

              Ascending order - Click to sort in descending order
              Cruz Miraback created issue -
              Cruz Miraback made changes -
              Field Original Value New Value
              Summary REGRESSION: ace:list > Submitting selection inside iterative container cases de-select (MyFaces) REGRESSION: ace:list > Submitting selection inside iterative container causes de-select (MyFaces)
              Ken Fyten made changes -
              Assignee Nils Lundquist [ nils.lundquist ]
              Assignee Priority P2 [ 10011 ]
              Ken Fyten made changes -
              Fix Version/s EE-3.3.0.GA [ 10572 ]
              Hide
              Permalink
              Nils Lundquist added a comment - - edited

              Possibly resolved with ICE-9215 changes? Cannot reproduce - ui:repeat example mentioned in ICE-9215 works as it did in Mojarra.

              Show
              Nils Lundquist added a comment - - edited Possibly resolved with ICE-9215 changes? Cannot reproduce - ui:repeat example mentioned in ICE-9215 works as it did in Mojarra.
              Nils Lundquist made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Please test this issue exclusively using the listRepeat.xhtml and listTable.xhtml example until the rest of tests are corrected.

              As I suggested in my last comment this issue was resolved by app level corrections I made to listRepeat.xhtml as part of ICE-9215. I just confirmed this theory by fixing listTable.xhtml with similar corrections.

              Any test presenting this issue must have similar corrections applied. Ids should not change during iteration (id changes are handled by the UIData parent) and EL should be output as a component attributes whenever possible to prevent unexpected updates to regions.

              Show
              Nils Lundquist added a comment - Please test this issue exclusively using the listRepeat.xhtml and listTable.xhtml example until the rest of tests are corrected. As I suggested in my last comment this issue was resolved by app level corrections I made to listRepeat.xhtml as part of ICE-9215 . I just confirmed this theory by fixing listTable.xhtml with similar corrections. Any test presenting this issue must have similar corrections applied. Ids should not change during iteration (id changes are handled by the UIData parent) and EL should be output as a component attributes whenever possible to prevent unexpected updates to regions.
              Nils Lundquist made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Test pages have been updated according instructions; all tests pass manually now except List in Table (Add Row to Top/Bottom), where once a new row is added to table top or bottom, the previous list selections are lost (tested in Firefox/Chrome, myfaces specific, trunk rev# 36592).

              Show
              Carmen Cristurean added a comment - Test pages have been updated according instructions; all tests pass manually now except List in Table (Add Row to Top/Bottom), where once a new row is added to table top or bottom, the previous list selections are lost (tested in Firefox/Chrome, myfaces specific, trunk rev# 36592).
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #36629
              Committed by nils.lundquist
              Moments ago
              ICE-9209 - Refresh dataModel at start of child rendering. Fixes ace:list table nested state problem in MyFaces.

              Show
              Nils Lundquist added a comment - Revision #36629 Committed by nils.lundquist Moments ago ICE-9209 - Refresh dataModel at start of child rendering. Fixes ace:list table nested state problem in MyFaces.
              Nils Lundquist made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #36629 Wed Jul 10 17:15:56 MDT 2013 nils.lundquist ICE-9209 - Refresh dataModel at start of child rendering. Fixes ace:list table nested state problem in MyFaces.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/list/ListRenderer.java
              Ken Fyten made changes -
              Fix Version/s EE-3.3.0.GA_P01 [ 11174 ]
              Fix Version/s EE-3.3.0.GA [ 10572 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #37264
              Committed by nils.lundquist
              A minute ago
              ICE-9209 - Refresh dataModel at start of child rendering. Fixes ace:list table nested state problem in MyFaces.

              Show
              Nils Lundquist added a comment - Revision #37264 Committed by nils.lundquist A minute ago ICE-9209 - Refresh dataModel at start of child rendering. Fixes ace:list table nested state problem in MyFaces.
              Nils Lundquist made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Confirmed Fixed with Myfaces trunk revision #37298, Tomcat 6, FF 21, Chrome, IE 7 & 9.

              Show
              Liana Munroe added a comment - Confirmed Fixed with Myfaces trunk revision #37298, Tomcat 6, FF 21, Chrome, IE 7 & 9.
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Cruz Miraback
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: