Support for spring-data 4.0.0

Author: Andreas Austing

core/pom.xml

@@ -5,20 +5,22 @@
   <parent>
     <groupId>com.cosium.spring.data</groupId>
     <artifactId>spring-data-jpa-entity-graph-parent</artifactId>
-    <version>3.5.1-SNAPSHOT</version>
+    <version>4.0.0-SNAPSHOT</version>
   </parent>
 
   <name>Spring Data JPA EntityGraph</name>
   <artifactId>spring-data-jpa-entity-graph</artifactId>
 
   <dependencies>
+
+    <!--Needed by SpringHibernateJpaPersistenceProvider for spring-data-jpa while building with JDK 11
     <dependency>
-      <!--Needed by SpringHibernateJpaPersistenceProvider for spring-data-jpa while building with JDK 11-->
       <groupId>javax.xml.bind</groupId>
       <artifactId>jaxb-api</artifactId>
       <version>2.4.0-b180830.0359</version>
       <scope>provided</scope>
     </dependency>
+    -->
 
     <dependency>
       <groupId>org.springframework.data</groupId>
@@ -27,13 +29,21 @@
     </dependency>
 
     <dependency>
-      <!--Needed by querydsl for @Generated annotation-->
+      <groupId>jakarta.persistence</groupId>
+      <artifactId>jakarta.persistence-api</artifactId>
+      <version>3.2.0</version>
+      <scope>provided</scope>
+    </dependency>
+
+    <!--Needed by querydsl for @Generated annotation-->
+    <dependency>
       <groupId>javax.annotation</groupId>
       <artifactId>javax.annotation-api</artifactId>
       <version>1.3.2</version>
       <scope>provided</scope>
     </dependency>
-
+    
+    <!--
     <dependency>
       <groupId>com.querydsl</groupId>
       <artifactId>querydsl-apt</artifactId>
@@ -43,7 +53,8 @@
 
     <dependency>
       <groupId>org.hibernate.orm</groupId>
-      <artifactId>hibernate-jpamodelgen</artifactId>
+      <artifactId>hibernate-processor</artifactId>
+      <version>${hibernate.version}</version>
       <scope>provided</scope>
     </dependency>
 
@@ -53,6 +64,7 @@
       <version>${project.version}</version>
       <scope>provided</scope>
     </dependency>
+    -->
 
     <dependency>
       <groupId>com.querydsl</groupId>
@@ -118,6 +130,34 @@
           </archive>
         </configuration>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-compiler-plugin</artifactId>
+        <configuration>
+          <annotationProcessorPaths>
+            <annotationProcessorPath>
+              <groupId>com.querydsl</groupId>
+              <artifactId>querydsl-apt</artifactId>
+              <version>${querydsl}</version>
+              <classifier>jakarta</classifier>
+            </annotationProcessorPath>
+            <annotationProcessorPath>
+              <groupId>jakarta.persistence</groupId>
+              <artifactId>jakarta.persistence-api</artifactId>
+            </annotationProcessorPath>
+            <path>
+              <groupId>org.hibernate.orm</groupId>
+              <artifactId>hibernate-processor</artifactId>
+              <version>${hibernate.version}</version>
+            </path>
+            <path>
+              <groupId>com.cosium.spring.data</groupId>
+              <artifactId>spring-data-jpa-entity-graph-generator</artifactId>
+              <version>3.5.0</version>
+            </path>
+          </annotationProcessorPaths>
+        </configuration>
+      </plugin>
     </plugins>
   </build>
 

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/EntityGraphJpaRepository.java

@@ -11,5 +11,6 @@
 @NoRepositoryBean
 public interface EntityGraphJpaRepository<T, ID>
     extends JpaRepository<T, ID>,
-        EntityGraphPagingAndSortingRepository<T, ID>,
+        EntityGraphListCrudRepository<T, ID>,
+        EntityGraphListPagingAndSortingRepository<T, ID>,
         EntityGraphQueryByExampleExecutor<T> {}

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/EntityGraphListCrudRepository.java

