From 5501a40dd20cd21c96a142979276b746cfd0f728 Mon Sep 17 00:00:00 2001
From: The Geeky Asian <msafeeransari@gmail.com>
Date: Sat, 8 Feb 2020 01:31:41 +0400
Subject: [PATCH] HHH-13855 - Remove unnecessary delared variable JtaManager in
 HibernatePersistenceProviderAdaptor

In the HibernatePersistenceProviderAdaptor, there is an unnecessary declaration of JtaManager. The sole purpose of having this variable is only to use in the overridden method named InjectJtaManager.

This was be improved by removing the JtaManager as there is no use of the variable, as of now at least.
---
 .../hibernate5/HibernatePersistenceProviderAdaptor.java   | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/hibernate-jipijapa/src/main/java/org/jboss/as/jpa/hibernate5/HibernatePersistenceProviderAdaptor.java b/hibernate-jipijapa/src/main/java/org/jboss/as/jpa/hibernate5/HibernatePersistenceProviderAdaptor.java
index d8a330d830..66e66dc727 100644
--- a/hibernate-jipijapa/src/main/java/org/jboss/as/jpa/hibernate5/HibernatePersistenceProviderAdaptor.java
+++ b/hibernate-jipijapa/src/main/java/org/jboss/as/jpa/hibernate5/HibernatePersistenceProviderAdaptor.java
@@ -43,17 +43,11 @@ public class HibernatePersistenceProviderAdaptor implements PersistenceProviderA
 	@SuppressWarnings("WeakerAccess")
 	public static final String HIBERNATE_EXTENDED_BEANMANAGER = "org.hibernate.resource.beans.container.spi.ExtendedBeanManager";
 
-	private volatile JtaManager jtaManager;
 	private volatile Platform platform;
 	private static final String NONE = SharedCacheMode.NONE.name();
 
 	@Override
-	public void injectJtaManager(JtaManager jtaManager) {
-		// todo : why?  `this.jtaManager` is never used aside from setting here in this method
-		if ( this.jtaManager != jtaManager ) {
-			this.jtaManager = jtaManager;
-		}
-	}
+	public void injectJtaManager(JtaManager jtaManager) { }
 
 	@Override
 	public void injectPlatform(Platform platform) {