[ 
http://jira.xwiki.org/jira/browse/XWIKI-1820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_19343
 ] 

Sergiu Dumitriu commented on XWIKI-1820:
----------------------------------------

I committed the patch with several changes:
- The right way to declare the log object is private static final Log LOG = 
LogFactory.getLog(JodaTimePlugin.class);
- I added/modified some javadoc
- Changed the version to 1.0-SNAPSHOT
- I enabled the plugin in XE, as it was not enabled, because it does not 
consider the list of plugins defined in xwiki-platform-pom/pom.xml, but the one 
in xwiki-products-enterprise/web/pom.xml

I manually tested it to see if it works, but we need some functional tests for: 
basic usage (the plugin is found and calling a method returns something), 
localization, style, style+localization.

You also have to write the documentation on xwiki.org, in the code/plugins area.

> Joda Time Plugin
> ----------------
>
>                 Key: XWIKI-1820
>                 URL: http://jira.xwiki.org/jira/browse/XWIKI-1820
>             Project: XWiki Platform
>          Issue Type: New Feature
>          Components: Plugin - Other
>    Affects Versions: 1.2 M1
>         Environment: any
>            Reporter: Marius Dumitru Florea
>         Assigned To: Marius Dumitru Florea
>         Attachments: jodatime-plugin.diff.txt, xwiki-platform-pom.diff.txt
>
>
> XWiki could use a DateTime plugin for those that need to easily manipulate 
> dates from velocity scripts inside xwiki documents. JodaTime 
> (http://joda-time.sourceforge.net/) is a great framework for this purpose.

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

        
_______________________________________________
notifications mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/notifications

Reply via email to