From 09e27159dd7f0fe8e2a9cf07f885e734c0459e25 Mon Sep 17 00:00:00 2001
From: Marty Pradere <martyp@labkey.com>
Date: Sun, 1 Mar 2026 09:16:50 -0800
Subject: [PATCH] Don't do the case statement when casting boolean value.

---
 query/src/org/labkey/query/sql/QuerySelect.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/query/src/org/labkey/query/sql/QuerySelect.java b/query/src/org/labkey/query/sql/QuerySelect.java
index c863c3ad90d..98adee324e7 100644
--- a/query/src/org/labkey/query/sql/QuerySelect.java
+++ b/query/src/org/labkey/query/sql/QuerySelect.java
@@ -2264,7 +2264,9 @@ SQLFragment getInternalSql()
             QExpr expr = getResolvedField();
 
             // NOTE SqlServer does not like predicates (A=B) in select list, try to help out
-            if (expr instanceof QMethodCall && expr.getJdbcType() == JdbcType.BOOLEAN && b.getDialect().isSqlServer())
+            // Exclude CAST/CONVERT expressions — they produce BIT values, not boolean predicates
+            if (expr instanceof QMethodCall mc && mc.getJdbcType() == JdbcType.BOOLEAN && b.getDialect().isSqlServer()
+                && !(mc.getMethod(b.getDialect()) instanceof Method.ConvertInfo))
             {
                 b.append("CASE WHEN (");
                 expr.appendSql(b, _query);