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

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: