Introduce transaction management

[gpl/argeo-slc.git] / org.argeo.slc.server / src / main / java / org / argeo / slc / ws / process / SlcExecutionStepsRequestEp.java

diff --git a/org.argeo.slc.server/src/main/java/org/argeo/slc/ws/process/SlcExecutionStepsRequestEp.java b/org.argeo.slc.server/src/main/java/org/argeo/slc/ws/process/SlcExecutionStepsRequestEp.java
index 36cf45a68f4dc3668d6dac3b35076e928a4b68d9..c15b4de758eb1a352bda27a985f4ea75a26af1c7 100644 (file)
--- a/org.argeo.slc.server/src/main/java/org/argeo/slc/ws/process/SlcExecutionStepsRequestEp.java
+++ b/org.argeo.slc.server/src/main/java/org/argeo/slc/ws/process/SlcExecutionStepsRequestEp.java
@@ -1,17 +1,11 @@
 package org.argeo.slc.ws.process;
 
-import java.util.List;
-
-import org.springframework.ws.server.endpoint.AbstractMarshallingPayloadEndpoint;
-
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
-
-import org.argeo.slc.core.SlcException;
-import org.argeo.slc.core.process.SlcExecution;
 import org.argeo.slc.core.process.SlcExecutionStep;
 import org.argeo.slc.dao.process.SlcExecutionDao;
 import org.argeo.slc.msg.process.SlcExecutionStepsRequest;
+import org.springframework.ws.server.endpoint.AbstractMarshallingPayloadEndpoint;
 
 public class SlcExecutionStepsRequestEp extends
                AbstractMarshallingPayloadEndpoint {
@@ -30,28 +24,21 @@ public class SlcExecutionStepsRequestEp extends
                try {
                        SlcExecutionStepsRequest msg = (SlcExecutionStepsRequest) requestObject;
                        uuid = msg.getSlcExecutionUuid();
-                       SlcExecution slcExecution = slcExecutionDao.getSlcExecution(uuid);
-
-                       if (slcExecution == null)
-                               throw new SlcException("Could not find slc execution " + uuid);
-
-                       List<SlcExecutionStep> additionalSteps = msg.getSteps();
-                       if(log.isTraceEnabled()){
-                               log.trace("Trying to add additional steps to slc execution "+uuid+":");
-                               for(SlcExecutionStep step: additionalSteps){
-                                       log.trace("Step "+step.getUuid()+" (in slc execution "+uuid+")");
+                       if (log.isTraceEnabled()) {
+                               log.trace("Trying to add additional steps to SLC execution #"
+                                               + uuid + ":");
+                               for (SlcExecutionStep step : msg.getSteps()) {
+                                       log.trace("Step " + step.getUuid() + " (in SLC execution #"
+                                                       + uuid + ")");
                                }
+                               log.trace("Adding " + msg.getSteps().size()
+                                               + " steps to SLC execution #" + uuid);
                        }
-                       slcExecution.getSteps().addAll(additionalSteps);
-
-                       log.debug("Adding " + msg.getSteps().size()
-                                       + " steps to SlcExecution with uuid "
-                                       + slcExecution.getUuid());
 
-                       slcExecutionDao.update(slcExecution);
+                       slcExecutionDao.addSteps(uuid, msg.getSteps());
                        return null;
                } catch (Exception e) {
-                       log.error("Could not update SlcExecution " + uuid
+                       log.error("Could not update SLC execution #" + uuid
                                        + " with additional steps", e);
                        throw e;
                }