[jira] [Commented] (WW-4799) make DateConverter configurable

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on WW-4799:
------------------------------------

GitHub user dcabasson opened a pull request:

    https://github.com/apache/struts/pull/141

    WW-4799 Adding struts.date.format as a conversion format for DateConv…

    …erter

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dcabasson/struts master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/struts/pull/141.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #141
    
----

----


> make DateConverter configurable
> -------------------------------
>
>                 Key: WW-4799
>                 URL: https://issues.apache.org/jira/browse/WW-4799
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Christoph Nenning
>            Priority: Minor
>             Fix For: 2.6
>
>
> Currently DateConverter has some logic to test which DateFormat is able to parse given Date and use that one.
> The order in which DateFormats are tested is:
> * Locale specific DateTime LONG
> * Locale specific DateTime MEDIUM
> * Locale specific DateTime SHORT
> * rfc3339 ("yyyy-MM-dd'T'HH:mm:ss")
> * Locale specific Date SHORT
> * Locale specific Date MEDIUM
> * Locale specific Date LONG
> * rfc3339dateOnly ("yyyy-MM-dd")
> Would be great if apps could configure this. E.g. by:
> * defining an own list
> * defining order of objects in current list
> * defining just one DateFormat to be used
> * defining just which of Locale specific DateFormats to use (SHORT vs LONG)



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