fixed automode bug in web controller

ob-f · Oct 17, 2024 · 78c720c · 78c720c
1 parent f56fa0b
commit 78c720c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 10 deletions.
diff --git a/android/robot/src/main/java/org/openbot/autopilot/AutopilotFragment.java b/android/robot/src/main/java/org/openbot/autopilot/AutopilotFragment.java
@@ -155,7 +155,6 @@ public void onNothingSelected(AdapterView<?> parent) {
 
         setSpeedMode(Enums.SpeedMode.getByID(preferencesManager.getSpeedMode()));
         setControlMode(Enums.ControlMode.getByID(preferencesManager.getControlMode()));
-        System.out.println("drive mode--->" + preferencesManager.getDriveMode());
         setDriveMode(Enums.DriveMode.getByID(preferencesManager.getDriveMode()));
 
         binding.controllerContainer.controlMode.setOnClickListener(
@@ -390,6 +389,9 @@ private void setNetworkEnabled(boolean b) {
             if (Enums.DriveMode.getByID(preferencesManager.getDriveMode()) == Enums.DriveMode.DUAL && Enums.ControlMode.getByID(preferencesManager.getControlMode()) == Enums.ControlMode.PHONE) {
                 binding.controllerContainer.driveMode.setAlpha(0.5f);
                 binding.controllerContainer.driveMode.setEnabled(false);
+            } else if (Enums.DriveMode.getByID(preferencesManager.getDriveMode()) == Enums.DriveMode.GAME && Enums.ControlMode.getByID(preferencesManager.getControlMode()) == Enums.ControlMode.WEBSERVER) {
+                binding.controllerContainer.driveMode.setAlpha(0.5f);
+                binding.controllerContainer.driveMode.setEnabled(false);
             }
         }
 
@@ -463,12 +465,10 @@ protected Model getModel() {
 
     private void connectWebController() {
         phoneController.connectWebServer();
-        Enums.DriveMode oldDriveMode = currentDriveMode;
         // Currently only dual drive mode supported
         setDriveMode(Enums.DriveMode.GAME);
         binding.controllerContainer.driveMode.setAlpha(0.5f);
         binding.controllerContainer.driveMode.setEnabled(false);
-        preferencesManager.setDriveMode(oldDriveMode.getValue());
     }
 
     protected void setModel(Model model) {
@@ -579,12 +579,10 @@ protected void setDriveMode(Enums.DriveMode driveMode) {
 
     private void connectPhoneController() {
         phoneController.connect(requireContext());
-        Enums.DriveMode oldDriveMode = currentDriveMode;
         // Currently only dual drive mode supported
         setDriveMode(Enums.DriveMode.DUAL);
         binding.controllerContainer.driveMode.setAlpha(0.5f);
         binding.controllerContainer.driveMode.setEnabled(false);
-        preferencesManager.setDriveMode(oldDriveMode.getValue());
     }
 
     private void disconnectPhoneController() {

diff --git a/android/robot/src/main/java/org/openbot/objectNav/ObjectNavFragment.java b/android/robot/src/main/java/org/openbot/objectNav/ObjectNavFragment.java
@@ -428,6 +428,10 @@ private void setNetworkEnabled(boolean b) {
         binding.controllerContainer.driveMode.setAlpha(0.5f);
         binding.controllerContainer.driveMode.setEnabled(false);
       }
+     else if (Enums.DriveMode.getByID(preferencesManager.getDriveMode()) == Enums.DriveMode.GAME && Enums.ControlMode.getByID(preferencesManager.getControlMode()) == Enums.ControlMode.WEBSERVER) {
+        binding.controllerContainer.driveMode.setAlpha(0.5f);
+        binding.controllerContainer.driveMode.setEnabled(false);
+      }
     }
 
     resetFpsUi();
@@ -667,12 +671,10 @@ private void connectPhoneController() {
 
   private void connectWebController() {
     phoneController.connectWebServer();
-    Enums.DriveMode oldDriveMode = currentDriveMode;
     // Currently only dual drive mode supported
     setDriveMode(Enums.DriveMode.GAME);
     binding.controllerContainer.driveMode.setAlpha(0.5f);
     binding.controllerContainer.driveMode.setEnabled(false);
-    preferencesManager.setDriveMode(oldDriveMode.getValue());
   }
 
   private void disconnectPhoneController() {

diff --git a/android/robot/src/main/java/org/openbot/robot/FreeRoamFragment.java b/android/robot/src/main/java/org/openbot/robot/FreeRoamFragment.java
@@ -262,7 +262,6 @@ protected void setDriveMode(DriveMode driveMode) {
 
   private void connectPhoneController() {
     phoneController.connect(requireContext());
-    DriveMode oldDriveMode = currentDriveMode;
     // Currently only dual drive mode supported
     setDriveMode(DriveMode.DUAL);
     binding.controllerContainer.driveMode.setAlpha(0.5f);
@@ -271,12 +270,10 @@ private void connectPhoneController() {
 
   private void connectWebController() {
     phoneController.connectWebServer();
-    Enums.DriveMode oldDriveMode = currentDriveMode;
     // Currently only dual drive mode supported
     setDriveMode(Enums.DriveMode.GAME);
     binding.controllerContainer.driveMode.setAlpha(0.5f);
     binding.controllerContainer.driveMode.setEnabled(false);
-    preferencesManager.setDriveMode(oldDriveMode.getValue());
   }
 
   private void disconnectPhoneController() {