ver 2.4.3

ChangeLog

@@ -1,3 +1,8 @@
+2.4.3, 2019-10-12
+- KSP 1.7
+- Small correction to maneuver select logic
+- Remove compatibility checker. It's more annoyance than help.
+
 2.4.2, 2019-03-23
 - KSP 1.6
 - German translation

PreciseManeuver.csproj

@@ -57,7 +57,6 @@
     <Compile Include="src\PreciseManeuverToolbar.cs" />
     <Compile Include="src\KACWrapper.cs" />
     <Compile Include="src\PreciseManeuverHotkeys.cs" />
-    <Compile Include="src\CompatibilityChecker.cs" />
     <Compile Include="src\NodeManager.cs" />
     <Compile Include="src\NodeTools.cs" />
     <Compile Include="src\PreciseManeuverConfig.cs" />

PreciseManeuver.version

@@ -9,22 +9,22 @@
     "VERSION": {
         "MAJOR": 2,
         "MINOR": 4,
-        "PATCH": 2,
+        "PATCH": 3,
         "BUILD": 0
     },
     "KSP_VERSION": {
         "MAJOR": 1,
-        "MINOR": 6,
+        "MINOR": 7,
         "PATCH": 0
     },
     "KSP_VERSION_MIN": {
         "MAJOR": 1,
-        "MINOR": 6,
+        "MINOR": 7,
         "PATCH": 0
     },
     "KSP_VERSION_MAX": {
         "MAJOR": 1,
-        "MINOR": 6,
+        "MINOR": 7,
         "PATCH": 99
     }
 }

properties/AssemblyInfo.Unity.cs

@@ -6,5 +6,5 @@
 [assembly: AssemblyCopyright("Copyleft (ɔ) 2017. Modify and distribute it at will! Hail open source!")]
 [assembly: ComVisible(false)]
 [assembly: Guid("3ebb00a5-1bbe-46c1-ae28-01b5f6c603f6")]
-[assembly: AssemblyVersion("2.4.0.0")]
-[assembly: AssemblyFileVersion("2.4.0.0")]
+[assembly: AssemblyVersion("2.4.3.0")]
+[assembly: AssemblyFileVersion("2.4.3.0")]

properties/AssemblyInfo.cs

@@ -31,8 +31,8 @@
 // You can specify all the values or you can default the Build and Revision Numbers
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("2.4.2")]
-[assembly: AssemblyFileVersion("2.4.2")]
+[assembly: AssemblyVersion("2.4.3")]
+[assembly: AssemblyFileVersion("2.4.3")]
 [assembly: AssemblyInformationalVersion("")]
 
 [assembly: KSPAssembly("PreciseManeuver", 2, 4)]

src/NodeManager.cs

@@ -60,8 +60,6 @@ internal ManeuverNode CurrentNode {
     }
     internal int CurrentNodeIdx { get; private set; } = -1;
     private int nodeCount = 0;
