Add an exception to getting the manipulator without checking it first.

also do not set an invalid one

Author: madhephaestus

src/main/java/eu/mihosoft/vrl/v3d/CSG.java

@@ -346,7 +346,12 @@ public MeshView newMesh() {
 			current.getTransforms().clear();
 
 		if (hasManipulator)
-			current.getTransforms().add(getManipulator());
+			try {
+				current.getTransforms().add(getManipulator());
+			} catch (MissingManipulatorException e) {
+				// TODO Auto-generated catch block
+				e.printStackTrace();
+			}
 		if (hasAssembly)
 			current.getTransforms().add((Affine) getAssemblyStorage().getValue("AssembleAffine").get());
 
@@ -2913,9 +2918,9 @@ public CSG makeKeepaway(Number sn) {
 	public boolean hasManipulator() {
 		return manipulator.get(uniqueId) != null;
 	}
-	public Affine getManipulator() {
+	public Affine getManipulator() throws MissingManipulatorException{
 		if (!hasManipulator() )
-			manipulator.put(uniqueId, new Affine());
+			throw new MissingManipulatorException("Can not get a manipulator that does not exist");
 		return manipulator.get(uniqueId);
 	}
 
@@ -3140,7 +3145,12 @@ public CSG regenerate() {
 		CSG regenerate2 = regenerate.get(getUniqueId()).regenerate(this);
 		if (regenerate2 != null) {
 			if(hasManipulator())
-				regenerate2.setManipulator(this.getManipulator());
+				try {
+					regenerate2.setManipulator(this.getManipulator());
+				} catch (MissingManipulatorException e) {
+					// TODO Auto-generated catch block
+					e.printStackTrace();
+				}
 			return regenerate2.historySync(this);
 		}
 		return this;

src/main/java/eu/mihosoft/vrl/v3d/MissingManipulatorException.java

@@ -0,0 +1,9 @@
+package eu.mihosoft.vrl.v3d;
+
+public class MissingManipulatorException extends Exception {
+
+	public MissingManipulatorException(String string) {
+		super(string);
+	}
+
+}

src/main/java/eu/mihosoft/vrl/v3d/parametrics/CSGDatabaseInstance.java

@@ -18,6 +18,7 @@
 import com.google.gson.reflect.TypeToken;
 
 import eu.mihosoft.vrl.v3d.CSG;
+import eu.mihosoft.vrl.v3d.MissingManipulatorException;
 
 public class CSGDatabaseInstance {
 
@@ -101,7 +102,12 @@ public CSGDatabaseInstance setParameterNewValue(CSG instance, String key, double
 		if (function != null) {
 			CSG setManipulator = function.change(instance, key, new Long((long) (newValue * 1000)));
 			if(setManipulator.hasManipulator())
-				setManipulator.setManipulator(instance.getManipulator());
+				try {
+					setManipulator.setManipulator(instance.getManipulator());
+				} catch (MissingManipulatorException e) {
+					// TODO Auto-generated catch block
+					e.printStackTrace();
+				}
 			setManipulator.setColor(instance.getColor());
 			return this;
 		}

src/main/java/eu/mihosoft/vrl/v3d/thumbnail/ThumbnailImage.java

@@ -10,6 +10,7 @@
 
 import eu.mihosoft.vrl.v3d.Bounds;
 import eu.mihosoft.vrl.v3d.CSG;
+import eu.mihosoft.vrl.v3d.MissingManipulatorException;
 import eu.mihosoft.vrl.v3d.Vector3d;
 import eu.mihosoft.vrl.v3d.parametrics.CSGDatabaseInstance;
 import javafx.scene.Group;
@@ -71,7 +72,12 @@ public static WritableImage get(List<CSG> c,CSGDatabaseInstance instance) {
 		ArrayList<CSG> csgList = new ArrayList<CSG>();
 		for (CSG cs : c) {
 			if (cs.hasManipulator()) {
-				csgList.add(cs.transformed(TransformConverter.fromAffine(cs.getManipulator())).syncProperties(instance,cs));
+				try {
+					csgList.add(cs.transformed(TransformConverter.fromAffine(cs.getManipulator())).syncProperties(instance,cs));
+				} catch (MissingManipulatorException e) {
+					// TODO Auto-generated catch block
+					e.printStackTrace();
+				}
 			} else
 				csgList.add(cs);
 		}