[ 
https://issues.apache.org/jira/browse/CAY-1213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12911183#action_12911183
 ] 

Andrus Adamchik commented on CAY-1213:
--------------------------------------

Andrei, I applied the patch. Looks good. One thing that I had to change is the 
use of StringUtils. Looks like commons-lange library leaked into Cayenne via 
Velocity, although Cayenne itself doesn't (and shouldn't) have a dependency on 
commons-lang. So I replaced with with a simple 'indexOf' lookup, which should 
also perform better, as it doesn't do cloning or regex matching, just a simple 
substring.

> Cayenne should support enum types in qualifier statements/expression handling
> -----------------------------------------------------------------------------
>
>                 Key: CAY-1213
>                 URL: https://issues.apache.org/jira/browse/CAY-1213
>             Project: Cayenne
>          Issue Type: Improvement
>          Components: Core Library
>    Affects Versions: 3.0M5
>            Reporter: Robert Zeigler
>            Assignee: Andrus Adamchik
>             Fix For: Undefined future
>
>         Attachments: CAY-1213.patch
>
>
> Currently, cayenne doesn't support java enums in its expression syntax and, 
> by extension, in qualifier statements.  This makes it impossible to use an 
> enum type as a class descriminator for inheritance.
> Enum expressions would look like:
> propertyName = some.package.Type.CONSTANT
> to make it consistent with JPA.

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

Reply via email to