From c6d684155d58c59e70f5347685db1346e3eb44d9 Mon Sep 17 00:00:00 2001
From: Adam Rauch <adam@labkey.com>
Date: Wed, 27 Nov 2024 15:15:06 -0800
Subject: [PATCH] Fix some date compare clauses

---
 api/src/org/labkey/api/data/CompareType.java | 33 ++------------------
 1 file changed, 2 insertions(+), 31 deletions(-)

diff --git a/api/src/org/labkey/api/data/CompareType.java b/api/src/org/labkey/api/data/CompareType.java
index a415d190d04..b29e48de183 100644
--- a/api/src/org/labkey/api/data/CompareType.java
+++ b/api/src/org/labkey/api/data/CompareType.java
@@ -1653,17 +1653,12 @@ private abstract static class DateCompareClause extends CompareClause
         @Override
         String toWhereClause(SqlDialect dialect, String alias)
         {
+            // See Issue 51472
             return dialect.isSqlServer() ?
-                dialect.getColumnSelectName(alias) + getSqlServerComparison() :
+                dialect.getColumnSelectName(alias) + _comparison.getSql().replace("?", SS_CAST) :
                 super.toWhereClause(dialect, alias);
         }
 
-        // See Issue 51472
-        protected String getSqlServerComparison()
-        {
-            throw new IllegalStateException("Must override either toWhereClause() or getSqlServerComparison()");
-        }
-
         @Override
         protected void appendFilterText(StringBuilder sb, ColumnNameFormatter formatter)
         {
@@ -1776,12 +1771,6 @@ static class DateGtCompareClause extends DateCompareClause
             super(fieldKey, DATE_GT, " > ", rawFilterValue, addOneDay(startValue));
         }
 
-        @Override
-        protected String getSqlServerComparison()
-        {
-            return " > " + SS_CAST;
-        }
-
         @Override
         protected boolean meetsCriteria(ColumnRenderProperties col, Object value)
         {
@@ -1806,12 +1795,6 @@ static class DateGteCompareClause extends DateCompareClause
             super(fieldKey, DATE_GTE, " >= ", rawFilterValue, startValue);
         }
 
-        @Override
-        protected String getSqlServerComparison()
-        {
-            return " >= " + SS_CAST;
-        }
-
         @Override
         protected boolean meetsCriteria(ColumnRenderProperties col, Object value)
         {
@@ -1836,12 +1819,6 @@ static class DateLtCompareClause extends DateCompareClause
             super(fieldKey, DATE_LT, " < ", rawFilterValue, startValue);
         }
 
-        @Override
-        protected String getSqlServerComparison()
-        {
-            return " < " + SS_CAST;
-        }
-
         @Override
         protected boolean meetsCriteria(ColumnRenderProperties col, Object value)
         {
@@ -1866,12 +1843,6 @@ static class DateLteCompareClause extends DateCompareClause
             super(fieldKey, DATE_LTE, " <= ", rawFilterValue, addOneDay(startValue));
         }
 
-        @Override
-        protected String getSqlServerComparison()
-        {
-            return " <= " + SS_CAST;
-        }
-
         @Override
         protected boolean meetsCriteria(ColumnRenderProperties col, Object value)
         {