X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=eclipse%2Fplugins%2Forg.argeo.slc.client.ui%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fclient%2Fui%2Feditors%2FProcessBuilderPage.java;h=f5e8708aed17483f9a984a1d73bb9ebefca3b317;hb=da80480f09f8efc5e0278ed992e17e586b0a0e62;hp=6b6fed542111422db7d9b34db36c327b5e374f6b;hpb=86c47402780f41526382267ff1597d2f3a0d0dd7;p=gpl%2Fargeo-slc.git

diff --git a/eclipse/plugins/org.argeo.slc.client.ui/src/main/java/org/argeo/slc/client/ui/editors/ProcessBuilderPage.java b/eclipse/plugins/org.argeo.slc.client.ui/src/main/java/org/argeo/slc/client/ui/editors/ProcessBuilderPage.java
index 6b6fed542..f5e8708ae 100644
--- a/eclipse/plugins/org.argeo.slc.client.ui/src/main/java/org/argeo/slc/client/ui/editors/ProcessBuilderPage.java
+++ b/eclipse/plugins/org.argeo.slc.client.ui/src/main/java/org/argeo/slc/client/ui/editors/ProcessBuilderPage.java
@@ -3,6 +3,8 @@ package org.argeo.slc.client.ui.editors;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
+import java.util.SortedSet;
+import java.util.TreeSet;
 import java.util.UUID;
 
 import javax.jcr.Node;
@@ -18,8 +20,6 @@ import javax.jcr.observation.ObservationManager;
 import javax.jcr.query.Query;
 import javax.jcr.query.QueryManager;
 
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
 import org.argeo.ArgeoException;
 import org.argeo.eclipse.ui.jcr.AsyncUiEventListener;
 import org.argeo.jcr.JcrUtils;
@@ -74,7 +74,8 @@ import org.eclipse.ui.forms.widgets.ScrolledForm;
 
 public class ProcessBuilderPage extends FormPage implements SlcNames {
 	public final static String ID = "processBuilderPage";
-	private final static Log log = LogFactory.getLog(ProcessBuilderPage.class);
+	// private final static Log log =
+	// LogFactory.getLog(ProcessBuilderPage.class);
 
 	private Node processNode;
 
@@ -113,7 +114,7 @@ public class ProcessBuilderPage extends FormPage implements SlcNames {
 			getManagedForm().addPart(formPart);
 
 			// observation
-			statusObserver = new AsyncUiEventListener() {
+			statusObserver = new AsyncUiEventListener(form.getDisplay()) {
 				protected void onEventInUiThread(EventIterator events) {
 					statusChanged();
 				}
@@ -372,6 +373,7 @@ public class ProcessBuilderPage extends FormPage implements SlcNames {
 			specAttrs: for (NodeIterator nit = specAttrsBase.getNodes(); nit
 					.hasNext();) {
 				Node specAttrNode = nit.nextNode();
+				String attrName = specAttrNode.getName();
 				if (!specAttrNode
 						.isNodeType(SlcTypes.SLC_EXECUTION_SPEC_ATTRIBUTE))
 					continue specAttrs;
@@ -379,13 +381,13 @@ public class ProcessBuilderPage extends FormPage implements SlcNames {
 						.getName());
 				JcrUtils.copy(specAttrNode, realizedAttrNode);
 
-				// for (PropertyIterator pit = realizedAttrNode.getProperties();
-				// pit
-				// .hasNext();) {
-				// Property p = pit.nextProperty();
-				// if (!p.isMultiple())
-				// log.debug(p.getName() + "=" + p.getValue().getString());
-				// }
+				// ovveride with flow value
+				if (flowNode.hasNode(attrName)) {
+					// assuming this is a primitive
+					realizedAttrNode.setProperty(SLC_VALUE,
+							flowNode.getNode(attrName).getProperty(SLC_VALUE)
+									.getValue());
+				}
 			}
 
 			flowsViewer.refresh();
@@ -595,11 +597,18 @@ public class ProcessBuilderPage extends FormPage implements SlcNames {
 						+ SlcTypes.SLC_EXECUTION_FLOW
 						+ "] WHERE ISDESCENDANTNODE(['" + path
 						+ "']) OR ISSAMENODE(['" + path + "'])";
-				//log.debug(statement);
+				// log.debug(statement);
 				Query query = qm.createQuery(statement, Query.JCR_SQL2);
+
+				// order paths
+				SortedSet<String> paths = new TreeSet<String>();
 				for (NodeIterator nit = query.execute().getNodes(); nit
 						.hasNext();) {
-					addFlow(nit.nextNode().getPath());
+					paths.add(nit.nextNode().getPath());
+				}
+
+				for (String p : paths) {
+					addFlow(p);
 				}
 				return true;
 			} catch (RepositoryException e) {