Fix a bunch of issues encountered with ORM 6

Blazebit · Nov 9, 2024 · 0876fab · 0876fab
1 parent ad184b3
commit 0876fab
Show file tree

Hide file tree

Showing 16 changed files with 271 additions and 143 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,7 +10,9 @@ None yet
 
 ### Bug fixes
 
-None yet
+* Fix missing cascading deletes for deletes that contain joins or CTEs
+* Fix some issues with PostgreSQL and Oracle multiset SQL rendering with Hibernate ORM 6+
+* Fix returning clause rendering for PostgreSQL with Hibernate ORM 6+
 
 ### Backwards-incompatible changes
 

diff --git a/core/impl/src/main/java/com/blazebit/persistence/impl/ResolvingQueryGenerator.java b/core/impl/src/main/java/com/blazebit/persistence/impl/ResolvingQueryGenerator.java
@@ -950,7 +950,7 @@ private void renderEquality(Expression left, Expression right, boolean negated,
                 rewriteToIdParam(right);
             }
         } else {
-            if (expressionToSplit == null || dbmsDialect.supportsAnsiRowValueConstructor() || !(left instanceof ParameterExpression) && !(right instanceof ParameterExpression)) {
+            if (expressionToSplit == null || !(left instanceof ParameterExpression) && !(right instanceof ParameterExpression)) {
                 left.accept(this);
                 sb.append(operator);
                 if (quantifier != PredicateQuantifier.ONE) {

diff --git a/.../java/com/blazebit/persistence/impl/function/tostringjson/OracleToStringJsonFunction.java b/.../java/com/blazebit/persistence/impl/function/tostringjson/OracleToStringJsonFunction.java
@@ -38,8 +38,7 @@ public void render(FunctionRenderContext context, String[] fields, String[] sele
             context.addChunk(postChunk);
             context.addChunk(subquery.substring(fromIndex));
         } else {
-            int limitIndex = SqlUtils.indexOfLimit(subquery, orderByIndex);
-            if (limitIndex == -1) {
+            if (SqlUtils.indexOfLimit(subquery, orderByIndex) == -1 && SqlUtils.indexOfFetchFirst(subquery, orderByIndex) == -1) {
                 context.addChunk(preChunk);
 
                 StringBuilder sb = new StringBuilder(fromIndex);

diff --git a/...a/com/blazebit/persistence/impl/function/tostringjson/PostgreSQLToStringJsonFunction.java b/...a/com/blazebit/persistence/impl/function/tostringjson/PostgreSQLToStringJsonFunction.java
@@ -26,8 +26,7 @@ public void render(FunctionRenderContext context, String[] fields, String[] sele
             context.addChunk("))");
             context.addChunk(subquery.substring(fromIndex));
         } else {
-            int limitIndex = SqlUtils.indexOfLimit(subquery, orderByIndex);
-            if (limitIndex == -1) {
+            if (SqlUtils.indexOfLimit(subquery, orderByIndex) == -1 && SqlUtils.indexOfFetchFirst(subquery, orderByIndex) == -1) {
                 renderJsonObjectArguments(context, fields, selectItemExpressions);
                 context.addChunk("))");
                 context.addChunk(" OVER (");

diff --git a/...ava/com/blazebit/persistence/impl/function/tostringxml/PostgreSQLToStringXmlFunction.java b/...ava/com/blazebit/persistence/impl/function/tostringxml/PostgreSQLToStringXmlFunction.java
@@ -26,8 +26,7 @@ public void render(FunctionRenderContext context, String[] fields, String[] sele
             context.addChunk("))");
             context.addChunk(subquery.substring(fromIndex));
         } else {
-            int limitIndex = SqlUtils.indexOfLimit(subquery, orderByIndex);
-            if (limitIndex == -1) {
+            if (SqlUtils.indexOfLimit(subquery, orderByIndex) == -1 && SqlUtils.indexOfFetchFirst(subquery, orderByIndex) == -1) {
                 renderXmlElementArguments(context, fields, selectItemExpressions);
                 context.addChunk("))");
                 context.addChunk(" OVER (");

diff --git a/core/testsuite-hibernate/pom.xml b/core/testsuite-hibernate/pom.xml
@@ -13,7 +13,7 @@
 
     <properties>
         <module.name>com.blazebit.persistence.core.testsuite.hibernate</module.name>
-        <testBasePackage>com.blazebit.persistence.testsuite</testBasePackage>
+        <testBasePackage>com.blazebit.persistence.testsuite.hibernate</testBasePackage>
         <surefireForkCount>0</surefireForkCount>
     </properties>
 
@@ -634,5 +634,20 @@
                 </plugins>
             </build>
         </profile>
+        <profile>
+            <!-- No need to run this with EclipseLink -->
+            <id>eclipselink</id>
+            <build>
+                <plugins>
+                    <plugin>
+                        <artifactId>maven-compiler-plugin</artifactId>
+                        <configuration>
+                            <skip>true</skip>
+                            <skipMain>true</skipMain>
+                        </configuration>
+                    </plugin>
+                </plugins>
+            </build>
+        </profile>
     </profiles>
 </project>
diff --git a/...te/hibernate6/entity/IntegerProperty.java → ...ite/hibernate/entity/IntegerProperty.java b/...te/hibernate6/entity/IntegerProperty.java → ...ite/hibernate/entity/IntegerProperty.java
@@ -2,7 +2,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright Blazebit
  */
-package com.blazebit.persistence.testsuite.hibernate6.entity;
+package com.blazebit.persistence.testsuite.hibernate.entity;
 
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;

diff --git a/...testsuite/hibernate6/entity/Property.java → .../testsuite/hibernate/entity/Property.java b/...testsuite/hibernate6/entity/Property.java → .../testsuite/hibernate/entity/Property.java
@@ -2,7 +2,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright Blazebit
  */
-package com.blazebit.persistence.testsuite.hibernate6.entity;
+package com.blazebit.persistence.testsuite.hibernate.entity;
 
 public interface Property<T> {
 

diff --git a/...ite/hibernate6/entity/PropertyHolder.java → ...uite/hibernate/entity/PropertyHolder.java b/...ite/hibernate6/entity/PropertyHolder.java → ...uite/hibernate/entity/PropertyHolder.java
@@ -2,7 +2,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright Blazebit
  */
-package com.blazebit.persistence.testsuite.hibernate6.entity;
+package com.blazebit.persistence.testsuite.hibernate.entity;
 
 import jakarta.persistence.Column;
 import jakarta.persistence.DiscriminatorType;

diff --git a/...ite/hibernate6/entity/StringProperty.java → ...uite/hibernate/entity/StringProperty.java b/...ite/hibernate6/entity/StringProperty.java → ...uite/hibernate/entity/StringProperty.java
@@ -2,7 +2,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright Blazebit
  */
-package com.blazebit.persistence.testsuite.hibernate6.entity;
+package com.blazebit.persistence.testsuite.hibernate.entity;
 
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;

diff --git a/...-hibernate/src/test/java/com/blazebit/persistence/testsuite/hibernate/AnyMappingTest.java b/...-hibernate/src/test/java/com/blazebit/persistence/testsuite/hibernate/AnyMappingTest.java
@@ -6,7 +6,7 @@
 
 import com.blazebit.persistence.CriteriaBuilder;
 import com.blazebit.persistence.testsuite.AbstractCoreTest;
-import com.blazebit.persistence.testsuite.hibernate6.entity.PropertyHolder;
+import com.blazebit.persistence.testsuite.hibernate.entity.PropertyHolder;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.Assert.assertTrue;

diff --git a/...ersistence/view/testsuite/update/remove/cascade/AbstractEntityViewRemoveDocumentTest.java b/...ersistence/view/testsuite/update/remove/cascade/AbstractEntityViewRemoveDocumentTest.java
@@ -205,6 +205,9 @@ protected AssertStatementBuilder deleteDocumentOwned(AssertStatementBuilder buil
                     .assertDelete().forRelation(Document.class, "peopleListBag").and()
                     .assertDelete().forRelation(Document.class, "peopleCollectionBag").and();
         }
+        if (isQueryStrategy() && !simpleDelete && dbmsDialect.supportsModificationQueryInWithClause()) {
+            return builder;
+        }
         if (supportsNestedEmbeddables()) {
             builder
                     .assertDelete().forRelation(Document.class, "nameContainerMap").and()
@@ -228,7 +231,9 @@ protected AssertStatementBuilder deletePersonOwned(AssertStatementBuilder builde
                     .assertDelete().forRelation(Person.class, "favoriteDocuments").and()
             ;
         }
-
+        if (isQueryStrategy() && !simpleDelete && dbmsDialect.supportsModificationQueryInWithClause()) {
+            return builder;
+        }
         return builder
                 .assertDelete().forRelation(Person.class, "localized").and()
                 ;

diff --git a/...ate/subview/inverse/onetoone/entity/EntityViewUpdateSubviewInverseOneToOneEntityTest.java b/...ate/subview/inverse/onetoone/entity/EntityViewUpdateSubviewInverseOneToOneEntityTest.java
@@ -13,6 +13,7 @@
 
 import com.blazebit.persistence.testsuite.base.jpa.assertion.AssertStatementBuilder;
 import com.blazebit.persistence.testsuite.base.jpa.category.NoEclipselink;
+import com.blazebit.persistence.testsuite.base.jpa.category.NoHibernate;
 import com.blazebit.persistence.testsuite.entity.Document;
 import com.blazebit.persistence.testsuite.entity.DocumentForSimpleOneToOne;
 import com.blazebit.persistence.testsuite.entity.DocumentInfoSimple;
@@ -32,7 +33,7 @@
  */
 @RunWith(Parameterized.class)
 // NOTE: Hibernate has a bug when selecting inverse one-to-ones: https://hibernate.atlassian.net/browse/HHH-12885
-@Category({ NoEclipselink.class})
+@Category({ NoHibernate.class, NoEclipselink.class})
 public class EntityViewUpdateSubviewInverseOneToOneEntityTest extends AbstractEntityViewUpdateTest<UpdatableDocumentForOneToOneView> {
 
     private DocumentForSimpleOneToOne doc1;