X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=blobdiff_plain;f=org.simantics.g3d.vtk%2Fsrc%2Forg%2Fsimantics%2Fg3d%2Fvtk%2Fcommon%2FAbstractVTKNodeMap.java;fp=org.simantics.g3d.vtk%2Fsrc%2Forg%2Fsimantics%2Fg3d%2Fvtk%2Fcommon%2FAbstractVTKNodeMap.java;h=bb5493f6b7a76ee806d712d402691f5b862cdc8d;hb=251cee0bb175b6a6ed1123509bc7a54c22d3443d;hp=2c37521b8367b751451d3a73c5b02cbca60977cd;hpb=f537452332e2b78352677bd50ac875aa536f0a1c;p=simantics%2F3d.git

diff --git a/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java b/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
index 2c37521b..bb5493f6 100644
--- a/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
+++ b/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
@@ -227,6 +227,7 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 			}
 			if (changeTracking) {
 				mapping.rangeModified((E)node.getParent());
+				mapping.rangeModified((E)node);
 			}
 			added.add(new Pair<E, String>(node, id));
 			rangeModified = true;
@@ -242,8 +243,10 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 				if (n.first.equals(node))
 					return;
 			}
-			if (changeTracking && !db)
+			if (changeTracking && !db) {
+			    mapping.rangeModified((E)node);
 				mapping.rangeModified((E)node.getParent());
+			}
 			removed.add(new Pair<E, String>(node, id));
 			rangeModified = true;
 		}
@@ -438,6 +441,7 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 		for (E n : delete) {
 			if (DEBUG) System.out.println(debugString(n));
 			mapping.getRange().remove(n);
+			stopListening(n);
 		}
 		delete.clear();
 	}
@@ -623,7 +627,6 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 		if (DEBUG) System.out.println("Node added " + child + " parent " + node);
 		//receiveAdd((E)child, rel ,graphUpdates);
 		receiveAdd((E)child, rel ,graphModified.contains(node));
-		
 	}
 	
 	@SuppressWarnings("unchecked")
@@ -634,9 +637,10 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 		//receiveRemove((E)child, rel, graphUpdates);
 		receiveRemove((E)child, rel, graphModified.contains(node));
 		
-		//FIXME : sometimes removed structural models cause ObjMap to add their children again.
-		//        removing the listener here prevents corruption of visual model, but better fix is needed.
-		stopListening(child);
+		//FIXME : 1. sometimes removed structural models cause ObjMap to add their children again.
+		//           removing the listener here prevents corruption of visual model, but better fix is needed.
+		//        2. detach causes nodeRemoved event, which then causes other critical events to be missed. Took out th 
+		//stopListening(child);
 	}
 	
 	@Override