From 54674c101532811fbaa8104e7d164427937362bd Mon Sep 17 00:00:00 2001
From: gmantele <gmantele@ari.uni-heidelberg.de>
Date: Tue, 9 Sep 2014 09:47:37 +0200
Subject: [PATCH] [ADQL] Fix SQL translation of NOT BETWEEN (before it was
 translated by just BETWEEN ; NOT was forgotten)

---
 src/adql/translator/JDBCTranslator.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/adql/translator/JDBCTranslator.java b/src/adql/translator/JDBCTranslator.java
index aecda78..274be1e 100644
--- a/src/adql/translator/JDBCTranslator.java
+++ b/src/adql/translator/JDBCTranslator.java
@@ -162,7 +162,7 @@ import adql.query.operand.function.geometry.RegionFunction;
  * </p>
  * 
  * @author Gr&eacute;gory Mantelet (ARI)
- * @version 2.0 (08/2014)
+ * @version 2.0 (09/2014)
  * @since 2.0
  * 
  * @see PostgreSQLTranslator
@@ -688,7 +688,7 @@ public abstract class JDBCTranslator implements ADQLTranslator {
 
 	@Override
 	public String translate(Between comp) throws TranslationException{
-		return translate(comp.getLeftOperand()) + " BETWEEN " + translate(comp.getMinOperand()) + " AND " + translate(comp.getMaxOperand());
+		return translate(comp.getLeftOperand()) + " " + comp.getName() + " " + translate(comp.getMinOperand()) + " AND " + translate(comp.getMaxOperand());
 	}
 
 	@Override
@@ -703,7 +703,7 @@ public abstract class JDBCTranslator implements ADQLTranslator {
 
 	@Override
 	public String translate(IsNull isNull) throws TranslationException{
-		return translate(isNull.getColumn()) + " IS " + (isNull.isNotNull() ? "NOT " : "") + "NULL";
+		return translate(isNull.getColumn()) + " " + isNull.getName();
 	}
 
 	@Override
-- 
GitLab