ICEfaces
  1. ICEfaces
  2. ICE-9087

ace:dataTable rowEditSubmit event combined with toggleOnInvalidEdit breaks editing

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      *
    • Assignee Priority:
      P2

      Description

      When using ace:ajax to execute rowEditSubmit events, the DataTable itself is executed, and uses iterate() to execute children, rather than allowing visitTree to descend within the component to execute them.

      When iterate() is used, visibility of child components is tested before phases are processed on them. However, since toggleOnInvalidEdit is used, visibility toggling occurs early in the lifecycle so that it is not prevented by a invalid input.

      This early visibility change combined with the testing done by iterate() results in no changes being applied in valid editing cases.

        Activity

        Nils Lundquist created issue -
        Nils Lundquist made changes -
        Field Original Value New Value
        Assignee Nils Lundquist [ nils.lundquist ]
        Ken Fyten made changes -
        Assignee Priority P1 [ 10010 ]
        Ken Fyten made changes -
        Fix Version/s 3.4 [ 10770 ]
        Fix Version/s 3.3 [ 10370 ]
        Ken Fyten made changes -
        Assignee Priority P1 [ 10010 ] P2 [ 10011 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA [ 10572 ]
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: