remove unused part field from SmaterButton

FirstPersonKSP · Aug 14, 2024 · 6139c6f · 6139c6f
1 parent 742e0f1
commit 6139c6f
Showing 1 changed file with 11 additions and 18 deletions.
diff --git a/RasterPropMonitor/Core/ButtonHandler.cs b/RasterPropMonitor/Core/ButtonHandler.cs
@@ -31,7 +31,6 @@ public class SmarterButton : MonoBehaviour
         private readonly List<Action> clickHandlers = new List<Action>();
         private readonly List<Action> releaseHandlers = new List<Action>();
         private readonly List<PageTriggerSet> pageTriggers = new List<PageTriggerSet>();
-        private Part part;
 
         private struct HandlerID
         {
@@ -81,24 +80,21 @@ public void ShowNext()
 
         public void OnMouseDown()
         {
-            if (part != null)
+            Kerbal k = CameraManager.Instance.ivaCameraActiveKerbal;
+            if (k != null)
             {
-                Kerbal k = CameraManager.Instance.ivaCameraActiveKerbal;
-                if (k != null)
+                // Disallow tourists using props
+                if (k.protoCrewMember.type == ProtoCrewMember.KerbalType.Tourist)
                 {
-                    // Disallow tourists using props
-                    if (k.protoCrewMember.type == ProtoCrewMember.KerbalType.Tourist)
+                    if (UnityEngine.Random.Range(0, 10) > 8)
                     {
-                        if (UnityEngine.Random.Range(0, 10) > 8)
-                        {
-                            ScreenMessages.PostScreenMessage(string.Format("Stop touching buttons, {0}!", k.name), 4.0f, ScreenMessageStyle.UPPER_CENTER);
-                        }
-                        else
-                        {
-                            ScreenMessages.PostScreenMessage(string.Format("Tourist {0} may not operate equipment.", k.name), 4.0f, ScreenMessageStyle.UPPER_CENTER);
-                        }
-                        return;
+                        ScreenMessages.PostScreenMessage(string.Format("Stop touching buttons, {0}!", k.name), 4.0f, ScreenMessageStyle.UPPER_CENTER);
+                    }
+                    else
+                    {
+                        ScreenMessages.PostScreenMessage(string.Format("Tourist {0} may not operate equipment.", k.name), 4.0f, ScreenMessageStyle.UPPER_CENTER);
                     }
+                    return;
                 }
             }
             foreach (PageTriggerSet monitor in pageTriggers)
@@ -194,7 +190,6 @@ public static void CreateButton(InternalProp thatProp, string buttonName, Monito
             }
 
             buttonBehaviour.pageTriggers.Add(new PageTriggerSet(handlerFunction, thatPage));
-            buttonBehaviour.part = (thatModel == null) ? thatProp.part : thatModel.part;
         }
 
         public static void CreateButton(InternalProp thatProp, string buttonName, int numericID, Action<int> clickHandlerFunction, Action<int> releaseHandlerFunction, InternalModel thatModel = null)
@@ -215,7 +210,6 @@ public static void CreateButton(InternalProp thatProp, string buttonName, int nu
                 function = releaseHandlerFunction,
                 idValue = numericID
             });
-            buttonBehaviour.part = (thatModel == null) ? thatProp.part : thatModel.part;
         }
 
         public static void CreateButton(InternalProp thatProp, string buttonName, Action handlerFunction, Action releaseHandlerFunction = null, InternalModel thatModel = null)
@@ -231,7 +225,6 @@ public static void CreateButton(InternalProp thatProp, string buttonName, Action
             {
                 buttonBehaviour.releaseHandlers.Add(releaseHandlerFunction);
             }
-            buttonBehaviour.part = (thatModel == null) ? thatProp.part : thatModel.part;
         }
     }
 }