Skip to content

Commit 53fca2c

Browse files
committed
upgraded to release version of spring boot 4
1 parent e8f9ed4 commit 53fca2c

File tree

13 files changed

+49
-58
lines changed

13 files changed

+49
-58
lines changed

infobip-spring-data-common/src/main/java/com/infobip/spring/data/common/PreferredConstructorDiscoverer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@
66
import org.springframework.core.DefaultParameterNameDiscoverer;
77
import org.springframework.core.ParameterNameDiscoverer;
88
import org.springframework.data.annotation.PersistenceCreator;
9+
import org.springframework.data.core.TypeInformation;
910
import org.springframework.data.mapping.*;
1011
import org.springframework.data.util.KotlinReflectionUtils;
11-
import org.springframework.data.util.TypeInformation;
1212
import org.springframework.lang.Nullable;
1313

1414
import java.lang.reflect.Constructor;

infobip-spring-data-jdbc-querydsl-boot-starter/src/main/java/com/infobip/spring/data/jdbc/QuerydslJdbcRepositoriesAutoConfiguration.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@
22

33
import com.infobip.spring.data.common.QuerydslSqlQueryConfiguration;
44
import org.springframework.boot.autoconfigure.AutoConfigureBefore;
5-
import org.springframework.boot.autoconfigure.data.jdbc.JdbcRepositoriesAutoConfiguration;
5+
import org.springframework.boot.data.jdbc.autoconfigure.DataJdbcRepositoriesAutoConfiguration;
66
import org.springframework.context.annotation.Configuration;
77
import org.springframework.context.annotation.Import;
88

9-
@AutoConfigureBefore(JdbcRepositoriesAutoConfiguration.class)
9+
@AutoConfigureBefore(DataJdbcRepositoriesAutoConfiguration.class)
1010
@Import({QuerydslSqlQueryConfiguration.class, QuerydslJdbcRepositoryConfigExtension.class, QuerydslJdbcRepositoriesRegistrar.class})
1111
@Configuration
1212
public class QuerydslJdbcRepositoriesAutoConfiguration {

infobip-spring-data-jdbc-querydsl/src/main/java/com/infobip/spring/data/jdbc/QuerydslJdbcRepositoryFactory.java

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ protected RepositoryComposition.RepositoryFragments getRepositoryFragments(Repos
7979
constructorExpression,
8080
path,
8181
querydslJdbcPredicateExecutor
82-
);
82+
);
8383
return fragments.append(implemented(simpleQuerydslJdbcFragment))
8484
.append(implemented(querydslJdbcPredicateExecutor));
8585
}
@@ -89,21 +89,21 @@ private QuerydslJdbcPredicateExecutor<?> createQuerydslJdbcPredicateExecutor(Rep
8989
RelationalPathBase<?> path) {
9090
var entity = context.getRequiredPersistentEntity(metadata.getDomainType());
9191
var querydsl = new Querydsl(sqlQueryFactory, entity);
92-
return getTargetRepositoryViaReflection(QuerydslJdbcPredicateExecutor.class,
93-
entity,
94-
converter,
95-
constructorExpression,
96-
path,
97-
querydsl);
92+
return instantiateClass(QuerydslJdbcPredicateExecutor.class,
93+
entity,
94+
converter,
95+
constructorExpression,
96+
path,
97+
querydsl);
9898
}
9999

100100
private SimpleQuerydslJdbcFragment<?> createSimpleQuerydslJdbcFragment(ConstructorExpression<?> constructor,
101101
RelationalPath<?> path,
102102
QuerydslPredicateExecutor<?> executor) {
103-
return getTargetRepositoryViaReflection(SimpleQuerydslJdbcFragment.class,
104-
executor,
105-
sqlQueryFactory,
106-
constructor,
107-
path);
103+
return instantiateClass(SimpleQuerydslJdbcFragment.class,
104+
executor,
105+
sqlQueryFactory,
106+
constructor,
107+
path);
108108
}
109109
}

infobip-spring-data-jdbc-querydsl/src/main/java/com/infobip/spring/data/jdbc/QuerydslJdbcRepositoryFactoryBean.java

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import org.springframework.beans.factory.annotation.Autowired;
2323
import org.springframework.context.ApplicationEventPublisher;
2424
import org.springframework.data.jdbc.core.convert.*;
25-
import org.springframework.data.jdbc.repository.QueryMappingConfiguration;
2625
import org.springframework.data.jdbc.repository.support.JdbcRepositoryFactoryBean;
2726
import org.springframework.data.mapping.callback.EntityCallbacks;
2827
import org.springframework.data.relational.core.dialect.Dialect;
@@ -42,7 +41,6 @@ public class QuerydslJdbcRepositoryFactoryBean<T extends Repository<S, ID>, S, I
4241
private DataAccessStrategy dataAccessStrategy;
4342
private QueryMappingConfiguration queryMappingConfiguration = QueryMappingConfiguration.EMPTY;
4443
private NamedParameterJdbcOperations operations;
45-
private EntityCallbacks entityCallbacks;
4644
private SQLQueryFactory sqlQueryFactory;
4745
private Dialect dialect;
4846

@@ -68,7 +66,6 @@ protected RepositoryFactorySupport doCreateRepositoryFactory() {
6866
operations,
6967
sqlQueryFactory);
7068
jdbcRepositoryFactory.setQueryMappingConfiguration(queryMappingConfiguration);
71-
jdbcRepositoryFactory.setEntityCallbacks(entityCallbacks);
7269

7370
return jdbcRepositoryFactory;
7471
}
@@ -134,6 +131,10 @@ public void afterPropertiesSet() {
134131
this.operations = beanFactory.getBean(NamedParameterJdbcOperations.class);
135132
}
136133

134+
if (this.queryMappingConfiguration == null) {
135+
this.queryMappingConfiguration = QueryMappingConfiguration.EMPTY;
136+
}
137+
137138
if (this.dataAccessStrategy == null) {
138139

139140
Assert.state(beanFactory != null, "If no DataAccessStrategy is set a BeanFactory must be available.");
@@ -148,18 +149,10 @@ public void afterPropertiesSet() {
148149
var sqlParametersFactory = new SqlParametersFactory(this.mappingContext, this.converter);
149150
var insertStrategyFactory = new InsertStrategyFactory(this.operations, this.dialect);
150151
return new DefaultDataAccessStrategy(sqlGeneratorSource, this.mappingContext, this.converter,
151-
this.operations, sqlParametersFactory, insertStrategyFactory);
152+
this.operations, sqlParametersFactory, insertStrategyFactory, queryMappingConfiguration);
152153
});
153154
}
154155

155-
if (this.queryMappingConfiguration == null) {
156-
this.queryMappingConfiguration = QueryMappingConfiguration.EMPTY;
157-
}
158-
159-
if (beanFactory != null) {
160-
entityCallbacks = EntityCallbacks.create(beanFactory);
161-
}
162-
163156
super.afterPropertiesSet();
164157
}
165158
}

infobip-spring-data-jdbc-querydsl/src/test/java/com/infobip/spring/data/jdbc/TestConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package com.infobip.spring.data.jdbc;
22

33
import org.flywaydb.core.Flyway;
4-
import org.springframework.boot.autoconfigure.flyway.FlywayMigrationStrategy;
4+
import org.springframework.boot.flyway.autoconfigure.FlywayMigrationStrategy;
55
import org.springframework.context.annotation.Bean;
66
import org.springframework.context.annotation.Configuration;
77

Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
package com.infobip.spring.data.jpa;
22

33
import org.springframework.boot.autoconfigure.AutoConfigureBefore;
4-
import org.springframework.boot.autoconfigure.data.jpa.JpaRepositoriesAutoConfiguration;
4+
import org.springframework.boot.data.jpa.autoconfigure.DataJpaRepositoriesAutoConfiguration;
55
import org.springframework.context.annotation.Configuration;
66
import org.springframework.context.annotation.Import;
77

88
@Import({ExtendedQuerydslJpaConfiguration.class, QuerydslJpaRepositoryConfigExtension.class, QuerydslJpaRepositoriesRegistrar.class})
9-
@AutoConfigureBefore(JpaRepositoriesAutoConfiguration.class)
9+
@AutoConfigureBefore(DataJpaRepositoriesAutoConfiguration.class)
1010
@Configuration
1111
public class QuerydslJpaRepositoriesAutoConfiguration {
1212
}

infobip-spring-data-jpa-querydsl-boot-starter/src/test/java/com/infobip/spring/data/jpa/TestConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package com.infobip.spring.data.jpa;
22

33
import org.flywaydb.core.Flyway;
4-
import org.springframework.boot.autoconfigure.flyway.FlywayMigrationStrategy;
4+
import org.springframework.boot.flyway.autoconfigure.FlywayMigrationStrategy;
55
import org.springframework.context.annotation.Bean;
66
import org.springframework.context.annotation.Configuration;
77

infobip-spring-data-jpa-querydsl/src/main/java/com/infobip/spring/data/jpa/ExtendedQuerydslJpaRepositoryFactory.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,11 @@ protected RepositoryComposition.RepositoryFragments getRepositoryFragments(Repos
3535
var fragments = super.getRepositoryFragments(metadata);
3636
JpaEntityInformation<?, Serializable> entityInformation = getEntityInformation(metadata.getDomainType());
3737
var path = entityPathResolver.createPath(entityInformation.getJavaType());
38-
var simpleJPAQuerydslFragment = getTargetRepositoryViaReflection(SimpleQuerydslJpaFragment.class,
39-
path,
40-
jpaQueryFactory,
41-
jpaSQLQueryFactory,
42-
entityManager);
38+
var simpleJPAQuerydslFragment = instantiateClass(SimpleQuerydslJpaFragment.class,
39+
path,
40+
jpaQueryFactory,
41+
jpaSQLQueryFactory,
42+
entityManager);
4343
var fragment = RepositoryFragment.implemented(simpleJPAQuerydslFragment);
4444
return fragments.append(fragment);
4545
}

infobip-spring-data-jpa-querydsl/src/main/java/com/infobip/spring/data/jpa/ExtendedQuerydslJpaRepositoryFactoryBean.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,6 @@ public void setEntityPathResolver(ObjectProvider<EntityPathResolver> resolver) {
5757
this.entityPathResolver = resolver.getIfAvailable(() -> SimpleEntityPathResolver.INSTANCE);
5858
}
5959

60-
@Autowired
61-
public void setQueryMethodFactory(@Nullable JpaQueryMethodFactory factory) {
62-
super.setQueryMethodFactory(factory);
63-
if (factory != null) {
64-
this.queryMethodFactory = factory;
65-
}
66-
}
67-
6860
@Autowired
6961
public void setJPAQueryFactory(JPAQueryFactory jpaQueryFactory) {
7062
this.jpaQueryFactory = jpaQueryFactory;

infobip-spring-data-jpa-querydsl/src/test/java/com/infobip/spring/data/jpa/TestConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package com.infobip.spring.data.jpa;
22

33
import org.flywaydb.core.Flyway;
4-
import org.springframework.boot.autoconfigure.flyway.FlywayMigrationStrategy;
4+
import org.springframework.boot.flyway.autoconfigure.FlywayMigrationStrategy;
55
import org.springframework.context.annotation.Bean;
66
import org.springframework.context.annotation.Configuration;
77

0 commit comments

Comments
 (0)