From: Hannu Niemistö <hannu.niemisto@semantum.fi>
Date: Wed, 17 Oct 2018 11:57:58 +0000 (+0000)
Subject: Merge "Log some information about missing property in SCLLabelRule"
X-Git-Tag: v1.43.0~136^2~326
X-Git-Url: https://gerrit.simantics.org/r/gitweb?p=simantics%2Fplatform.git;a=commitdiff_plain;h=2ccb3d785e9239edfeb9327e92b95be6d303caa8;hp=0d09c1eadf0d26185bc3d5b1467523beb585f551

Merge "Log some information about missing property in SCLLabelRule"
---

diff --git a/bundles/org.simantics.modeling/src/org/simantics/modeling/adapters/SCLLabelRule.java b/bundles/org.simantics.modeling/src/org/simantics/modeling/adapters/SCLLabelRule.java
index e0bff0fe1..243ef4d36 100644
--- a/bundles/org.simantics.modeling/src/org/simantics/modeling/adapters/SCLLabelRule.java
+++ b/bundles/org.simantics.modeling/src/org/simantics/modeling/adapters/SCLLabelRule.java
@@ -14,8 +14,11 @@ import org.simantics.db.layer0.variable.Variables;
 import org.simantics.modeling.ModelingResources;
 import org.simantics.scl.runtime.SCLContext;
 import org.simantics.scl.runtime.function.Function1;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class SCLLabelRule implements LabelRule {
+    private static final Logger LOGGER = LoggerFactory.getLogger(SCLLabelRule.class);
 
     private Resource rule;
     
@@ -35,7 +38,10 @@ public class SCLLabelRule implements LabelRule {
         Variable ruleVariable = Variables.getVariable(graph, rule);
 
         Function1<Object,List<String>> getLabels = ruleVariable.getPossiblePropertyValue(graph, MOD.SCLLabelRule_getLabels);
-        if(getLabels == null) return Collections.emptyMap();
+        if(getLabels == null) {
+            LOGGER.warn("Didn't find value for subject={}, predicate={}.", rule, ModelingResources.URIs.SCLLabelRule_getLabels);
+            return Collections.emptyMap();
+        }
 
         SCLContext sclContext = SCLContext.getCurrent();
         Object oldGraph = sclContext.get("graph");
@@ -48,6 +54,7 @@ public class SCLLabelRule implements LabelRule {
             }
             return result;
         } catch (Throwable t) {
+            LOGGER.error("Calculating label failed.", t);
         	throw new DatabaseException(t);
         } finally {
             sclContext.put("graph", oldGraph);