You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cayenne.apache.org by "Andrus Adamchik (JIRA)" <de...@cayenne.apache.org> on 2008/10/01 22:15:11 UTC

[jira] Commented: (CAY-1111) Expression.toEJBQL() should not output ObjectId

    [ https://issues.apache.org/cayenne/browse/CAY-1111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133#action_13133 ] 

Andrus Adamchik commented on CAY-1111:
--------------------------------------

I acknowledge the problem. It was discussed here some time back: http://markmail.org/message/jfopksclou3hqa7j

The solution seems to be a little more involved... What if scalar is date? or a multi-column PK object?My suggestion per quoted dev list message was to reorg the method and pass a parameter map down to the method to handle all those cases as query parameters.

> Expression.toEJBQL() should not output ObjectId
> -----------------------------------------------
>
>                 Key: CAY-1111
>                 URL: https://issues.apache.org/cayenne/browse/CAY-1111
>             Project: Cayenne
>          Issue Type: Bug
>          Components: Cayenne Core Library
>    Affects Versions: 3.0
>         Environment: N/A
>            Reporter: Dave Dombrosky
>            Assignee: Andrus Adamchik
>         Attachments: encodeAsEJBQL.patch
>
>
> The Expression.toEJBQL method in ASTScalar.java needs to be fixed so that the PK is used as the value instead of the ObjectId.
> Currently, the method outputs things like "a.artist = <ObjectId:Artist, id=1234>", which is not valid EJBQL syntax.

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