[ 
http://jira.codehaus.org/browse/MOJO-1305?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mark Struberg reassigned MOJO-1305:
-----------------------------------

    Assignee: Mark Struberg

> Polish code
> -----------
>
>                 Key: MOJO-1305
>                 URL: http://jira.codehaus.org/browse/MOJO-1305
>             Project: Mojo
>          Issue Type: Task
>          Components: openjpa
>            Reporter: Benjamin Bentmann
>            Assignee: Mark Struberg
>            Priority: Trivial
>         Attachments: openjpa.patch
>
>
> See attached patch for some proposed updates.
> {code:java}
> /**
>  * Location where <code>persistence-enabled</code> classes are located.
>  * 
>  * @parameter expression="${openjpa.classDir}"
>  *            default-value="${project.build.outputDirectory}"
>  * @required
>  */
> protected File classes;
> {code}
> Looks inconsistent to name the mojo paramater "classes" but the expression 
> refers to "openjpa.classesDir" (instead of "openjpa.classes").
> Given that's the first official release IIRC, it's worth to also have a look 
> at MOJO-1100. Would be either now or never.

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

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to