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.