From: Marko Luukkainen <marko.luukkainen@semantum.fi>
Date: Fri, 1 Feb 2019 13:01:16 +0000 (+0200)
Subject: Fix selectPath returning true for nodes that haven't been loaded.
X-Git-Url: https://gerrit.simantics.org/r/gitweb?p=simantics%2Fplatform.git;a=commitdiff_plain;h=73136207456be831c4859ad365071bc359e67a0d

Fix selectPath returning true for nodes that haven't been loaded.

gitlab #253

Change-Id: Ifdc5853e34f2c213acc6dcbffb4da3085936d60d
(cherry picked from commit cd343042b5518b7a45fffb35ccb91cb4d810a6e7)
---

diff --git a/bundles/org.simantics.browsing.ui.swt/src/org/simantics/browsing/ui/swt/GraphExplorerImpl.java b/bundles/org.simantics.browsing.ui.swt/src/org/simantics/browsing/ui/swt/GraphExplorerImpl.java
index 3cf5f7478..817a8d961 100644
--- a/bundles/org.simantics.browsing.ui.swt/src/org/simantics/browsing/ui/swt/GraphExplorerImpl.java
+++ b/bundles/org.simantics.browsing.ui.swt/src/org/simantics/browsing/ui/swt/GraphExplorerImpl.java
@@ -3118,7 +3118,10 @@ class GraphExplorerImpl extends GraphExplorerImplBase implements Listener, Graph
     	//System.out.println("NodeContext path : " + contexts);
 
     	NodeContext head = tryFind(contexts[position]);
-
+    	// tryFind may return null for positions, that actually have NodeContext. 
+    	if (head == null)
+    	    return false;
+    	
     	if(position == contexts.length-1) {
     		return select(head);