From a62996f2bb622cf77c1da8257fb90de16609368e Mon Sep 17 00:00:00 2001
From: Mark Paluch <mark.paluch@broadcom.com>
Date: Mon, 14 Apr 2025 16:48:32 +0200
Subject: [PATCH] Allow HQL HAVING without enforcing GROUP BY.

Closes #3840
---
 .../springframework/data/jpa/repository/query/Hql.g4 |  4 ++--
 .../jpa/repository/query/EqlQueryRendererTests.java  |  8 ++++++++
 .../jpa/repository/query/HqlQueryRendererTests.java  | 12 ++++++++++++
 .../jpa/repository/query/JpqlQueryRendererTests.java |  8 ++++++++
 4 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/spring-data-jpa/src/main/antlr4/org/springframework/data/jpa/repository/query/Hql.g4 b/spring-data-jpa/src/main/antlr4/org/springframework/data/jpa/repository/query/Hql.g4
index da0518d08..d18a924a5 100644
--- a/spring-data-jpa/src/main/antlr4/org/springframework/data/jpa/repository/query/Hql.g4
+++ b/spring-data-jpa/src/main/antlr4/org/springframework/data/jpa/repository/query/Hql.g4
@@ -89,8 +89,8 @@ orderedQuery
     ;
 
 query
-    : selectClause fromClause? whereClause? (groupByClause havingClause?)? # SelectQuery
-    | fromClause whereClause? (groupByClause havingClause?)? selectClause? # FromQuery
+    : selectClause fromClause? whereClause? groupByClause? havingClause? # SelectQuery
+    | fromClause whereClause? groupByClause? havingClause? selectClause? # FromQuery
     ;
 
 queryOrder
diff --git a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/EqlQueryRendererTests.java b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/EqlQueryRendererTests.java
index 72bdfc3b1..f954bcc74 100644
--- a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/EqlQueryRendererTests.java
+++ b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/EqlQueryRendererTests.java
@@ -597,6 +597,14 @@ class EqlQueryRendererTests {
 				GROUP BY c.country
 				HAVING COUNT(c) > 30
 				""");
+
+		assertQuery("""
+				SELECT COUNT(f)
+				FROM FooEntity f
+				WHERE f.name IN ('Y', 'Basic', 'Remit')
+							AND f.size = 10
+				HAVING COUNT(f) > 0
+				""");
 	}
 
 	@Test
diff --git a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/HqlQueryRendererTests.java b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/HqlQueryRendererTests.java
index cdb81af4d..349b06467 100644
--- a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/HqlQueryRendererTests.java
+++ b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/HqlQueryRendererTests.java
@@ -791,6 +791,18 @@ class HqlQueryRendererTests {
 				""");
 	}
 
+	@Test
+	void shouldRenderHavingWithFunction() {
+
+		assertQuery("""
+				SELECT COUNT(f)
+				FROM FooEntity f
+				WHERE f.name IN ('Y', 'Basic', 'Remit')
+							AND f.size = 10
+				HAVING COUNT(f) > 0
+				""");
+	}
+
 	@Test
 	void theRest8() {
 
diff --git a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/JpqlQueryRendererTests.java b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/JpqlQueryRendererTests.java
index a16a5a880..33bef4a39 100644
--- a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/JpqlQueryRendererTests.java
+++ b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/query/JpqlQueryRendererTests.java
@@ -598,6 +598,14 @@ class JpqlQueryRendererTests {
 				GROUP BY c.country
 				HAVING COUNT(c) > 30
 				""");
+
+		assertQuery("""
+				SELECT COUNT(f)
+				FROM FooEntity f
+				WHERE f.name IN ('Y', 'Basic', 'Remit')
+							AND f.size = 10
+				HAVING COUNT(f) > 0
+				""");
 	}
 
 	@Test