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.

Original changes have been backed out.

Won't Fix.

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.

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.

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

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

Doesn't seem to work with length validator.

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 after the fix. No validation error message for input field anymore.

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