Skip to content

Commit 398bf84

Browse files
committed
fix GroovyScriptRunningContext
1 parent 161143f commit 398bf84

2 files changed

Lines changed: 14 additions & 13 deletions

File tree

springboot-starter/src/main/java/com/codingapi/springboot/framework/transaction/TransactionManagerContext.java

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -23,41 +23,41 @@ private TransactionManagerContext() {
2323

2424
public void setPlatformTransactionManager(PlatformTransactionManager platformTransactionManager) {
2525
this.platformTransactionManager = platformTransactionManager;
26-
log.info("platformTransactionManager:{} load success", platformTransactionManager);
26+
if (platformTransactionManager != null) {
27+
log.info("platformTransactionManager:{} load success", platformTransactionManager);
28+
}
2729
}
2830

2931

3032
public <T> T commit(Supplier<T> supplier) {
31-
PlatformTransactionManager transactionManager = TransactionManagerContext.getInstance().getPlatformTransactionManager();
32-
if (transactionManager != null) {
33+
if (platformTransactionManager != null) {
3334
DefaultTransactionDefinition def = new DefaultTransactionDefinition();
3435
def.setPropagationBehavior(TransactionDefinition.PROPAGATION_REQUIRES_NEW);
35-
TransactionStatus transactionStatus = transactionManager.getTransaction(def);
36+
TransactionStatus transactionStatus = platformTransactionManager.getTransaction(def);
3637
try {
3738
T result = supplier.get();
38-
transactionManager.commit(transactionStatus);
39+
platformTransactionManager.commit(transactionStatus);
3940
return result;
4041
} catch (Exception e) {
41-
transactionManager.rollback(transactionStatus);
42+
platformTransactionManager.rollback(transactionStatus);
4243
throw e;
4344
}
4445
}
4546
return supplier.get();
4647
}
4748

48-
public <T> T readOnly(Supplier<T> supplier){
49-
PlatformTransactionManager transactionManager = TransactionManagerContext.getInstance().getPlatformTransactionManager();
50-
if (transactionManager != null) {
49+
public <T> T readOnly(Supplier<T> supplier) {
50+
if (platformTransactionManager != null) {
5151
DefaultTransactionDefinition def = new DefaultTransactionDefinition();
5252
def.setPropagationBehavior(TransactionDefinition.PROPAGATION_REQUIRES_NEW);
5353
def.setReadOnly(true);
54-
TransactionStatus transactionStatus = transactionManager.getTransaction(def);
54+
TransactionStatus transactionStatus = platformTransactionManager.getTransaction(def);
5555
try {
5656
T result = supplier.get();
57-
transactionManager.rollback(transactionStatus);
57+
platformTransactionManager.rollback(transactionStatus);
5858
return result;
5959
} catch (Exception e) {
60-
transactionManager.rollback(transactionStatus);
60+
platformTransactionManager.rollback(transactionStatus);
6161
throw e;
6262
}
6363
}

springboot-starter/src/main/java/com/codingapi/springboot/framework/transaction/TransactionManagerContextConfiguration.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package com.codingapi.springboot.framework.transaction;
22

3+
import org.springframework.beans.factory.annotation.Autowired;
34
import org.springframework.context.annotation.Bean;
45
import org.springframework.context.annotation.Configuration;
56
import org.springframework.transaction.PlatformTransactionManager;
@@ -8,7 +9,7 @@
89
public class TransactionManagerContextConfiguration {
910

1011
@Bean
11-
public TransactionManagerContextRegister transactionManagerContextRegister(PlatformTransactionManager platformTransactionManager){
12+
public TransactionManagerContextRegister transactionManagerContextRegister(@Autowired(required = false) PlatformTransactionManager platformTransactionManager){
1213
return new TransactionManagerContextRegister(platformTransactionManager);
1314
}
1415

0 commit comments

Comments
 (0)