volosied commented on code in PR #492: URL: https://github.com/apache/myfaces/pull/492#discussion_r1082923271
########## api/src/main/java/jakarta/faces/component/UISelectOne.java: ########## @@ -69,19 +69,20 @@ public String getFamily() } /** - * Verify that when ever there is a ValueExpression and submitted value is not empty, then + * Check whether a group exists and then * visit all the UISelectItem elements within the UISelectOne radio components to check if - * the submitted value exists in any of the select items. + * the submitted value is empty (ie. not submitted) or if a previous group item has been + * has failed to be validated (if no so further validation processing is needed) * * @see jakarta.faces.component.UIInput#processValidators(jakarta.faces.context.FacesContext) */ @Override public void processValidators(FacesContext context) { String group = getGroup(); - ValueExpression ve = getValueExpression("value"); String submittedValue = (String) getSubmittedValue(); - if (group != null && !group.isEmpty() && ve != null && !isEmpty(submittedValue)) + + if (group != null && !group.isEmpty()) Review Comment: I need to check for the "non-empty UIInput.getSubmittedValue()" case. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@myfaces.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org