The problem is that if you set the submitted value to null, the
validation will not be triggered. But this whole issue is something we
need to get over again, I just got rid of this else part cause
checkboxes and radiobuttons were using their own decoding anyways; and
for normal inputs the else branch is not necessary.

hope that fixes some of the issues.

regards,

Martin



On 5/25/05, Oliver Rossmueller <[EMAIL PROTECTED]> wrote:
> I stumbled over the following code (lines 82ff of
> org.apache.myfaces.renderkit.html.HtmlRendererUtils):
> 
>             //request parameter not found, nothing to decode - set
> submitted value to empty
>             //if the component has not been disabled
>             if(!isDisabledOrReadOnly(component))
>             {
>                 ((EditableValueHolder) component).setSubmittedValue(
> RendererUtils.EMPTY_STRING );
>             }
> 
> 
> The whole else branch is for components like checkboxes or radiobuttons
> where no value is submitted if they are not checked. But I could not
> find a reason for why submittedValue is set to empty string. IMHO line
> 86 should be
> 
>                 ((EditableValueHolder) component).setSubmittedValue(null);
> 
> So what am I missing here?
> 
> Oliver
> 
> 
> --
> Oliver Rossmueller
> Software Engineer and IT-Consultant
> Hamburg, Germany
> http://www.rossmueller.com
> 
>

Reply via email to