From c738effd39f007447eb1683d01bed1b60862d81e Mon Sep 17 00:00:00 2001
From: Michael Hunger <github@jexp.de>
Date: Tue, 1 Mar 2011 13:43:32 +0100
Subject: [PATCH] fixed bug with stateholder holding onto the temporary state
 for too long

---
 .../support/AbstractConstructorEntityInstantiator.java   | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/spring-data-commons-core/src/main/java/org/springframework/persistence/support/AbstractConstructorEntityInstantiator.java b/spring-data-commons-core/src/main/java/org/springframework/persistence/support/AbstractConstructorEntityInstantiator.java
index 6747a900b..795d7c059 100644
--- a/spring-data-commons-core/src/main/java/org/springframework/persistence/support/AbstractConstructorEntityInstantiator.java
+++ b/spring-data-commons-core/src/main/java/org/springframework/persistence/support/AbstractConstructorEntityInstantiator.java
@@ -44,8 +44,13 @@ public abstract class AbstractConstructorEntityInstantiator<BACKING_INTERFACE, S
 		if (noArgConstructor != null) {
 			log.info("Using " + c + " no-arg constructor");
             StateProvider.setUnderlyingState(n);
-			T t = noArgConstructor.newInstance();
-			setState(t, n);
+            T t;
+            try {
+                t = noArgConstructor.newInstance();
+                setState(t, n);
+            } finally {
+            StateProvider.retrieveState();
+            }
 			return t;
 		}