Assigning to Ken for delegation.

I believe ace:cellEditor component is the one not respecting the JSF contracts. When the row is selected (and not in edit mode) the input elements used to edit the cells are not rendered. The problem is that the decision to render them is not based on isRendered() component state. So when the row is only selected the submit will not include the cell input values because they are not rendered but the server side sees them as rendered during processDecode phase (because the render state of the input elements is "true").

Modified cell editor's processDecodes method to modify the render state of the "input" facet based on the edit mode of the row state. Also modified RowState to change the render state of the cells when they are added or removed from the list of edited cells.

Confirmed fixed using ICEfaces EE 3.2.0.GA.Build13 on Tomcat7 in Firefox18.

Re-opening because one of the issues that this JIRA fixed started to occur again. See ICE-8962 for specific details.

The ICE-8962 regression is absolutely unrelated to this issue. Resolving this one but reopening ICE-8962.

This issue is known, but cannot be worked around without the cooperation of the MyFaces team or extensive hacking of the CellEditor visiting.

The 'rendered' property of components is not state saved correctly in iterative cases. Without proper 'rendered' state MyFaces will continuously complain about the the lack of input for components inside an conditionally excluded input facet.

Revision #36342
Committed by nils.lundquist
A minute ago
ICE-8903 - CellEditor decode temporarily alters facet rendered state to avoid MyFaces warning about lack of input for conditonally unrendered facets. Programmatic conditional rendering within a UIData may not use the rendered attribute permanently due to it not being state saved iteratively.

This change avoids the warning, but doesn't change the input processing greatly. Hopefully it avoids whatever this warning was attempting to have us avoid.