You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by "Andrus Adamchik (JIRA)" <ji...@apache.org> on 2013/02/24 14:10:12 UTC
[jira] [Closed] (CAY-1522) EJBQL query don't support quotes
[ https://issues.apache.org/jira/browse/CAY-1522?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrus Adamchik closed CAY-1522.
--------------------------------
Resolution: Fixed
Fix Version/s: 3.2M1
Fixed. Ilya, thanks for the patch as usual. I reworked it a bit. I completely agree that determining quote/no-quote flag should be folded in a common reusable class. I just made this class QuotingStrategy istelf, instead of an extra QuotingSupport (I also removed all cases unrelated to EJBQL where we'd check for quote/no-quote logic inline). Otherwise I used your patch.
A note to self - we can probably port commit r1449473 to 3.1 branch with some effort. Don't have time to do it now.
> EJBQL query don't support quotes
> ---------------------------------
>
> Key: CAY-1522
> URL: https://issues.apache.org/jira/browse/CAY-1522
> Project: Cayenne
> Issue Type: Bug
> Components: Core Library
> Reporter: Olga Tkacheva
> Fix For: 3.2M1
>
> Attachments: 0001-added-test-EJBQLQuery-with-quote.patch, 0001-CAY-1522-EJBQL-query-don-t-support-quotes.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira