stop double calling on_disable callbacks

Author: LucienMorey

magicbot/magiccomponent.py

@@ -60,7 +60,7 @@ def on_enable(self) -> None:
 
     def on_disable(self) -> None:
         """
-        Called when the robot leaves autonomous, teleoperated or test mode
+        Called when the robot enters disabled mode.
         """
 
     def execute(self) -> None:

magicbot/magicrobot.py

@@ -413,8 +413,6 @@ def autonomous(self) -> None:
             watchdog=self.watchdog,
         )
 
-        self._on_mode_disable_components()
-
     def _disabled(self) -> None:
         """
         This function is called in disabled mode. You should not
@@ -513,8 +511,6 @@ def _operatorControl(self) -> None:
                 delay.wait()
                 watchdog.reset()
 
-        self._on_mode_disable_components()
-
     def _test(self) -> None:
         """Called when the robot is in test mode"""
         watchdog = self.watchdog
@@ -562,8 +558,6 @@ def _test(self) -> None:
         wpilib.LiveWindow.setEnabled(False)
         # Shuffleboard.disableActuatorWidgets()
 
-        self._on_mode_disable_components()
-
     def _on_mode_enable_components(self) -> None:
         # initialize things
         for _, component in self._components: