Merge pull request #124 from AAPS-Omnipod/omnipod_eros_dev_upstream_merge

Omnipod updates

app/src/main/java/info/nightscout/androidaps/plugins/pump/combo/ComboPlugin.java

@@ -338,7 +338,7 @@ public class ComboPlugin extends PumpPluginBase implements PumpInterface, Constr
      * Runs pump initialization if needed and reads the pump state from the main screen.
      */
     @Override
-    public synchronized void getPumpStatus() {
+    public synchronized void getPumpStatus(String reason) {
         getAapsLogger().debug(LTag.PUMP, "getPumpStatus called");
         if (!pump.initialized) {
             initializePump();

app/src/main/java/info/nightscout/androidaps/plugins/pump/insight/LocalInsightPlugin.java

@@ -370,7 +370,7 @@ public class LocalInsightPlugin extends PumpPluginBase implements PumpInterface,
     }
 
     @Override
-    public void getPumpStatus() {
+    public void getPumpStatus(String reason) {
         try {
             tbrOverNotificationBlock = ParameterBlockUtil.readParameterBlock(connectionService, Service.CONFIGURATION, TBROverNotificationBlock.class);
             readHistory();

app/src/main/java/info/nightscout/androidaps/plugins/pump/mdi/MDIPlugin.java

@@ -132,7 +132,7 @@ public class MDIPlugin extends PumpPluginBase implements PumpInterface {
     }
 
     @Override
-    public void getPumpStatus() {
+    public void getPumpStatus(String reason) {
     }
 
     @NonNull @Override

app/src/main/java/info/nightscout/androidaps/plugins/pump/virtual/VirtualPumpPlugin.kt

@@ -178,7 +178,7 @@ class VirtualPumpPlugin @Inject constructor(
 
     override fun disconnect(reason: String) {}
     override fun stopConnecting() {}
-    override fun getPumpStatus() {
+    override fun getPumpStatus(reason: String?) {
         lastDataTime = System.currentTimeMillis()
     }
 

app/src/main/java/info/nightscout/androidaps/queue/commands/CommandReadStatus.kt

@@ -11,7 +11,7 @@ import javax.inject.Inject
 
 class CommandReadStatus(
     injector: HasAndroidInjector,
-    private val reason: String,
+    val reason: String,
     callback: Callback?
 ) : Command(injector, CommandType.READSTATUS, callback) {
 
@@ -19,7 +19,7 @@ class CommandReadStatus(
     @Inject lateinit var localAlertUtils: LocalAlertUtils
 
     override fun execute() {
-        activePlugin.activePump.getPumpStatus()
+        activePlugin.activePump.getPumpStatus(reason)
         localAlertUtils.notifyPumpStatusRead()
         aapsLogger.debug(LTag.PUMPQUEUE, "CommandReadStatus executed. Reason: $reason")
         val pump = activePlugin.activePump

core/src/main/java/info/nightscout/androidaps/interfaces/PumpInterface.java

@@ -43,7 +43,7 @@ public interface PumpInterface {
 
     void stopConnecting();
 
-    void getPumpStatus();
+    void getPumpStatus(String reason);
 
     // Upload to pump new basal profile
     @NotNull

danar/src/main/java/info/nightscout/androidaps/danar/AbstractDanaRPlugin.java

@@ -361,7 +361,7 @@ public abstract class AbstractDanaRPlugin extends PumpPluginBase implements Pump
     }
 
     @Override
-    public void getPumpStatus() {
+    public void getPumpStatus(String reason) {
         if (sExecutionService != null) {
             sExecutionService.getPumpStatus();
             pumpDescription.basalStep = danaPump.getBasalStep();

danars/src/main/java/info/nightscout/androidaps/danars/DanaRSPlugin.kt

@@ -165,7 +165,7 @@ class DanaRSPlugin @Inject constructor(
         danaRSService?.stopConnecting()
     }
 
-    override fun getPumpStatus() {
+    override fun getPumpStatus(reason: String?) {
         danaRSService?.readPumpStatus()
         pumpDesc.basalStep = danaPump.basalStep
         pumpDesc.bolusStep = danaPump.bolusStep

medtronic/src/main/java/info/nightscout/androidaps/plugins/pump/medtronic/MedtronicPumpPlugin.java

@@ -421,7 +421,7 @@ public class MedtronicPumpPlugin extends PumpPluginAbstract implements PumpInter
 
 
     @Override
-    public void getPumpStatus() {
+    public void getPumpStatus(String reason) {
 
         if (firstRun) {
             initializePump(!isRefresh);

omnipod/src/main/java/info/nightscout/androidaps/plugins/pump/omnipod/OmnipodPumpPlugin.java

@@ -537,18 +537,24 @@ public class OmnipodPumpPlugin extends PumpPluginBase implements PumpInterface,
 
     /**
      * We don't do periodical status requests because that could drain the Pod's battery
-     * The only actual status requests we send to the Pod here are on startup (in {@link #initializeAfterRileyLinkConnection() initializeAfterRileyLinkConnection()})
+     * The only actual status requests we send to the Pod here are on startup (in {@link #initializeAfterRileyLinkConnection() initializeAfterRileyLinkConnection()}),
+     * When explicitly requested through SMS commands
      * And when the basal and/or temp basal status is uncertain
      * When the user explicitly requested it by clicking the Refresh button on the Omnipod tab (which is executed through {@link #executeCustomCommand(CustomCommand)})
      */
     @Override
-    public void getPumpStatus() {
+    public void getPumpStatus(String reason) {
         if (firstRun) {
             initializeAfterRileyLinkConnection();
             firstRun = false;
-        } else if (!podStateManager.isBasalCertain() || !podStateManager.isTempBasalCertain()) {
-            aapsLogger.info(LTag.PUMP, "Acknowledged AAPS getPumpStatus request because basal and/or temp basal is uncertain");
-            getPodStatus();
+        } else {
+            if ("SMS".equals(reason)) {
+                aapsLogger.info(LTag.PUMP, "Acknowledged AAPS getPumpStatus request it was requested through an SMS");
+                getPodStatus();
+            } else if (!podStateManager.isBasalCertain() || !podStateManager.isTempBasalCertain()) {
+                aapsLogger.info(LTag.PUMP, "Acknowledged AAPS getPumpStatus request because basal and/or temp basal is uncertain");
+                getPodStatus();
+            }
         }
     }