[richfaces-issues] [JBoss JIRA] Commented: (RF-997) Calendar - incorrect use of converters

Daniel Soneira (JIRA) jira-events at lists.jboss.org
Thu Nov 15 06:09:18 EST 2007


    [ http://jira.jboss.com/jira/browse/RF-997?page=comments#action_12387689 ] 
            
Daniel Soneira commented on RF-997:
-----------------------------------

my problem still persists with richfaces-ui-3.1.3-20071115.052920-19-bin.zip

simple code: 

<h:form>
        <rich:calendar enableManualInput="true" popup="true" />
        <h:commandButton value="submit"/>
</h:form>

===> enter "asdf" into the input field -> wham!! - still an exception is thrown in RENDER phase.

same as if you would enter characters into a field where you expect numbers and the whole application crashes because of that.

expected behaviour: just show the page with the last valid value -> add the conversion error to the faces messages (which automatically is done when a converter throws an ConverterException).

again: There should be NO CONVERSION whatsoever in the render phase.

> Calendar - incorrect use of converters
> --------------------------------------
>
>                 Key: RF-997
>                 URL: http://jira.jboss.com/jira/browse/RF-997
>             Project: RichFaces
>          Issue Type: Bug
>    Affects Versions: 3.1.0
>            Reporter: Maksim Kaszynski
>         Assigned To: Alexej Kushunin
>             Fix For: 3.2.0
>
>
> Calendar component uses converter.getAsObject on render phase. 
> This is wrong. 
> Correct behavior - use converter.getAsObject  on process validations phase.
> In case of bad user input - converter will give conversion exception, display submitted value in input field, in calendar grid select last user selected day (if exists).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        



More information about the richfaces-issues mailing list