You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jackrabbit.apache.org by th...@apache.org on 2011/06/20 12:00:50 UTC

svn commit: r1137570 - /jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java

Author: thomasm
Date: Mon Jun 20 10:00:50 2011
New Revision: 1137570

URL: http://svn.apache.org/viewvc?rev=1137570&view=rev
Log:
JCR-2996: QOM to SQL2 doesn't cast numeric literals

Modified:
    jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java

Modified: jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java?rev=1137570&r1=1137569&r2=1137570&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java Mon Jun 20 10:00:50 2011
@@ -234,13 +234,13 @@ public class QOMFormatter implements Que
                 appendCastLiteral(v.getString(), "DATE");
                 break;
             case PropertyType.DECIMAL:
-                append(v.getString());
+                appendCastLiteral(v.getString(), "DECIMAL");
                 break;
             case PropertyType.DOUBLE:
-                append(v.getString());
+                appendCastLiteral(v.getString(), "DOUBLE");
                 break;
             case PropertyType.LONG:
-                append(v.getString());
+                appendCastLiteral(v.getString(), "LONG");
                 break;
             case PropertyType.NAME:
                 appendCastLiteral(v.getString(), "NAME");