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

        Hide
        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
        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
        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
        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
        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
        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: