From 90ab2f741ccf85208c990ac363dbf0c93a17cc22 Mon Sep 17 00:00:00 2001 From: Jens Schauder Date: Wed, 1 Jun 2022 11:38:49 +0200 Subject: [PATCH] Polishing. See #1254 --- .../data/jdbc/core/JdbcAggregateTemplate.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/JdbcAggregateTemplate.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/JdbcAggregateTemplate.java index 91a85875..c9761397 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/JdbcAggregateTemplate.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/JdbcAggregateTemplate.java @@ -131,7 +131,9 @@ public class JdbcAggregateTemplate implements JdbcAggregateOperations { } /** - * @param entityCallbacks + * Sets the callbacks to be invoked on life cycle events. + * + * @param entityCallbacks must not be {@literal null}. * @since 1.1 */ public void setEntityCallbacks(EntityCallbacks entityCallbacks) { @@ -216,10 +218,10 @@ public class JdbcAggregateTemplate implements JdbcAggregateOperations { Assert.notNull(domainType, "Domain type must not be null!"); T entity = accessStrategy.findById(id, domainType); - if (entity != null) { - return triggerAfterConvert(entity); + if (entity == null) { + return null; } - return entity; + return triggerAfterConvert(entity); } /*