You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by Josh Kennedy <jo...@meltmedia.com> on 2009/09/28 22:56:56 UTC

ContentLoader Date Parsing Patch

I have created a patch to the ContentLoader to include the ability to  
be date aware from the JSON reader.  It runs a regular expression  
against the value if it is a string to try and determine how it should  
be handled, and calls the ContentCreator with the PropertyType.Date as  
the third parameter.  The Content Creator runs the regular expression  
again to enable support for multiple date formats.  It handles both  
date arrays and single date fields.

Not sure if I can create a JIRA issue for this, or what the process  
for inclusion is.  Let me know if there are any questions or changes  
needed for it to be included.


Re: ContentLoader Date Parsing Patch

Posted by Bertrand Delacretaz <bd...@apache.org>.
Hi Josh,

On Mon, Sep 28, 2009 at 10:56 PM, Josh Kennedy <jo...@meltmedia.com> wrote:
> ...Not sure if I can create a JIRA issue for this, or what the process for
> inclusion is.  Let me know if there are any questions or changes needed for
> it to be included....

Creating a JIRA issue is the right thing to do, attach your patch
there and check the "grant permission to Apache for inclusion"
checkbox if you agree with that.

Having the code there allows us to review the patch.

Automated tests (or patches to existing tests) help in deciding to
apply the patch ;-)

-Bertrand