X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=blobdiff_plain;f=org.simantics.interop.update%2Fsrc%2Forg%2Fsimantics%2Finterop%2Fupdate%2Fmodel%2FModelUpdate.java;h=d76c47bdf655998c902de59f6f2b7f08c58f7bd8;hb=ab4fc05981803ce2c430f93f34b8b6a8f85a53ce;hp=e736e22e8747758bf2aa640936ae788d102b4c07;hpb=9a54e6b7fe17613852fc676deb4dc42c3c5e2b8f;p=simantics%2Finterop.git

diff --git a/org.simantics.interop.update/src/org/simantics/interop/update/model/ModelUpdate.java b/org.simantics.interop.update/src/org/simantics/interop/update/model/ModelUpdate.java
index e736e22..d76c47b 100644
--- a/org.simantics.interop.update/src/org/simantics/interop/update/model/ModelUpdate.java
+++ b/org.simantics.interop.update/src/org/simantics/interop/update/model/ModelUpdate.java
@@ -71,7 +71,7 @@ public abstract class ModelUpdate {
 				showWarning(result2.second);
 			comparator2.test(getSession());
 			changes2 = comparator2.getChanges();
-			changes2 = getSession().syncRequest(new FilterChangesRead(changes2, filters));
+			changes2 = getSession().syncRequest(createFilterRead(changes2, filters));
 			updateTree2 = getUpdateTree(changes2);
 			updateList2 = getUpdateList(changes2);
 			
@@ -82,7 +82,7 @@ public abstract class ModelUpdate {
 				showWarning(result3.second);
 			comparator3.test(getSession());
 			changes3 = comparator3.getChanges();
-			changes3 = getSession().syncRequest(new FilterChangesRead(changes3, filters));
+			changes3 = getSession().syncRequest(createFilterRead(changes3, filters));
 		}
 		
 		Pair<GraphComparator,String> result = getChanges(oldModel,newModel);
@@ -124,7 +124,7 @@ public abstract class ModelUpdate {
 		}
 		comparator.test(getSession());
 		changes = comparator.getChanges();
-		changes = getSession().syncRequest(new FilterChangesRead(changes, filters));
+		changes = getSession().syncRequest(createFilterRead(changes, filters));
 		updateTree = getUpdateTree(changes);
 		updateList = getUpdateList(changes);
 		if (userFilters.size() != 0) {
@@ -194,7 +194,7 @@ public abstract class ModelUpdate {
 			// Create filtered changes
 			List<ChangeFilter> combined = new ArrayList<>(filters);
 			combined.addAll(userFilters);
-			GraphChanges filteredChanges = getSession().syncRequest(new FilterChangesRead(changes, combined));
+			GraphChanges filteredChanges = getSession().syncRequest(createFilterRead(changes, combined));
 			UpdateTree updateTreeF = getUpdateTree(filteredChanges);
 			UpdateList updateListF = getUpdateList(filteredChanges);
 			// hide changes that are not contained within the filtered changes.
@@ -240,7 +240,11 @@ public abstract class ModelUpdate {
 			for (PropertyChange rc : r) {
 				if (comparable.containsRight(rc))
 					continue;
-				if (lc.getFirst().equals(rc.getFirst())) {
+				if (lc.getFirst() != null && lc.getFirst().equals(rc.getFirst())) {
+					comparable.map(lc, rc);
+					break;
+				}
+				if (lc.getSecond() != null && lc.getSecond().equals(rc.getSecond())) {
 					comparable.map(lc, rc);
 					break;
 				}
@@ -339,6 +343,10 @@ public abstract class ModelUpdate {
 		return Simantics.getSession();
 	}
 	
+	public Read<GraphChanges> createFilterRead(GraphChanges changes, List<ChangeFilter> filters) {
+		return new FilterChangesRead(changes, filters);
+	}
+	
 	
 	
 	public static class FilterChangesRead implements Read<GraphChanges> {
@@ -467,7 +475,7 @@ public abstract class ModelUpdate {
 			if (pair.getFirst() != null) {
 				boolean found = false;
 				for (PropertyChange pair2 : updateList2.getChanges()) {
-					if (pair.getFirst().equals(pair2.getSecond())) {
+					if (pair.getFirst() != null && pair.getFirst().equals(pair2.getSecond())) {
 						found = true;
 						break;
 					}