WebSphereUowTransactionManager suspends synchronizations even without existing transaction (SPR-6167)

jhoeller · jhoeller · commit 90f8e5dcf850 · 2009-11-09T19:59:27.000Z
diff --git a/org.springframework.transaction/src/main/java/org/springframework/transaction/jta/WebSphereUowTransactionManager.java b/org.springframework.transaction/src/main/java/org/springframework/transaction/jta/WebSphereUowTransactionManager.java
@@ -268,7 +268,7 @@ else if (pb == TransactionDefinition.PROPAGATION_NOT_SUPPORTED) {
 		if (debug) {
 			logger.debug("Creating new transaction with name [" + definition.getName() + "]: " + definition);
 		}
-		SuspendedResourcesHolder suspendedResources = (existingTx && !joinTx ? suspend(null) : null);
+		SuspendedResourcesHolder suspendedResources = (!joinTx ? suspend(null) : null);
 		try {
 			if (definition.getTimeout() > TransactionDefinition.TIMEOUT_DEFAULT) {
 				this.uowManager.setUOWTimeout(uowType, definition.getTimeout());

Original file line number	Diff line number	Diff line change
`@@ -268,7 +268,7 @@ else if (pb == TransactionDefinition.PROPAGATION_NOT_SUPPORTED) {`
`268`	`268`	`if (debug) {`
`269`	`269`	`logger.debug("Creating new transaction with name [" + definition.getName() + "]: " + definition);`
`270`	`270`	`}`
`271`		`- SuspendedResourcesHolder suspendedResources = (existingTx && !joinTx ? suspend(null) : null);`
	`271`	`+ SuspendedResourcesHolder suspendedResources = (!joinTx ? suspend(null) : null);`
`272`	`272`	`try {`
`273`	`273`	`if (definition.getTimeout() > TransactionDefinition.TIMEOUT_DEFAULT) {`
`274`	`274`	`this.uowManager.setUOWTimeout(uowType, definition.getTimeout());`