Spring/spring-data-commons
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DATAJPA-28 - Changed invocation order of createIsNewStrategy(…).

Browse Source 
Lazily initializing the IsNewAware on first access now as classes implementing the method might wanna delegate to a template method using a constructor parameter. If createIsNewStrategy(…) is called from our constructor already the subclasses additional constructor arguments are not available yet.
This commit is contained in:
Oliver Gierke
2011-02-22 21:42:49 +01:00
parent dd45054da2
commit 7753b9b446
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
spring-data-commons-core/src/main/java/org/springframework/data/repository/support/RepositorySupport.java
View File

@@ -33,7 +33,7 @@ public abstract class RepositorySupport<T, ID extends Serializable> implements
Repository<T, ID> {
private final Class<T> domainClass;
private final IsNewAware isNewStrategy;
private IsNewAware isNewStrategy;
/**
@@ -45,8 +45,6 @@ public abstract class RepositorySupport<T, ID extends Serializable> implements
Assert.notNull(domainClass);
this.domainClass = domainClass;
this.isNewStrategy = createIsNewStrategy(domainClass);
Assert.notNull(isNewStrategy);
}
@@ -81,6 +79,11 @@ public abstract class RepositorySupport<T, ID extends Serializable> implements
*/
protected IsNewAware getIsNewStrategy() {
if (isNewStrategy == null) {
this.isNewStrategy = createIsNewStrategy(domainClass);
Assert.notNull(isNewStrategy);
}
return isNewStrategy;
}
}