@@ -0,0 +1,24 @@
+package com.cosium.spring.data.jpa.entity.graph.repository;
+
+import com.cosium.spring.data.jpa.entity.graph.domain2.EntityGraph;
+import java.util.List;
+import org.springframework.data.repository.ListCrudRepository;
+import org.springframework.data.repository.NoRepositoryBean;
+
+/**
+ * @author Andreas Austing
+ */
+@NoRepositoryBean
+public interface EntityGraphListCrudRepository<T, ID>
+    extends ListCrudRepository<T, ID>, EntityGraphCrudRepository<T, ID> {
+
+  /**
+   * @see ListCrudRepository#findAllById(Iterable)
+   */
+  List<T> findAllById(Iterable<ID> ids, EntityGraph entityGraph);
+
+  /**
+   * @see ListCrudRepository#findAll()
+   */
+  List<T> findAll(EntityGraph entityGraph);
+}

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/EntityGraphListPagingAndSortingRepository.java

@@ -0,0 +1,20 @@
+package com.cosium.spring.data.jpa.entity.graph.repository;
+
+import com.cosium.spring.data.jpa.entity.graph.domain2.EntityGraph;
+import java.util.List;
+import org.springframework.data.domain.Sort;
+import org.springframework.data.repository.ListPagingAndSortingRepository;
+import org.springframework.data.repository.NoRepositoryBean;
+
+/**
+ * @author Andreas Austing
+ */
+@NoRepositoryBean
+public interface EntityGraphListPagingAndSortingRepository<T, ID>
+    extends ListPagingAndSortingRepository<T, ID>, EntityGraphPagingAndSortingRepository<T, ID> {
+
+  /**
+   * @see ListPagingAndSortingRepository#findAll(Sort)
+   */
+  List<T> findAll(Sort sort, EntityGraph entityGraph);
+}

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/EntityGraphPagingAndSortingRepository.java

@@ -14,7 +14,7 @@
  */
 @NoRepositoryBean
 public interface EntityGraphPagingAndSortingRepository<T, ID>
