[ 
https://issues.apache.org/jira/browse/MYFACES-4169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16255426#comment-16255426
 ] 

Eduardo Breijo edited comment on MYFACES-4169 at 11/16/17 2:47 PM:
-------------------------------------------------------------------

I noticed that moving the super.processValidators inside the if block will only 
work if the radio buttons have a group defined. If no group is defined, then 
validation will not be processed. It makes more sense to only call 
super.processValidators when there is a submittedValue. I'll provide a new 
patch with that minor change.


was (Author: eduardobreijo):
I noticed that there moving the super.processValidators inside the if block 
will only work if the radio buttons have a group defined. If no group is 
defined, then validation will not be processed. It makes more sense to only 
call super.processValidators when there is a submittedValue. I'll provide a new 
patch with that minor change.

> selectOneRadio grouping broken
> ------------------------------
>
>                 Key: MYFACES-4169
>                 URL: https://issues.apache.org/jira/browse/MYFACES-4169
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-372
>    Affects Versions: 2.3.0-beta
>            Reporter: Paul Nicolucci
>            Assignee: Thomas Andraschko
>             Fix For: 2.3.0
>
>         Attachments: JSF23SelectOneRadioGroup.war, MYFACES-4169.patch, 
> SelectOneRadioGroupBean.java, index.xhtml, selectOneRadioGroupOneValue.xhtml
>
>
> I was testing out the new group attribute on the <h:selectOneRadio/> 
> component and found that it does not look to work in all scenarios:
> I've provided a test application to shows a working scenario and then 
> multiple scenarios that I think should work but don't.
> The app can be deployed on tomcat. 
> You can drive a request to the index page which will contain the links what 
> doesn't work: 
> http://localhost:8080/JSF23SelectOneRadioGroup/index.xhtml
> Or you can directly verify what does not work by sending requests to:
> localhost:8080/JSF23SelectOneRadioGroup/error.xhtml
> localhost:8080/JSF23SelectOneRadioGroup/error2.xhtml
> localhost:8080/JSF23SelectOneRadioGroup/error3.xhtml
> localhost:8080/JSF23SelectOneRadioGroup/error4.xhtml



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to