root
/
spring-framework
mirror of https://github.com/spring-projects/spring-framework.git
1
0
Fork 0
Code Issues Actions 8 Packages Projects Releases Wiki Activity

Revert "Auto-unwrap SessionFactoryBuilder-created proxies" 

This reverts commit 7903d6489f.

Issue: SPR-8492
This commit is contained in:
Chris Beams 2011-10-09 07:55:14 +00:00
parent d122ecb42e
commit 363e08df22
2 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
org.springframework.integration-tests/src/test/java/org/springframework/orm/hibernate3/HibernateSessionFactoryConfigurationTests.java
View File

@ -42,7 +42,6 @@ import org.springframework.context.annotation.AnnotationConfigApplicationContext
import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration; import org.springframework.context.annotation.Configuration;
import org.springframework.context.annotation.ImportResource; import org.springframework.context.annotation.ImportResource;
import org.springframework.core.InfrastructureProxy;
import org.springframework.dao.DataAccessException; import org.springframework.dao.DataAccessException;
import org.springframework.dao.annotation.PersistenceExceptionTranslationPostProcessor; import org.springframework.dao.annotation.PersistenceExceptionTranslationPostProcessor;
import org.springframework.dao.support.PersistenceExceptionTranslator; import org.springframework.dao.support.PersistenceExceptionTranslator;
@ -138,7 +137,6 @@ public class HibernateSessionFactoryConfigurationTests {
SessionFactory sessionFactory = ctx.getBean(SessionFactory.class); SessionFactory sessionFactory = ctx.getBean(SessionFactory.class);
assertThat(sessionFactory, instanceOf(DisposableBean.class)); assertThat(sessionFactory, instanceOf(DisposableBean.class));
assertThat(sessionFactory, instanceOf(SessionFactoryImplementor.class)); assertThat(sessionFactory, instanceOf(SessionFactoryImplementor.class));
assertThat(sessionFactory, instanceOf(InfrastructureProxy.class));
assertThat(sessionFactory.toString(), startsWith("DisposableBean proxy for SessionFactory")); assertThat(sessionFactory.toString(), startsWith("DisposableBean proxy for SessionFactory"));
ctx.close(); ctx.close();
assertTrue("SessionFactory was not closed as expected", sessionFactory.isClosed()); assertTrue("SessionFactory was not closed as expected", sessionFactory.isClosed());

5
org.springframework.orm/src/main/java/org/springframework/orm/hibernate3/SessionFactoryBuilderSupport.java
View File

@ -52,7 +52,6 @@ import org.hibernate.transaction.JTATransactionFactory;
import org.springframework.beans.BeanUtils; import org.springframework.beans.BeanUtils;
import org.springframework.beans.factory.DisposableBean; import org.springframework.beans.factory.DisposableBean;
import org.springframework.context.ConfigurableApplicationContext; import org.springframework.context.ConfigurableApplicationContext;
import org.springframework.core.InfrastructureProxy;
import org.springframework.core.io.ClassPathResource; import org.springframework.core.io.ClassPathResource;
import org.springframework.core.io.Resource; import org.springframework.core.io.Resource;
import org.springframework.dao.DataAccessException; import org.springframework.dao.DataAccessException;
@ -567,7 +566,6 @@ public abstract class SessionFactoryBuilderSupport<This extends SessionFactoryBu
new Class<?>[] { new Class<?>[] {
SessionFactory.class, SessionFactory.class,
SessionFactoryImplementor.class, SessionFactoryImplementor.class,
InfrastructureProxy.class,
DisposableBean.class DisposableBean.class
}, },
new InvocationHandler() { new InvocationHandler() {
@ -575,9 +573,6 @@ public abstract class SessionFactoryBuilderSupport<This extends SessionFactoryBu
if (ReflectionUtils.isToStringMethod(method)) { if (ReflectionUtils.isToStringMethod(method)) {
return String.format("DisposableBean proxy for SessionFactory [%s]", rawSf.toString()); return String.format("DisposableBean proxy for SessionFactory [%s]", rawSf.toString());
} }
if (method.equals(InfrastructureProxy.class.getMethod("getWrappedObject"))) {
return rawSf;
}
if (method.equals(DisposableBean.class.getMethod("destroy"))) { if (method.equals(DisposableBean.class.getMethod("destroy"))) {
closeHibernateSessionFactory(SessionFactoryBuilderSupport.this, rawSf); closeHibernateSessionFactory(SessionFactoryBuilderSupport.this, rawSf);
rawSf.close(); rawSf.close();