-    extends PagingAndSortingRepository<T, ID>, EntityGraphCrudRepository<T, ID> {
+    extends PagingAndSortingRepository<T, ID>, EntityGraphRepository<T, ID> {
 
   /**
    * @see PagingAndSortingRepository#findAll(Pageable)

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/support/EntityGraphJpaRepositoryFactory.java

@@ -5,6 +5,7 @@
 import com.cosium.spring.data.jpa.entity.graph.domain2.EntityGraph;
 import com.cosium.spring.data.jpa.entity.graph.repository.query.EntityGraphAwareJpaQueryMethodFactory;
 import jakarta.persistence.EntityManager;
+import org.jspecify.annotations.Nullable;
 import org.springframework.data.jpa.provider.PersistenceProvider;
 import org.springframework.data.jpa.repository.support.JpaRepositoryFactory;
 import org.springframework.data.querydsl.QuerydslPredicateExecutor;
@@ -33,9 +34,18 @@ public EntityGraphJpaRepositoryFactory(EntityManager entityManager) {
             PersistenceProvider.fromEntityManager(entityManager)));
   }
 
+  @Override
+  public void setRepositoryBaseClass(@Nullable Class<?> repositoryBaseClass) {
+    // ignore
+  }
+
   @Override
   protected Class<?> getRepositoryBaseClass(RepositoryMetadata metadata) {
-    if (isQueryDslExecutor(metadata.getRepositoryInterface())) {
+    return getRepositoryBaseClass(metadata.getRepositoryInterface());
+  }
+
+  protected Class<?> getRepositoryBaseClass(Class<?> repositoryInterface) {
+    if (isQueryDslExecutor(repositoryInterface)) {
       return EntityGraphQuerydslRepository.class;
     } else {
       return EntityGraphSimpleJpaRepository.class;

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/support/EntityGraphSimpleJpaRepository.java

@@ -90,12 +90,12 @@ public List<T> findAllById(Iterable<ID> ids, EntityGraph entityGraph) {
   }
 
   @Override
-  public Iterable<T> findAll(Sort sort, EntityGraph entityGraph) {
+  public List<T> findAll(Sort sort, EntityGraph entityGraph) {
     return findAll(sort);
   }
 
   @Override
-  public Iterable<T> findAll(EntityGraph entityGraph) {
+  public List<T> findAll(EntityGraph entityGraph) {
     return findAll();
   }
 }

core/src/main/java/com/cosium/spring/data/jpa/entity/graph/repository/support/RepositoryQueryEntityGraphInjector.java

@@ -23,7 +23,8 @@ class RepositoryQueryEntityGraphInjector implements MethodInterceptor {
       LoggerFactory.getLogger(RepositoryQueryEntityGraphInjector.class);
 
   private static final List<String> EXECUTE_QUERY_METHODS =
-      Arrays.asList("getResultList", "getSingleResult", "getResultStream", "scroll");
+      Arrays.asList(
+          "getResultList", "getSingleResult", "getSingleResultOrNull", "getResultStream", "scroll");
   private static final String UNWRAP_METHOD = "unwrap";
   private final EntityGraphQueryHintCandidate entityGraphCandidate;
 

core/src/test/java/com/cosium/spring/data/jpa/entity/graph/repository/DefaultEntityGraphTest.java

@@ -17,10 +17,10 @@
 import jakarta.persistence.ManyToOne;
 import jakarta.persistence.Table;
 import java.util.Optional;
-import javax.inject.Inject;
 import org.hibernate.Hibernate;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.transaction.annotation.Transactional;
 
 /**
@@ -29,7 +29,7 @@
 @DatabaseSetup(BaseTest.DATASET)
 class DefaultEntityGraphTest extends BaseTest {
 
-  @Inject private MyEntityRepository repository;
+  @Autowired private MyEntityRepository repository;
 
   @Transactional
   @Test
@@ -86,7 +86,7 @@ public void setMaker(Maker maker) {
     }
   }
 
-  interface MyEntityRepository extends EntityGraphCrudRepository<MyEntity, Long> {
+  interface MyEntityRepository extends EntityGraphListCrudRepository<MyEntity, Long> {
 
     @Override
     default Optional<EntityGraph> defaultEntityGraph() {

core/src/test/java/com/cosium/spring/data/jpa/entity/graph/repository/EntityGraphCustomJpaRepositoryTest.java

@@ -12,10 +12,10 @@
 import jakarta.persistence.PersistenceContext;
 import java.util.List;
 import java.util.Optional;
-import javax.inject.Inject;
 import org.hibernate.Hibernate;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.repository.Repository;
 import org.springframework.stereotype.Component;
 import org.springframework.transaction.annotation.Transactional;
@@ -28,7 +28,7 @@
 @DatabaseSetup(BaseTest.DATASET)
 class EntityGraphCustomJpaRepositoryTest extends BaseTest {
 
-  @Inject private EntityGraphCustomJpaRepositoryTestProductRepository productRepository;
+  @Autowired private EntityGraphCustomJpaRepositoryTestProductRepository productRepository;
 
   @Test
   @Transactional
@@ -68,8 +68,10 @@ public interface ProductRepositoryCustom {
   public static class EntityGraphCustomJpaRepositoryTestProductRepositoryImpl
       implements ProductRepositoryCustom {
 
-    @Inject private BrandRepository brandRepository;
-    @Inject private EntityGraphCustomJpaRepositoryTestProductRepository productRepository;
+    @Autowired private BrandRepository brandRepository;
+
+    @Autowired private EntityGraphCustomJpaRepositoryTestProductRepository productRepository;
+
     @PersistenceContext private EntityManager entityManager;
 
     @Override