From 696611d392348825adbbbf2b4cb16cf40c7489e3 Mon Sep 17 00:00:00 2001
From: Andreas Shimokawa <shimokawa@fsfe.org>
Date: Tue, 20 Sep 2016 22:02:40 +0200
Subject: [PATCH] Do not show "your activity" if activity tracking is not
 supported

---
 .../gadgetbridge/activities/ControlCenter.java            | 3 +++
 .../gadgetbridge/devices/DeviceCoordinator.java           | 8 ++++++++
 .../gadgetbridge/devices/UnknownDeviceCoordinator.java    | 5 +++++
 .../gadgetbridge/devices/miband/MiBandCoordinator.java    | 5 +++++
 .../gadgetbridge/devices/pebble/PebbleCoordinator.java    | 5 +++++
 .../devices/vibratissimo/VibratissimoCoordinator.java     | 5 +++++
 6 files changed, 31 insertions(+)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/ControlCenter.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/ControlCenter.java
index 93b20dd2..ac421c0c 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/ControlCenter.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/ControlCenter.java
@@ -197,6 +197,9 @@ public class ControlCenter extends GBActivity {
         if (!coordinator.supportsAlarmConfiguration()) {
             menu.removeItem(R.id.controlcenter_configure_alarms);
         }
+        if (!coordinator.supportsActivityTracking()) {
+            menu.removeItem(R.id.controlcenter_start_sleepmonitor);
+        }
 
         if (selectedDevice.getState() == GBDevice.State.NOT_CONNECTED) {
             menu.removeItem(R.id.controlcenter_disconnect);
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/DeviceCoordinator.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/DeviceCoordinator.java
index 482ae9ae..349315cb 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/DeviceCoordinator.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/DeviceCoordinator.java
@@ -71,6 +71,14 @@ public interface DeviceCoordinator {
      */
     boolean supportsActivityDataFetching();
 
+    /**
+     * Returns true if activity tracking is supported by the device
+     * (with this coordinator).
+     *
+     * @return
+     */
+    boolean supportsActivityTracking();
+
     /**
      * Returns true if activity data fetching is supported AND possible at this
      * very moment. This will consider the device state (being connected/disconnected/busy...)
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/UnknownDeviceCoordinator.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/UnknownDeviceCoordinator.java
index 9116784f..389eab73 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/UnknownDeviceCoordinator.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/UnknownDeviceCoordinator.java
@@ -106,6 +106,11 @@ public class UnknownDeviceCoordinator extends AbstractDeviceCoordinator {
         return false;
     }
 
+    @Override
+    public boolean supportsActivityTracking() {
+        return false;
+    }
+
     @Override
     public boolean supportsScreenshots() {
         return false;
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/miband/MiBandCoordinator.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/miband/MiBandCoordinator.java
index 7ab22407..c91e4a17 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/miband/MiBandCoordinator.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/miband/MiBandCoordinator.java
@@ -93,6 +93,11 @@ public class MiBandCoordinator extends AbstractDeviceCoordinator {
         return true;
     }
 
+    @Override
+    public boolean supportsActivityTracking() {
+        return true;
+    }
+
     @Override
     public int getTapString() {
         return R.string.tap_connected_device_for_activity;
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/pebble/PebbleCoordinator.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/pebble/PebbleCoordinator.java
index 35054631..7cedf025 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/pebble/PebbleCoordinator.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/pebble/PebbleCoordinator.java
@@ -68,6 +68,11 @@ public class PebbleCoordinator extends AbstractDeviceCoordinator {
         return false;
     }
 
+    @Override
+    public boolean supportsActivityTracking() {
+        return true;
+    }
+
     @Override
     public boolean supportsScreenshots() {
         return true;
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/vibratissimo/VibratissimoCoordinator.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/vibratissimo/VibratissimoCoordinator.java
index 5bbaf23c..84029a32 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/vibratissimo/VibratissimoCoordinator.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/vibratissimo/VibratissimoCoordinator.java
@@ -47,6 +47,11 @@ public class VibratissimoCoordinator extends AbstractDeviceCoordinator {
         return false;
     }
 
+    @Override
+    public boolean supportsActivityTracking() {
+        return false;
+    }
+
     @Override
     public SampleProvider<? extends ActivitySample> getSampleProvider(GBDevice device, DaoSession session) {
         return null;