MyFaces Core
  1. MyFaces Core
  2. MYFACES-3398

CLONE - facet state is lost inside datatable


    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.2.11
    • Fix Version/s: 1.2.11
    • Component/s: None
    • Labels:


      Assume, a Composite Component contains input fields, i.e any subclass of EditableValueHolder (e.g. inputText, inputTextarea, ..). If this CC is than used in a subclass of UIData, e.g. a dataTable, the values entered in the CCs input fields get lost (despite for the last row in the Model of the UIData object).

      This seems to be related to the fact how the method "saveDescendantComponentStates(..)" of the UIData class checks, if a component has childs, whos state must be saved, before changing the rowIndex. The mentioned method only checks "child.getChildCount() > 0", but getChildCount() is 0 for a CC. I assume it would be necessary to check the FacetCount for a CC instead.

      The attached WAR file can help to reproduce the issue with a simple example.

        Issue Links


          Michael Dietrich created issue -
          Leonardo Uribe made changes -
          Field Original Value New Value
          Link This issue is a clone of MYFACES-3385 [ MYFACES-3385 ]
          Leonardo Uribe made changes -
          Fix Version/s 2.1.4 [ 12317868 ]
          Fix Version/s 2.0.10 [ 12317870 ]
          Affects Version/s 1.2.11 [ 12316650 ]
          Affects Version/s 2.1.2 [ 12316512 ]
          Leonardo Uribe added a comment -

          Regression of MYFACES-3385 into 1.2.x branch

          Leonardo Uribe added a comment - Regression of MYFACES-3385 into 1.2.x branch
          Leonardo Uribe made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Fix Version/s 1.2.11 [ 12316650 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          1m 34s 1 Leonardo Uribe 17/Nov/11 04:21


            • Assignee:
              Leonardo Uribe
              Michael Dietrich
            • Votes:
              0 Vote for this issue
              1 Start watching this issue


              • Created: