[JBPM-10088] Sonar comments

kiegroup · Nov 22, 2023 · 5e2aed8 · 5e2aed8
1 parent 475164c
commit 5e2aed8
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 18 deletions.
diff --git a/...m-persistence-jpa/src/main/java/org/jbpm/persistence/timer/GlobalJpaTimerJobInstance.java b/...m-persistence-jpa/src/main/java/org/jbpm/persistence/timer/GlobalJpaTimerJobInstance.java
@@ -159,23 +159,18 @@ protected boolean hasEnvironmentEntry(Environment environment, String name, Obje
     }
 
     protected TransactionManager startTxIfNeeded(Environment environment) {
-
     	try {	
     	    boolean isTimerCMT = hasEnvironmentEntry(environment, "IS_TIMER_CMT", true);
     	    logger.debug ("Timer CMT value is {}", isTimerCMT);
-	    	if (!isTimerCMT) {
-	    	    if (environment.get(EnvironmentName.TRANSACTION_MANAGER) instanceof ContainerManagedTransactionManager) {
-	    	        TransactionManager tm = TransactionManagerFactory.get().newTransactionManager();
-
-	    	        if (tm.begin()) {    			
-	    	            return tm;
-	    	        }
-	    	    }
-	    	}	    	
+    	    if (!isTimerCMT) {
+    	        TransactionManager tm = TransactionManagerFactory.get().newTransactionManager();
+    	        if (tm.begin()) {
+    	            return tm;
+    	        }
+    	    }
     	} catch (Exception e) {
     		logger.debug("Unable to optionally start transaction due to {}", e.getMessage(), e);
     	}
-
     	return null;
     }
 

diff --git a/...untime-manager/src/main/java/org/jbpm/runtime/manager/impl/DefaultRuntimeEnvironment.java b/...untime-manager/src/main/java/org/jbpm/runtime/manager/impl/DefaultRuntimeEnvironment.java
@@ -65,7 +65,6 @@ public DefaultRuntimeEnvironment(EntityManagerFactory emf, GlobalSchedulerServic
         if (globalSchedulerService != null) {
             globalSchedulerService.setEnvironment(this);
         }
-
     }
 
     public DefaultRuntimeEnvironment(EntityManagerFactory emf, boolean usePersistence) {

diff --git a/.../jbpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EJBTimerScheduler.java b/.../jbpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EJBTimerScheduler.java
@@ -45,7 +45,6 @@
 import javax.ejb.TimerService;
 import javax.ejb.TransactionAttribute;
 import javax.ejb.TransactionAttributeType;
-import javax.naming.InitialContext;
 
 import org.drools.core.time.JobHandle;
 import org.drools.core.time.impl.TimerJobInstance;

diff --git a/...bpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EjbSchedulerService.java b/...bpm-services-ejb-timer/src/main/java/org/jbpm/services/ejb/timer/EjbSchedulerService.java
@@ -28,7 +28,6 @@
 import javax.naming.InitialContext;
 import javax.naming.NamingException;
 import javax.persistence.EntityManager;
-import javax.persistence.EntityManagerFactory;
 
 import org.drools.core.time.InternalSchedulerService;
 import org.drools.core.time.Job;
@@ -37,9 +36,6 @@
 import org.drools.core.time.TimerService;
 import org.drools.core.time.Trigger;
 import org.drools.core.time.impl.TimerJobInstance;
-import org.drools.persistence.api.TransactionManager;
-import org.drools.persistence.api.TransactionManagerFactory;
-import org.drools.persistence.jta.JtaTransactionManager;
 import org.jbpm.process.core.timer.GlobalSchedulerService;
 import org.jbpm.process.core.timer.JobNameHelper;
 import org.jbpm.process.core.timer.NamedJobContext;
@@ -50,7 +46,6 @@
 import org.jbpm.runtime.manager.impl.SimpleRuntimeEnvironment;
 import org.jbpm.runtime.manager.impl.jpa.EntityManagerFactoryManager;
 import org.jbpm.runtime.manager.impl.jpa.TimerMappingInfo;
-import org.kie.internal.runtime.manager.InternalRuntimeManager;
 import org.kie.internal.runtime.manager.RuntimeEnvironment;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;