From 5ee18f4d019b61472c11dcb9e65e12cbff985bbd Mon Sep 17 00:00:00 2001
From: taojinlong <jinlong@fit2cloud.com>
Date: Wed, 4 Sep 2024 11:41:14 +0800
Subject: [PATCH] =?UTF-8?q?fix:=20=E4=BF=AE=E5=A4=8D=20sql=20=E8=A7=A3?=
 =?UTF-8?q?=E6=9E=90=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../java/io/dataease/commons/utils/SqlparserUtils.java     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/core/core-backend/src/main/java/io/dataease/commons/utils/SqlparserUtils.java b/core/core-backend/src/main/java/io/dataease/commons/utils/SqlparserUtils.java
index 877d535d92..74654e4784 100644
--- a/core/core-backend/src/main/java/io/dataease/commons/utils/SqlparserUtils.java
+++ b/core/core-backend/src/main/java/io/dataease/commons/utils/SqlparserUtils.java
@@ -177,8 +177,11 @@ public class SqlparserUtils {
         }
     }
 
-    private static String handleHaving(PlainSelect plainSelect) throws Exception {
+    private static void handleHaving(PlainSelect plainSelect) throws Exception {
         Expression expr = plainSelect.getHaving();
+        if (expr == null) {
+            return;
+        }
         StringBuilder stringBuilder = new StringBuilder();
         BinaryExpression binaryExpression = null;
         try {
@@ -192,12 +195,10 @@ public class SqlparserUtils {
             } else {
                 expr.accept(getExpressionDeParser(stringBuilder));
             }
-
         } else {
             expr.accept(getExpressionDeParser(stringBuilder));
         }
         plainSelect.setHaving(CCJSqlParserUtil.parseCondExpression(stringBuilder.toString()));
-        return plainSelect.toString();
     }
 
     private static String handleWhere(PlainSelect plainSelect, Select statementSelect, String dsType) throws Exception {