From: miettinen <miettinen@ac1ea38d-2e2b-0410-8846-a27921b304fc>
Date: Fri, 8 Feb 2013 06:19:24 +0000 (+0000)
Subject: * The sorting of variables in shortcut tab lost the data field. Now the data field... 
X-Git-Tag: simantics-1.10.1~50
X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=commitdiff_plain;h=ec4b7aa568b105a1686d6236d769d4a0dd87eb2f;p=simantics%2Fsysdyn.git

* The sorting of variables in shortcut tab lost the data field. Now the data field is added in addition to the text field (refs #4048).
* Added polarity signs for dependencies whose one end is connected to a module, similarly to those that were connected to variables (refs #3940).
* Validating names of variables and module instances now checks also that there are no similarly named modules within the current model (or current module instance). Such invalid names are not accepted (#refs #3040).
* Validating names of variables and module instances now checks also that there are no similarly named module types defined in the model. Such invalid names are not accepted (#refs #680).

git-svn-id: https://www.simantics.org/svn/simantics/sysdyn/trunk@26737 ac1ea38d-2e2b-0410-8846-a27921b304fc
---

diff --git a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/ResourceSelectionProcessor.java b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/ResourceSelectionProcessor.java
index 5581db47..ab4c39b5 100644
--- a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/ResourceSelectionProcessor.java
+++ b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/ResourceSelectionProcessor.java
@@ -22,7 +22,6 @@ import org.simantics.browsing.ui.model.nodetypes.NodeType;
 import org.simantics.browsing.ui.model.nodetypes.SpecialNodeType;
 import org.simantics.db.ReadGraph;
 import org.simantics.db.Resource;
-import org.simantics.db.common.request.ObjectsWithType;
 import org.simantics.db.common.utils.ListUtils;
 import org.simantics.db.exception.DatabaseException;
 import org.simantics.db.exception.ManyObjectsForFunctionalRelationException;
@@ -31,21 +30,12 @@ import org.simantics.db.layer0.request.PossibleActiveVariableFromVariable;
 import org.simantics.db.layer0.variable.Variable;
 import org.simantics.diagram.stubs.DiagramResource;
 import org.simantics.jfreechart.ChartSelectionTabContributor;
-import org.simantics.jfreechart.chart.properties.ChartTab;
-import org.simantics.jfreechart.chart.properties.bar.BarAxisTab;
-import org.simantics.jfreechart.chart.properties.bar.BarGeneralPropertiesTab;
-import org.simantics.jfreechart.chart.properties.bar.BarSeriesTab;
-import org.simantics.jfreechart.chart.properties.pie.PieGeneralPropertiesTab;
-import org.simantics.jfreechart.chart.properties.pie.PieSeriesTab;
-import org.simantics.jfreechart.chart.properties.xyline.XYLineAxisAndVariablesTab;
-import org.simantics.jfreechart.chart.properties.xyline.XYLineGeneralPropertiesTab;
 import org.simantics.layer0.Layer0;
 import org.simantics.modeling.ModelingResources;
 import org.simantics.modeling.ui.property.svg.SVGElementComposite;
 import org.simantics.selectionview.ComparableTabContributor;
 import org.simantics.selectionview.SelectionProcessor;
 import org.simantics.simulation.ontology.SimulationResource;
-import org.simantics.sysdyn.JFreeChartResource;
 import org.simantics.sysdyn.SysdynResource;
 import org.simantics.sysdyn.ui.browser.nodes.SharedFunctionsFolder;
 import org.simantics.ui.selection.AnyVariable;
@@ -57,6 +47,7 @@ import org.simantics.utils.ui.ISelectionUtils;
  * SelectionProcessor for processing selections for property view in system dynamics
  * 
  * @author Teemu Lempinen
+ * @author Tuomas Miettinen
  *
  */
 public class ResourceSelectionProcessor implements SelectionProcessor<Object, ReadGraph>  {
@@ -342,14 +333,15 @@ public class ResourceSelectionProcessor implements SelectionProcessor<Object, Re
 
             // Dependency
             if (backend.isInstanceOf(r, sr.Dependency))
-                if (backend.hasStatement(r, sr.Dependency_refersTo))
+                if (backend.hasStatement(r, sr.Dependency_refersTo)) {
+                    Resource diaConnection = backend.getPossibleObject(r, ModelingResources.getInstance(backend).ConnectionToDiagramConnection);
                     return Collections.singleton(
                             new ComparableTabContributor(
-                                    new ReferenceDependencyTab(),
+                                    new DependencyTab(),
                                     0,
-                                    r,
+                                    diaConnection,
                                     "Reference Properties"));
-                else {
+                } else {
                     Resource diaConnection = backend.getPossibleObject(r, ModelingResources.getInstance(backend).ConnectionToDiagramConnection);
                     return Collections.singleton(
                             new ComparableTabContributor(
diff --git a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/widgets/ShortcutTabWidget.java b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/widgets/ShortcutTabWidget.java
index 6b7d4595..70879feb 100644
--- a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/widgets/ShortcutTabWidget.java
+++ b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/properties/widgets/ShortcutTabWidget.java
@@ -237,6 +237,7 @@ public class ShortcutTabWidget implements Widget {
 					                connectedVariables[i].dispose();
 					                TableItem item2 = new TableItem(variableTable, SWT.NONE, j);
 					                item2.setText(values);
+					                item2.setData(values);
 					                connectedVariables = variableTable.getItems();
 					                break;
 					            }
diff --git a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/VariableNameUtils.java b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/VariableNameUtils.java
index 344a0963..f3b37044 100644
--- a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/VariableNameUtils.java
+++ b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/VariableNameUtils.java
@@ -29,6 +29,9 @@ import org.simantics.sysdyn.SysdynResource;
 import org.simantics.sysdyn.manager.SysdynModel;
 import org.simantics.sysdyn.representation.Configuration;
 import org.simantics.sysdyn.representation.IElement;
+import org.simantics.sysdyn.representation.Model;
+import org.simantics.sysdyn.representation.Module;
+import org.simantics.sysdyn.representation.ModuleType;
 import org.simantics.sysdyn.representation.Variable;
 import org.simantics.ui.SimanticsUI;
 
@@ -110,6 +113,8 @@ public class VariableNameUtils {
 		Configuration configuration = model.getConfiguration();
 		if(configuration == null)
 		    return true;
+		
+		// Browse through variables and modules within the current module
 		IElement current = model.getElement(variable);
 		for(IElement e : configuration.getElements()) {
 			if(e instanceof Variable) {
@@ -117,8 +122,29 @@ public class VariableNameUtils {
 				if(!v.equals(current) && v.getName() != null && v.getName().equals(name)) {
 					return true;
 				}
-			}
+			} else if (e instanceof Module) {
+                Module m = (Module)e;
+                if(!m.equals(current) && m.getName() != null && m.getName().equals(name)) {
+                    return true;
+                }
+            }
 		}
+		
+		// Browse through all ModuleTypes within the whole model
+		Model rootModel = null;
+		ModuleType moduleType = configuration.getModuleType();
+		if (moduleType != null) {
+		    Object o = moduleType.getParent();
+		    if (o instanceof Model) {
+		        rootModel = (Model)o;
+		    }
+		} else {
+		    rootModel = configuration.getModel();
+		}
+		if (rootModel != null)
+		    return rootModel.containsModuleType(name);
+		
+		// We should not go here I think
 		return false;
 	}
 
diff --git a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/imports/SysdynImportFormatHandler.java b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/imports/SysdynImportFormatHandler.java
index 713041e2..f7712f23 100644
--- a/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/imports/SysdynImportFormatHandler.java
+++ b/org.simantics.sysdyn.ui/src/org/simantics/sysdyn/ui/utils/imports/SysdynImportFormatHandler.java
@@ -36,10 +36,10 @@ import org.simantics.sysdyn.ui.Activator;
  */
 public class SysdynImportFormatHandler implements FormatHandler<Object> {
     
-    private Resource parent;
-    private IProgressMonitor monitor;
-    private String migrationURI;
-    private DefaultPasteImportAdvisor importAdvisor;
+    private final Resource parent;
+    private final IProgressMonitor monitor;
+    private final String migrationURI;
+    private final DefaultPasteImportAdvisor importAdvisor;
     
     /**
      * Works 2 (2 * 1) in the monitor
@@ -87,7 +87,7 @@ public class SysdynImportFormatHandler implements FormatHandler<Object> {
             MigrationStep step = MigrationUtils.getStep(session, migrationURI);
             MigrationState state = MigrationUtils.newState();
             state.setProperty(MigrationStateKeys.CURRENT_TG, tg);
-            step.applyTo(session, state);
+            step.applyTo(monitor, session, state);
 //            tg = state.getProperty(MigrationStateKeys.CURRENT_TG);
 
             worked(1);
diff --git a/org.simantics.sysdyn/src/org/simantics/sysdyn/representation/Model.java b/org.simantics.sysdyn/src/org/simantics/sysdyn/representation/Model.java
index 6e22190f..bb4e3f66 100644
--- a/org.simantics.sysdyn/src/org/simantics/sysdyn/representation/Model.java
+++ b/org.simantics.sysdyn/src/org/simantics/sysdyn/representation/Model.java
@@ -26,16 +26,17 @@ import org.simantics.sysdyn.representation.annotations.BuiltinFunctions;
 /**
  * Representation of a system dynamics model
  * @author Teemu Lempinen
+ * @author Tuomas Miettinen
  *
  */
 @GraphType(SysdynResource.URIs.SysdynModel)
 public class Model {
 
     @RelatedValue(SysdynResource.URIs.SysdynModel_startTime)
-    private Double startTime = 0.0;
+    private final Double startTime = 0.0;
 
     @RelatedValue(SysdynResource.URIs.SysdynModel_stopTime)
-    private Double stopTime = 10.0;
+    private final Double stopTime = 10.0;
 
     @RelatedValue(SysdynResource.URIs.SysdynModel_outputInterval)
     private Double outputInterval;
@@ -58,12 +59,12 @@ public class Model {
     @RelatedElements(
             value = Layer0.URIs.ConsistsOf,
             composition = true)
-    private ArrayList<Object> consistsOf = new ArrayList<Object>();
+    private final ArrayList<Object> consistsOf = new ArrayList<Object>();
     
     @RelatedElements(
             value = Layer0.URIs.IsLinkedTo,
             composition = true)
-    private ArrayList<Object> linkedTo = new ArrayList<Object>();
+    private final ArrayList<Object> linkedTo = new ArrayList<Object>();
     
     
     @BuiltinFunctions
@@ -159,6 +160,24 @@ public class Model {
         return functions;
     }
     
+    /**
+     * Determine if this Model consists of a certain ModuleType.
+     * @param moduleType The name of the ModuleType that is sought 
+     * @return true iff this consists of moduleType
+     */
+    public boolean containsModuleType(String moduleType) {
+        if(consistsOf != null ) {
+            for(Object o : consistsOf) {
+                if(o instanceof ModuleType) {
+                    ModuleType mt = (ModuleType)o;
+                    if (mt.getName().equals(moduleType)) 
+                        return true;
+                }
+            }
+        }
+        return false;
+    }
+    
     /**
      * Add built-in functions to functions map. Built-in functions do not have any library path.
      * @param library Built-in function library