Correct support for propagation SUPPORTS and multiple data sources. Fixes #9149

Author: graemerocher

grails-core/src/main/groovy/org/grails/transaction/ChainedTransactionManager.java

@@ -47,6 +47,9 @@
  * 
  * @author Michael Hunger
  * @author Oliver Gierke
+ * @author Lari Hotari
+ * @author Graeme Rocher
+ *
  * @since  2.3.6
  */
 public class ChainedTransactionManager implements PlatformTransactionManager {
@@ -92,7 +95,7 @@ public MultiTransactionStatus getTransaction(TransactionDefinition definition) t
 
 		MultiTransactionStatus mts = new MultiTransactionStatus(transactionManagers.get(0));
 
-		if (!synchronizationManager.isSynchronizationActive()) {
+		if (!synchronizationManager.isSynchronizationActive() && canCreateTransaction(definition)) {
 			synchronizationManager.initSynchronization();
 			mts.setNewSynchonization();
 		}
@@ -127,6 +130,12 @@ public MultiTransactionStatus getTransaction(TransactionDefinition definition) t
 		return mts;
 	}
 
+	protected boolean canCreateTransaction(TransactionDefinition definition) {
+		return definition.getPropagationBehavior() == TransactionDefinition.PROPAGATION_REQUIRED ||
+				definition.getPropagationBehavior() == TransactionDefinition.PROPAGATION_REQUIRES_NEW ||
+				definition.getPropagationBehavior() == TransactionDefinition.PROPAGATION_NESTED;
+	}
+
 	/*
 	 * (non-Javadoc)
 	 * @see org.springframework.transaction.PlatformTransactionManager#commit(org.springframework.transaction.TransactionStatus)