[WFLY-11381] Refactoring WeldTransactionServices to use non deprecate…

…d ServiceBuilder methods.
wildfly · Nov 20, 2018 · 64137ef · 64137ef
1 parent 220d1c9
commit 64137ef
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 20 deletions.
diff --git a/...java/org/jboss/as/weld/deployment/processor/TransactionsBootstrapDependencyInstaller.java b/...java/org/jboss/as/weld/deployment/processor/TransactionsBootstrapDependencyInstaller.java
@@ -29,21 +29,22 @@
 import org.jboss.msc.service.ServiceName;
 import org.jboss.msc.service.ServiceTarget;
 
+import java.util.function.Consumer;
+
 /**
- *
  * @author Martin Kouba
+ * @author <a href="mailto:ropalka@redhat.com">Richard Opalka</a>
  */
 public class TransactionsBootstrapDependencyInstaller implements BootstrapDependencyInstaller {
 
     @Override
     public ServiceName install(ServiceTarget serviceTarget, DeploymentUnit deploymentUnit, boolean jtsEnabled) {
-        final WeldTransactionServices weldTransactionServices = new WeldTransactionServices(jtsEnabled);
-
         final ServiceName weldTransactionServiceName = deploymentUnit.getServiceName().append(WeldTransactionServices.SERVICE_NAME);
-
-        final ServiceBuilder sb = serviceTarget.addService(weldTransactionServiceName, weldTransactionServices);
+        final ServiceBuilder<?> sb = serviceTarget.addService(weldTransactionServiceName);
+        final Consumer<WeldTransactionServices> weldTransactionServicesConsumer = sb.provides(weldTransactionServiceName);
         // Ensure the local transaction provider is started before we start
         sb.requires(ServiceNames.capabilityServiceName(deploymentUnit, "org.wildfly.transactions.global-default-local-provider"));
+        sb.setInstance(new WeldTransactionServices(jtsEnabled, weldTransactionServicesConsumer));
         sb.install();
 
         return weldTransactionServiceName;

diff --git a/...nsactions/src/main/java/org/jboss/as/weld/services/bootstrap/WeldTransactionServices.java b/...nsactions/src/main/java/org/jboss/as/weld/services/bootstrap/WeldTransactionServices.java
@@ -21,17 +21,18 @@
  */
 package org.jboss.as.weld.services.bootstrap;
 
+import java.util.function.Consumer;
+
 import javax.transaction.RollbackException;
 import javax.transaction.Status;
 import javax.transaction.Synchronization;
 import javax.transaction.SystemException;
 import javax.transaction.UserTransaction;
 
 import org.jboss.as.weld.ServiceNames;
-import org.jboss.msc.service.Service;
+import org.jboss.msc.Service;
 import org.jboss.msc.service.ServiceName;
 import org.jboss.msc.service.StartContext;
-import org.jboss.msc.service.StartException;
 import org.jboss.msc.service.StopContext;
 import org.jboss.weld.transaction.spi.TransactionServices;
 import org.wildfly.transaction.client.ContextTransactionManager;
@@ -44,16 +45,17 @@
  *
  * @author Stuart Douglas
  * @author <a href="mailto:tadamski@redhat.com">Tomasz Adamski</a>
- *
+ * @author <a href="mailto:ropalka@redhat.com">Richard Opalka</a>
  */
-public class WeldTransactionServices implements TransactionServices, Service<WeldTransactionServices> {
+public class WeldTransactionServices implements TransactionServices, Service {
 
     public static final ServiceName SERVICE_NAME = ServiceNames.WELD_TRANSACTION_SERVICES_SERVICE_NAME;
-
+    private final Consumer<WeldTransactionServices> weldTransactionServicesConsumer;
     private final boolean jtsEnabled;
 
-    public WeldTransactionServices(final boolean jtsEnabled) {
+    public WeldTransactionServices(final boolean jtsEnabled, final Consumer<WeldTransactionServices> weldTransactionServicesConsumer) {
         this.jtsEnabled = jtsEnabled;
+        this.weldTransactionServicesConsumer = weldTransactionServicesConsumer;
     }
 
     @Override
@@ -101,18 +103,13 @@ public void registerSynchronization(Synchronization synchronizedObserver) {
     }
 
     @Override
-    public void start(StartContext context) throws StartException {
-
-    }
-
-    @Override
-    public void stop(StopContext context) {
-
+    public void start(final StartContext context) {
+        weldTransactionServicesConsumer.accept(this);
     }
 
     @Override
-    public WeldTransactionServices getValue() throws IllegalStateException, IllegalArgumentException {
-        return this;
+    public void stop(final StopContext context) {
+        weldTransactionServicesConsumer.accept(null);
     }
 
 }