Merge "Log an error if there are two resources with the same GUID."

[simantics/platform.git] / bundles / org.simantics.structural2 / src / org / simantics / structural2 / StructuralRVIResolver.java

diff --git a/bundles/org.simantics.structural2/src/org/simantics/structural2/StructuralRVIResolver.java b/bundles/org.simantics.structural2/src/org/simantics/structural2/StructuralRVIResolver.java
index 672edfca65c4bfc6995654fdd6b935e4cdb886e4..ec6b583617fcdd6c42af134845d829838944980a 100644 (file)
--- a/bundles/org.simantics.structural2/src/org/simantics/structural2/StructuralRVIResolver.java
+++ b/bundles/org.simantics.structural2/src/org/simantics/structural2/StructuralRVIResolver.java
@@ -8,6 +8,7 @@ import org.simantics.databoard.binding.Binding;
 import org.simantics.datatypes.literal.GUID;
 import org.simantics.db.ReadGraph;
 import org.simantics.db.Resource;
+import org.simantics.db.common.utils.NameUtils;
 import org.simantics.db.exception.DatabaseException;
 import org.simantics.db.layer0.adapter.Instances;
 import org.simantics.db.layer0.exception.MissingVariableException;
@@ -21,8 +22,11 @@ import org.simantics.db.layer0.variable.Variables;
 import org.simantics.layer0.Layer0;
 import org.simantics.simulation.ontology.SimulationResource;
 import org.simantics.structural.stubs.StructuralResource2;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class StructuralRVIResolver extends StandardRVIResolver {
+    private static final Logger LOGGER = LoggerFactory.getLogger(StructuralRVIResolver.class);
 
     protected boolean isRVIBase(ReadGraph graph, Variable variable) throws DatabaseException {
         if (Variables.isContext(graph, variable)) return true;
@@ -118,7 +122,21 @@ public class StructuralRVIResolver extends StandardRVIResolver {
                Instances instances = graph.adapt(Layer0.getInstance(graph).Entity, Instances.class);
                GUID guid = new GUID(part.mostSignificant, part.leastSignificant);
                Collection<Resource> queryResult = instances.find(graph, indexRoot, "GUID:"+guid.indexString());
-               if(queryResult.size() != 1) return null;
+               if(queryResult.size() != 1) {
+                   if(queryResult.size() > 1) {
+                       StringBuilder b = new StringBuilder();
+                       boolean first = true;
+                       for(Resource r : queryResult) {
+                           if(first)
+                               first = false;
+                           else
+                               b.append(", ");
+                           b.append(NameUtils.getURIOrSafeNameInternal(graph, r));
+                       }
+                       LOGGER.error("Found {} resources with the same GUID {}: {}.", queryResult.size(), guid, b);
+                   }
+                   return null;
+               }
                return getRVIPath(graph, variable, queryResult.iterator().next());
        }