From f53795d48e705f68fcaebbb80f288d3a75c3a7d7 Mon Sep 17 00:00:00 2001
From: Oliver Gierke <ogierke@pivotal.io>
Date: Tue, 26 Jul 2016 09:46:15 +0200
Subject: [PATCH] =?UTF-8?q?DATACMNS-880=20-=20TransactionalRepositoryFacto?=
 =?UTF-8?q?ryBeanSupport.setBeanFactory(=E2=80=A6)=20now=20delegates=20to?=
 =?UTF-8?q?=20parent=20implementation.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Previously TransactionalRepositoryFactoryBeanSupport.setBeanFactory(…) didn't invoke the method of the super class preventing the BeanFactory to be propagated to the repository factory resulting in the ProjectionFactory implementation created being unaware of the BeanFactory and thus no bean references being available in projection interfaces.
---
 ...sactionalRepositoryFactoryBeanSupport.java |  4 +++-
 ...RepositoryFactoryBeanSupportUnitTests.java | 20 ++++++++++++++++---
 2 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/springframework/data/repository/core/support/TransactionalRepositoryFactoryBeanSupport.java b/src/main/java/org/springframework/data/repository/core/support/TransactionalRepositoryFactoryBeanSupport.java
index 2f5f47b4e..9d98d6ff4 100644
--- a/src/main/java/org/springframework/data/repository/core/support/TransactionalRepositoryFactoryBeanSupport.java
+++ b/src/main/java/org/springframework/data/repository/core/support/TransactionalRepositoryFactoryBeanSupport.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2008-2015 the original author or authors.
+ * Copyright 2008-2016 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -91,6 +91,8 @@ public abstract class TransactionalRepositoryFactoryBeanSupport<T extends Reposi
 
 		Assert.isInstanceOf(ListableBeanFactory.class, beanFactory);
 
+		super.setBeanFactory(beanFactory);
+
 		ListableBeanFactory listableBeanFactory = (ListableBeanFactory) beanFactory;
 		this.txPostProcessor = new TransactionalRepositoryProxyPostProcessor(listableBeanFactory, transactionManagerName,
 				enableDefaultTransactions);
diff --git a/src/test/java/org/springframework/data/repository/core/support/TransactionRepositoryFactoryBeanSupportUnitTests.java b/src/test/java/org/springframework/data/repository/core/support/TransactionRepositoryFactoryBeanSupportUnitTests.java
index 8c6251496..13d659968 100644
--- a/src/test/java/org/springframework/data/repository/core/support/TransactionRepositoryFactoryBeanSupportUnitTests.java
+++ b/src/test/java/org/springframework/data/repository/core/support/TransactionRepositoryFactoryBeanSupportUnitTests.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2015 the original author or authors.
+ * Copyright 2015-2016 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -25,6 +25,7 @@ import org.springframework.aop.Advisor;
 import org.springframework.aop.framework.Advised;
 import org.springframework.beans.factory.support.DefaultListableBeanFactory;
 import org.springframework.data.repository.CrudRepository;
+import org.springframework.test.util.ReflectionTestUtils;
 import org.springframework.transaction.interceptor.TransactionInterceptor;
 
 /**
@@ -67,9 +68,22 @@ public class TransactionRepositoryFactoryBeanSupportUnitTests {
 		assertThat(found, is(true));
 	}
 
+	/**
+	 * @see DATACMNS-880
+	 */
+	@Test
+	public void propagatesBeanFactoryToSuperClass() {
+
+		SampleTransactionalRepositoryFactoryBean factoryBean = new SampleTransactionalRepositoryFactoryBean();
+		factoryBean.setBeanFactory(new DefaultListableBeanFactory());
+
+		assertThat(ReflectionTestUtils.getField(factoryBean, RepositoryFactoryBeanSupport.class, "beanFactory"),
+				is(notNullValue()));
+	}
+
 	@SuppressWarnings({ "unchecked", "rawtypes" })
-	static class SampleTransactionalRepositoryFactoryBean extends
-			TransactionalRepositoryFactoryBeanSupport<CrudRepository<Object, Long>, Object, Long> {
+	static class SampleTransactionalRepositoryFactoryBean
+			extends TransactionalRepositoryFactoryBeanSupport<CrudRepository<Object, Long>, Object, Long> {
 
 		private final CrudRepository<Object, Long> repository = mock(CrudRepository.class);