Improve implementation of the Video Config Error Popup (#32)

opensensorhub · Sep 11, 2021 · 10d44e0 · 10d44e0
1 parent 92216c6
commit 10d44e0
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/sensorhub-android-app/src/org/sensorhub/android/MainActivity.java b/sensorhub-android-app/src/org/sensorhub/android/MainActivity.java
@@ -448,20 +448,17 @@ public void onClick(DialogInterface dialog, int whichButton)
                 updateConfig(PreferenceManager.getDefaultSharedPreferences(MainActivity.this), runName);
 
                 AndroidSensorsConfig androidSensorConfig = (AndroidSensorsConfig) sensorhubConfig.get("ANDROID_SENSORS");
-                Log.d("OSH-DEBUG", androidSensorConfig.toString());
                 VideoEncoderConfig videoConfig = androidSensorConfig.videoConfig;
 
-                if(videoConfig.selectedPreset >= 0 && videoConfig.selectedPreset < videoConfig.presets.length){
-                    Log.d("OSH-DEBUG", "TEST MESSAGE...");
-
+                if ((androidSensorConfig.activateBackCamera || androidSensorConfig.activateFrontCamera) && videoConfig.selectedPreset < 0 || videoConfig.selectedPreset >= videoConfig.presets.length) {
+                    showVideoConfigErrorPopup();
+                    newStatusMessage("Video Config Error: Check Settings");
+                } else {
                     sostClients.clear();
                     boundService.startSensorHub(sensorhubConfig, showVideo, MainActivity.this);
 
                     if (boundService.hasVideo())
                         mainInfoArea.setBackgroundColor(0x80FFFFFF);
-                }else{
-                    showVideoConfigErrorPopup();
-                    newStatusMessage("Video Config Error: Check Settings");
                 }
 
             }