-    /* list that helps find the newly selected nodes */
-    private List<ManeuverNode> prevGizmos = null;
     /* Internal copy of the node */
     private SavedNode currentSavedNode = null;
     private SavedNode CurrentSavedNode {
@@ -333,30 +331,11 @@ internal void CircularizeOrbit () {
 
     internal void SearchNewGizmo () {
       var solver = FlightGlobals.ActiveVessel.patchedConicSolver;
-      var curList = solver.maneuverNodes.Where (a => a.attachedGizmo != null);
-      var tmp = curList.ToList ();
-      /* let's see if user is hovering a mouse *
-       * over any gizmo. That would be a hint. */
-      if (curList.Count (a => a.attachedGizmo.MouseOverGizmo) == 1) {
-        var node = curList.First (a => a.attachedGizmo.MouseOverGizmo);
-        if (node != currentNode) {
-          currentNode = node;
-          NotifyNodeChanged ();
-        }
-      } else {
-        /* then, let's see if we can find any     *
-         * new gizmos that were created recently. */
-        if (prevGizmos != null)
-          curList = curList.Except (prevGizmos);
-        if (curList.Count () == 1) {
-          var node = curList.First ();
-          if (node != currentNode) {
-            currentNode = node;
-            NotifyNodeChanged ();
-          }
-        }
+      var curList = solver.maneuverNodes.Where (a => a.attachedGizmo != null && a != currentNode).ToArray();
+
+      if (curList.Length == 1) {
+        currentNode = curList.First ();
       }
-      prevGizmos = tmp;
     }
 
     private void UpdateCurrentNode () {

src/NodeTools.cs

@@ -38,6 +38,9 @@ internal static Orbit GetTargetOrbit (CelestialBody refbody) {
         return null;
 
       Orbit o = tgt.GetOrbit ();
+      if (o == null)
+        return null;
+
       if (o.referenceBody == refbody)
         return o;
 

src/PreciseManeuver.cs

@@ -31,7 +31,7 @@
 using KSP.Localization;
 
 namespace KSPPreciseManeuver {
-  [KSPAddon (KSPAddon.Startup.Flight, false)]
+  [KSPAddon (KSPAddon.Startup.Flight, true)]
   internal class PreciseManeuver : MonoBehaviour {
     private MainWindow mainWindow = new MainWindow();
     private PreciseManeuverHotkeys hotkeys = new PreciseManeuverHotkeys();
@@ -47,12 +47,18 @@ internal class PreciseManeuver : MonoBehaviour {
 
     private GameObject m_WindowPrefab = PreciseManeuverConfig.Instance.Prefabs.LoadAsset<GameObject> ("PreciseManeuverWindow");
 
-    private int waitForGizmo = 0;
-
     internal void Start () {
+      DontDestroyOnLoad (this);
+
+      GameEvents.onManeuverNodeSelected.Add (new EventVoid.OnEvent (manager.SearchNewGizmo));
+
       KACWrapper.InitKACWrapper ();
     }
 
+    internal void OnDestroy() {
+      GameEvents.onManeuverNodeSelected.Remove (new EventVoid.OnEvent (manager.SearchNewGizmo));
+    }
+
     internal void OnDisable () {
       CloseKeybindingsWindow ();
       CloseMainWindow ();
@@ -81,13 +87,6 @@ internal void Update () {
         if (m_MainWindowObject != null) {
           hotkeys.ProcessRegularHotkeys ();
 
-          if (Input.GetMouseButtonUp (0))
-            waitForGizmo = 3;
-          if (waitForGizmo > 0) {
-            if (waitForGizmo == 1)
-              manager.SearchNewGizmo ();
-            waitForGizmo--;
-          }
           manager.UpdateNodes ();
         }
       }

src/PreciseManeuverToolbar.cs

@@ -34,11 +34,11 @@ namespace KSPPreciseManeuver {
   class PreciseManeuverToolbar : MonoBehaviour, UI.IMenuControl {
 
     private ApplicationLauncherButton appButton = null;
-    private Texture appButtonTexture = PreciseManeuverConfig.Instance.Prefabs.LoadAsset<Texture> ("PreciseManeuverIcon");
+    private Texture appButtonTexture;
 
     private UI.ToolbarMenu m_ToolbarMenu;
     private GameObject m_MenuObject;
-    private GameObject m_MenuPrefab = PreciseManeuverConfig.Instance.Prefabs.LoadAsset<GameObject>("PreciseManeuverMenu");
+    private GameObject m_MenuPrefab;
 
     private bool m_MenuPointerIn = false;
 
@@ -138,9 +138,12 @@ public void OnMenuPointerExit () {
       InputLockManager.RemoveControlLock ("PreciseManeuverMenuControlLock");
     }
 
-    internal void Awake () {
+    internal void Start () {
       DontDestroyOnLoad (this);
 
+      appButtonTexture = PreciseManeuverConfig.Instance.Prefabs.LoadAsset<Texture> ("PreciseManeuverIcon");
+      m_MenuPrefab = PreciseManeuverConfig.Instance.Prefabs.LoadAsset<GameObject> ("PreciseManeuverMenu");
+
       // subscribe event listeners
       GameEvents.onGUIApplicationLauncherReady.Add (OnGUIApplicationLauncherReady);
       GameEvents.onGUIApplicationLauncherUnreadifying.Add (OnGUIApplicationLauncherUnreadifying);