From 6258d05d58c728fb69bd9e9bbaf9ded2ab512cb8 Mon Sep 17 00:00:00 2001
From: Oliver Gierke <info@olivergierke.de>
Date: Fri, 30 Sep 2011 12:11:36 +0200
Subject: [PATCH] DATACMNS-82 - Removed transaction manager bean configuration.

Always setting a transaction manager breaks repository factories that do not expose a transactionManager property. Thus it's now the responsibility of the concrete BeanDefinitionParser to parse transaction manager configuration.
---
 ...bstractRepositoryConfigDefinitionParser.java | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/repository/config/AbstractRepositoryConfigDefinitionParser.java b/spring-data-commons-core/src/main/java/org/springframework/data/repository/config/AbstractRepositoryConfigDefinitionParser.java
index 8db8d5e1c9..e1c7d60735 100644
--- a/spring-data-commons-core/src/main/java/org/springframework/data/repository/config/AbstractRepositoryConfigDefinitionParser.java
+++ b/spring-data-commons-core/src/main/java/org/springframework/data/repository/config/AbstractRepositoryConfigDefinitionParser.java
@@ -46,7 +46,6 @@
 import org.springframework.core.type.filter.RegexPatternTypeFilter;
 import org.springframework.data.repository.NoRepositoryBean;
 import org.springframework.data.repository.RepositoryDefinition;
-import org.springframework.util.StringUtils;
 import org.w3c.dom.Element;
 
 /**
@@ -61,16 +60,12 @@ public abstract class AbstractRepositoryConfigDefinitionParser<S extends GlobalR
 
 	private static final Log LOG = LogFactory.getLog(AbstractRepositoryConfigDefinitionParser.class);
 
-	private static final String REPOSITORY_INTERFACE_POST_PROCESSOR = "org.springframework.data.repository.core.support.RepositoryInterfaceAwareBeanPostProcessor";
-	private static final String DEFAULT_TRANSACTION_MANAGER_BEAN_NAME = "transactionManager"; 
+	private static final String REPOSITORY_INTERFACE_POST_PROCESSOR = "org.springframework.data.repository.core.support.RepositoryInterfaceAwareBeanPostProcessor"; 
 
 	/*
-			 * (non-Javadoc)
-			 *
-			 * @see
-			 * org.springframework.beans.factory.xml.BeanDefinitionParser#parse(org.
-			 * w3c.dom.Element, org.springframework.beans.factory.xml.ParserContext)
-			 */
+	 * (non-Javadoc)
+	 * @see org.springframework.beans.factory.xml.BeanDefinitionParser#parse(org.w3c.dom.Element, org.springframework.beans.factory.xml.ParserContext)
+	 */
 	public BeanDefinition parse(Element element, ParserContext parser) {
 
 		try {
@@ -181,10 +176,6 @@ private void registerGenericRepositoryFactoryBean(ParserContext parser, T contex
 			builder.addPropertyValue("namedQueries",
 					new NamedQueriesBeanDefinitionParser(context.getNamedQueriesLocation()).parse(context.getSource(), parser));
 
-			String transactionManagerRef = StringUtils.hasText(context.getTransactionManagerRef()) ? context
-					.getTransactionManagerRef() : DEFAULT_TRANSACTION_MANAGER_BEAN_NAME;
-			builder.addPropertyValue("transactionManager", transactionManagerRef);
-
 			String customImplementationBeanName = registerCustomImplementation(context, parser, beanSource);
 
 			if (customImplementationBeanName != null) {