X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;ds=sidebyside;f=runtime%2Forg.argeo.slc.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fcore%2Fexecution%2FDefaultExecutionSpec.java;h=002d691ae1802e00dda3d7a49d1a29aedc4cc378;hb=d1298659fe6f179d1cbbc8c89f108a0bbc5b4edf;hp=b8535c572a2f72150fa10d6c62ffd9b888f10aa6;hpb=1fdb1b4e7b1d2b0cabb6483238301b857a6392fa;p=gpl%2Fargeo-slc.git

diff --git a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/DefaultExecutionSpec.java b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/DefaultExecutionSpec.java
index b8535c572..002d691ae 100644
--- a/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/DefaultExecutionSpec.java
+++ b/runtime/org.argeo.slc.core/src/main/java/org/argeo/slc/core/execution/DefaultExecutionSpec.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Mathieu Baudier <mbaudier@argeo.org>
+ * Copyright (C) 2007-2012 Argeo GmbH
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -13,14 +13,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.argeo.slc.core.execution;
 
+import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.UUID;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
@@ -35,16 +34,18 @@ import org.springframework.context.ApplicationContext;
 import org.springframework.context.ApplicationContextAware;
 import org.springframework.context.ConfigurableApplicationContext;
 
+/** Spring based implementation of execution specifications. */
 public class DefaultExecutionSpec implements ExecutionSpec, BeanNameAware,
-		ApplicationContextAware, InitializingBean {
+		ApplicationContextAware, InitializingBean, Serializable {
+	private static final long serialVersionUID = 5159882223926926539L;
 	private final static Log log = LogFactory
 			.getLog(DefaultExecutionSpec.class);
-	private ApplicationContext applicationContext;
+	private transient ApplicationContext applicationContext;
 
 	private String description;
 	private Map<String, ExecutionSpecAttribute> attributes = new HashMap<String, ExecutionSpecAttribute>();
 
-	private String name = getClass().getName() + "#" + UUID.randomUUID();
+	private String name = INTERNAL_NAME;
 
 	public Map<String, ExecutionSpecAttribute> getAttributes() {
 		return attributes;
@@ -58,6 +59,10 @@ public class DefaultExecutionSpec implements ExecutionSpec, BeanNameAware,
 		this.name = name;
 	}
 
+	/**
+	 * The Spring bean name (only relevant for specs declared has high-level
+	 * beans)
+	 */
 	public String getName() {
 		return name;
 	}
@@ -66,6 +71,10 @@ public class DefaultExecutionSpec implements ExecutionSpec, BeanNameAware,
 		return ((ExecutionSpec) obj).getName().equals(name);
 	}
 
+	/**
+	 * The Spring bean description (only relevant for specs declared has
+	 * high-level beans)
+	 */
 	public String getDescription() {
 		return description;
 	}
@@ -104,6 +113,10 @@ public class DefaultExecutionSpec implements ExecutionSpec, BeanNameAware,
 		}
 	}
 
+	/**
+	 * Generates a list of ref value choices based on the bean available in the
+	 * application ocntext.
+	 */
 	protected List<RefValueChoice> buildRefValueChoices(RefSpecAttribute rsa) {
 		List<RefValueChoice> choices = new ArrayList<RefValueChoice>();
 		if (applicationContext == null) {
@@ -114,6 +127,12 @@ public class DefaultExecutionSpec implements ExecutionSpec, BeanNameAware,
 
 		for (String beanName : getBeanFactory().getBeanNamesForType(
 				rsa.getTargetClass(), true, false)) {
+
+			// Since Spring 3, systemProperties is implicitly defined but has no
+			// bean definition
+			// if (beanName.equals("systemProperties"))
+			// continue beanNames;
+
 			BeanDefinition bd = getBeanFactory().getBeanDefinition(beanName);
 			RefValueChoice choice = new RefValueChoice();
 			choice.setName(beanName);