Screenshot showing the problem. Validation of the input field is triggered by a row selection.

Screenshot after the fix. No validation error message for input field anymore.

How to test:

In File component-showcase/web/inc/components/tableRowSelection.jspx, add an input field with validation requirements, e.g.:

<ice:inputText id="testInput" required="true"/>
<ice:message for="testInput"/>

Before the fix, you would get a validation error when selecting a row; after the fix, there should be no validation triggered. See the attached screenshots.

Screenshot showing that the "required" validation is skipped but the length validation is not skipped.

Screenshot after another fix, showing that the length validation is skipped as well.

How to test:

In component-showcase/web/inc/components/tableRowSelection.jspx, add input fields with validation requirements, e.g.:

<ice:inputText id="testInput1" required="true"/>
<ice:message for="testInput1"/><br/>
<ice:inputText id="testInput2" required="false">
<f:validateLength maximum="2"/>
</ice:inputText>
<ice:message for="testInput2"/>

Before the fix, you would get validation errors when selecting a row; after the fix, there should be no validation triggered. See the attached screenshots.

We should revert this change prior to the 1.7 final release. The scenario that this was attempting to improve support for can affect numerous components, changing the rowSelector specifically is undesirable.

There is also a new request for an additional tutorial to demonstrate application examples that resolve the issues this fix tries to address: ICE-2701.

Original changes have been backed out.

Won't Fix.

While we wanted to remove the call to FacesContext.renderResponse() from the RowSelector component's decode method, the Javascript should have still used partialSubmit. This is fixed in: ICE-2749.