Pebble: make watchapp installation available on FW 3.x without the need for setting an option

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/AppManagerActivity.java

@@ -5,9 +5,7 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
-import android.content.SharedPreferences;
 import android.os.Bundle;
-import android.preference.PreferenceManager;
 import android.support.v4.app.NavUtils;
 import android.support.v4.content.LocalBroadcastManager;
 import android.view.ContextMenu;
@@ -25,7 +23,6 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.UUID;
 
-import nodomain.freeyourgadget.gadgetbridge.GBApplication;
 import nodomain.freeyourgadget.gadgetbridge.R;
 import nodomain.freeyourgadget.gadgetbridge.adapter.GBDeviceAppAdapter;
 import nodomain.freeyourgadget.gadgetbridge.impl.GBDeviceApp;
@@ -105,14 +102,10 @@ public class AppManagerActivity extends Activity {
 
         registerForContextMenu(appListView);
 
-        SharedPreferences sharedPrefs = PreferenceManager.getDefaultSharedPreferences(GBApplication.getContext());
-
-        if (sharedPrefs.getBoolean("pebble_force_untested", false)) {
         List<GBDeviceApp> cachedApps = getCachedApps();
         for (GBDeviceApp app : cachedApps) {
             appList.add(app);
         }
-        }
 
         IntentFilter filter = new IntentFilter();
         filter.addAction(ControlCenter.ACTION_QUIT);

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/FwAppInstallerActivity.java

@@ -101,7 +101,7 @@ public class FwAppInstallerActivity extends Activity implements InstallActivity
             @Override
             public void onClick(View v) {
                 setInstallEnabled(false);
-                installHandler.onStartInstall();
+                installHandler.onStartInstall(device);
                 Intent startIntent = new Intent(FwAppInstallerActivity.this, DeviceCommunicationService.class);
                 startIntent.setAction(DeviceCommunicationService.ACTION_INSTALL);
                 startIntent.putExtra("uri", uri);

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/InstallHandler.java

@@ -19,7 +19,7 @@ public interface InstallHandler {
      * Checks whether the installation of the 'element' on the device is possible
      * and configures the InstallActivity accordingly (sets helpful texts,
      * enables/disables the "Install" button, etc.
-     *
+     * <p/>
      * Note: may only be called if #isValid previously returned true.
      *
      * @param installActivity the activity to interact with
@@ -30,5 +30,5 @@ public interface InstallHandler {
     /**
      * Allows device specivic code to be execute just before the installation starts
      */
-    void onStartInstall();
+    void onStartInstall(GBDevice device);
 }

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/miband/MiBandFWInstallHandler.java

@@ -53,7 +53,7 @@ public class MiBandFWInstallHandler implements InstallHandler {
     }
 
     @Override
-    public void onStartInstall() {
+    public void onStartInstall(GBDevice device) {
 
     }
 

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/pebble/PBWInstallHandler.java

@@ -1,14 +1,11 @@
 package nodomain.freeyourgadget.gadgetbridge.devices.pebble;
 
 import android.content.Context;
-import android.content.SharedPreferences;
 import android.net.Uri;
-import android.preference.PreferenceManager;
 
 import java.io.File;
 import java.io.IOException;
 
-import nodomain.freeyourgadget.gadgetbridge.GBApplication;
 import nodomain.freeyourgadget.gadgetbridge.R;
 import nodomain.freeyourgadget.gadgetbridge.activities.InstallActivity;
 import nodomain.freeyourgadget.gadgetbridge.devices.InstallHandler;
@@ -59,13 +56,12 @@ public class PBWInstallHandler implements InstallHandler {
     }
 
     @Override
-    public void onStartInstall() {
+    public void onStartInstall(GBDevice device) {
         if (mPBWReader.isFirmware()) {
             return;
         }
 
-        SharedPreferences sharedPrefs = PreferenceManager.getDefaultSharedPreferences(GBApplication.getContext());
+        if (!device.getFirmwareVersion().startsWith("v3")) {
-        if (!sharedPrefs.getBoolean("pebble_force_untested", false)) {
             return;
         }
 

app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/pebble/PebbleIoThread.java

@@ -45,7 +45,6 @@ public class PebbleIoThread extends GBDeviceIoThread {
     private boolean mIsConnected = false;
     private boolean mIsInstalling = false;
     private int mConnectionAttempts = 0;
-    private boolean mForceUntested = false;
     private PBWReader mPBWReader = null;
     private int mAppInstallToken = -1;
     private ZipInputStream mZis = null;
@@ -86,7 +85,6 @@ public class PebbleIoThread extends GBDeviceIoThread {
 
         SharedPreferences sharedPrefs = PreferenceManager.getDefaultSharedPreferences(getContext());
         mPebbleProtocol.setForceProtocol(sharedPrefs.getBoolean("pebble_force_protocol", false));
-        mForceUntested = sharedPrefs.getBoolean("pebble_force_untested", false);
         gbDevice.setState(GBDevice.State.CONNECTED);
         gbDevice.sendDeviceUpdateIntent(getContext());
 
@@ -409,7 +407,7 @@ public class PebbleIoThread extends GBDeviceIoThread {
             writeInstallApp(mPebbleProtocol.encodeGetTime());
         } else {
             GBDeviceApp app = mPBWReader.getGBDeviceApp();
-            if (mPebbleProtocol.isFw3x && mForceUntested) {
+            if (mPebbleProtocol.isFw3x) {
                 if (appId == 0) {
                     // only install metadata - not the binaries
                     write(mPebbleProtocol.encodeInstallMetadata(app.getUUID(), app.getName(), mPBWReader.getAppVersion(), mPBWReader.getSdkVersion(), mPBWReader.getIconId()));