Version 1.1.0 closeout fixes

src/kOS.Safe/UpdateHandler.cs

@@ -61,7 +61,31 @@ public void UpdateFixedObservers(double deltaTime)
                 observer.KOSFixedUpdate(deltaTime);
             }
         }
-
+
+        public void ClearAllObservers()
+        {
+            var snapshot = new HashSet<IUpdateObserver>(observers);
+            foreach (var observer in snapshot)
+            {
+                IDisposable disp = observer as IDisposable;
+                if (disp != null)
+                {
+                    disp.Dispose();
+                }
+            }
+            var snapshotFixed = new HashSet<IFixedUpdateObserver>(fixedObservers);
+            foreach (var observer in snapshotFixed)
+            {
+                IDisposable disp = observer as IDisposable;
+                if (disp != null)
+                {
+                    disp.Dispose();
+                }
+            }
+            observers.Clear();
+            fixedObservers.Clear();
+        }
+
         /// <summary>
         /// Return all the registered fixed update handlers of a particular type
         /// </summary>

src/kOS/Communication/CommNetConnectivityManager.cs

@@ -105,15 +105,15 @@ public bool HasConnection(Vessel vessel1, Vessel vessel2)
                 return true;
 
             // We next need to query the network to find a connection between the two vessels.
-            // I found no exposed method for accessing chached paths directly, other than the
+            // I found no exposed method for accessing cached paths directly, other than the
             // control path.
 
             // WARNING: In stock this will only work for vessels with a relay antenna installed.
             // This is a limitation put in place to improve performance in the stock game, and
             // there isn't a very good way around it.
             var net = CommNetNetwork.Instance.CommNet;
             tempPath = new CommPath();
-            return net.FindPath(vessel1.Connection.Comm, tempPath, vessel2.Connection.Comm) || net.FindPath(vessel2.Connection.Comm, tempPath, vessel1.Connection.Comm);
+            return vessel1.id == vessel2.id || net.FindPath(vessel1.Connection.Comm, tempPath, vessel2.Connection.Comm) || net.FindPath(vessel2.Connection.Comm, tempPath, vessel1.Connection.Comm);
         }
 
         public void AddAutopilotHook(Vessel vessel, FlightInputCallback hook)

src/kOS/Function/HighlightStructure.cs

@@ -35,6 +35,8 @@ public HighlightStructure(UpdateHandler updateHandler, object toHighlight, RgbaC
             stale = true;
             enabled = true;
             updateHandler.AddObserver(this);
+            GameEvents.onStageSeparation.Add(OnHighlightStaleEvent);
+            GameEvents.onVesselPartCountChanged.Add(OnHighlightStaleEvent);
         }
 
         private void InitializeSuffixes()
@@ -111,9 +113,21 @@ private void HighlightPart(PartValue partValue)
             part.SetHighlight(enabled, false);
         }
 
+        private void OnHighlightStaleEvent(EventReport evt)
+        {
+            stale = true;
+        }
+
+        private void OnHighlightStaleEvent(Vessel ves)
+        {
+            stale = true;
+        }
+
         public void Dispose()
         {
             updateHandler.RemoveObserver(this);
+            GameEvents.onStageSeparation.Remove(OnHighlightStaleEvent);
+            GameEvents.onVesselPartCountChanged.Remove(OnHighlightStaleEvent);
         }
 
         public override string ToString()

src/kOS/SharedObjects.cs

@@ -38,10 +38,11 @@ public void DestroyObjects()
             if (BindingMgr != null) { BindingMgr.Dispose(); }
             if (Window != null) { UnityEngine.Object.Destroy(Window); }
             if (SoundMaker != null) { SoundMaker.StopAllVoices(); }
+            if (UpdateHandler != null) { UpdateHandler.ClearAllObservers(); }
             var props = typeof(SharedObjects).GetProperties();
             foreach (var prop in props)
             {
-                if (!prop.PropertyType.IsValueType)
+                if (!prop.PropertyType.IsValueType && prop.GetSetMethod() != null)
                 {
                     prop.SetValue(this, null, null);
                 }