X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=blobdiff_plain;f=bundles%2Forg.simantics.db.layer0%2Fsrc%2Forg%2Fsimantics%2Fdb%2Flayer0%2Fvariable%2FStandardGraphPropertyVariable.java;h=7927614cd37f9779292ee4ace98e5fd4ba7f00f0;hb=e5c9ce2de1b970b17c3ffc1151d7c49743df6183;hp=2dcb6e00ab08871f296c7af6d5a498115edf7d60;hpb=321607bdbd3ca800128d40b5511092f71cb66367;p=simantics%2Fplatform.git

diff --git a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/variable/StandardGraphPropertyVariable.java b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/variable/StandardGraphPropertyVariable.java
index 2dcb6e00a..7927614cd 100644
--- a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/variable/StandardGraphPropertyVariable.java
+++ b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/variable/StandardGraphPropertyVariable.java
@@ -11,6 +11,7 @@ import org.simantics.databoard.accessor.reference.ChildReference;
 import org.simantics.databoard.binding.Binding;
 import org.simantics.databoard.binding.impl.ObjectVariantBinding;
 import org.simantics.databoard.type.Datatype;
+import org.simantics.db.DevelopmentKeys;
 import org.simantics.db.ReadGraph;
 import org.simantics.db.Resource;
 import org.simantics.db.WriteGraph;
@@ -19,13 +20,13 @@ import org.simantics.db.common.validation.L0Validations;
 import org.simantics.db.exception.DatabaseException;
 import org.simantics.db.exception.DatatypeNotFoundException;
 import org.simantics.db.exception.ValidationException;
-import org.simantics.db.exception.VariableException;
 import org.simantics.db.layer0.exception.InvalidVariableException;
 import org.simantics.db.layer0.exception.MissingVariableValueException;
 import org.simantics.db.layer0.exception.PendingVariableException;
 import org.simantics.db.layer0.function.All;
 import org.simantics.db.layer0.request.PropertyInfo;
 import org.simantics.db.layer0.request.PropertyInfoRequest;
+import org.simantics.db.layer0.scl.SCLDatabaseException;
 import org.simantics.db.layer0.util.Layer0Utils;
 import org.simantics.layer0.Layer0;
 import org.simantics.utils.Development;
@@ -37,7 +38,7 @@ public class StandardGraphPropertyVariable extends AbstractPropertyVariable {
     private static final Logger LOGGER = LoggerFactory.getLogger(StandardGraphPropertyVariable.class);
     
 	protected static final PropertyInfo NO_PROPERTY = new PropertyInfo(null, null,
-			false, Collections.<String> emptySet(), null, null, null, null, null, null,
+			false, false, Collections.<String> emptySet(), null, null, null, null, null, null,
 			Collections.<String, Pair<Resource, ChildReference>> emptyMap(),
 			null, false);
 
@@ -121,10 +122,12 @@ public class StandardGraphPropertyVariable extends AbstractPropertyVariable {
 	public <T> T getValue(ReadGraph graph) throws DatabaseException {
 		
 		if(Development.DEVELOPMENT) {
-			String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
-			if(error != null) {
-			    LOGGER.error(error);
-				//throw new ValidationException(error);
+			if(Development.<Boolean>getProperty(DevelopmentKeys.L0_VALIDATION, Bindings.BOOLEAN)) {
+				String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
+				if(error != null) {
+				    LOGGER.error(error);
+					throw new ValidationException(error);
+				}
 			}
 		}
 		
@@ -140,18 +143,22 @@ public class StandardGraphPropertyVariable extends AbstractPropertyVariable {
 			return getValue(graph);		
 
 		if(Development.DEVELOPMENT) {
-			String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
-			if(error != null) {
-			    LOGGER.error(error);
-				throw new ValidationException(error);
+			if(Development.<Boolean>getProperty(DevelopmentKeys.L0_VALIDATION, Bindings.BOOLEAN)) {
+				String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
+				if(error != null) {
+					LOGGER.error(error);
+					throw new ValidationException(error);
+				}
 			}
 		}
         
 		try {
 			
-			return (T)getValueAccessor(graph).getValue(graph, this, binding);
-		} catch (PendingVariableException e) {
-		    throw e;
+			return (T) getValueAccessor(graph).getValue(graph, this, binding);
+		} catch (SCLDatabaseException e) { // these can be thrown when compiling e.g. derived properties
+			throw e;
+		} catch (MissingVariableValueException | PendingVariableException e) {
+			throw e;
 		} catch (Throwable t) {
 			throw new MissingVariableValueException(t);
 		}
@@ -177,10 +184,12 @@ public class StandardGraphPropertyVariable extends AbstractPropertyVariable {
 	public void setValue(WriteGraph graph, Object value, Binding binding) throws DatabaseException {
 		
 		if(Development.DEVELOPMENT) {
-			String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
-			if(error != null) {
-			    LOGGER.error(error);
-				//throw new ValidationException(error);
+			if(Development.<Boolean>getProperty(DevelopmentKeys.L0_VALIDATION, Bindings.BOOLEAN)) {
+				String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
+				if(error != null) {
+					LOGGER.error(error);
+					throw new ValidationException(error);
+				}
 			}
 		}
 		
@@ -192,10 +201,12 @@ public class StandardGraphPropertyVariable extends AbstractPropertyVariable {
 	public void setValue(WriteGraph graph, Object value) throws DatabaseException {
 		
 		if(Development.DEVELOPMENT) {
-			String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
-			if(error != null) {
-			    LOGGER.error(error);
-				throw new ValidationException(error);
+			if(Development.<Boolean>getProperty(DevelopmentKeys.L0_VALIDATION, Bindings.BOOLEAN)) {
+				String error = L0Validations.checkValueType(graph, parentResource, property.predicate);
+				if(error != null) {
+					LOGGER.error(error);
+					throw new ValidationException(error);
+				}
 			}
 		}