[jira] Commented: (TOMAHAWK-3) inputCalendar with CalendarConverter loses value data

["Jon Harley (JIRA)" <de...@myfaces.apache.org>]

    [ https://issues.apache.org/jira/browse/TOMAHAWK-3?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641398#action_12641398 ] 

Jon Harley commented on TOMAHAWK-3:
-----------------------------------

I agree that this component should work with any suitable converter - suggesting converters must have a dependency on a MyFaces interface is not on. The point of converters in JSF is that they should be general-purpose.

My use case is to use it with Joda objects in the model too. I agree that this is a major bug in this component.

> inputCalendar with CalendarConverter loses value data
> -----------------------------------------------------
>
>                 Key: TOMAHAWK-3
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-3
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: Calendar
>         Environment: windows xp pro
>            Reporter: Sven Vogt
>
> When I set a Converter for java.util.Calendar to inputCalendar, the value will not be rendered.
> <t:inputCalendar id="calendar_1"
> 	         value="#{exampleInputBean.calendar}"
> 	         renderAsPopup="true">
> 	<f:converter converterId="de.orgaplan.faces.CalendarConverter" />
> </t:inputCalendar>
> Because the HtmlCalendarRenderer throws an IllegalArgumentException and set the value to null.
> My Converter is not an Instance of DateConverter.
>         Date value;
>         try
>         {
>             // value = RendererUtils.getDateValue(inputCalendar);
>             Converter converter = getConverter(inputCalendar);
>             if (converter instanceof DateConverter)
>             {
>                 value = ((DateConverter) converter).getAsDate(facesContext, component);
>             }
>             else
>             {
>                 value = RendererUtils.getDateValue(inputCalendar);
>             }
> 		}
>         catch (IllegalArgumentException illegalArgumentException)
>         {
>             value = null;
>         }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.