caspervk/AndroidAPS
1
0
Fork 0
Code Releases Activity

Return false in isConnected and isConnecting when connection lock is not held.

Browse source
This commit is contained in:
TebbeUbben 2019-02-09 22:30:41 +01:00
parent 317f171286
commit 7a7bebc5fe
2 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/src/main/java/info/nightscout/androidaps/plugins/PumpInsightLocal/LocalInsightPlugin.java
View file

@ -261,13 +261,15 @@ public class LocalInsightPlugin extends PluginBase implements PumpInterface, Con
@Override
public boolean isConnected() {
if (connectionService == null || alertService == null) return false;
return connectionService.getState() == InsightState.CONNECTED;
return connectionService != null
&& alertService != null
&& connectionService.hasRequestedConnection(this)
&& connectionService.getState() == InsightState.CONNECTED;
}
@Override
public boolean isConnecting() {
if (connectionService == null || alertService == null) return false;
if (connectionService == null || alertService == null || !connectionService.hasRequestedConnection(this)) return false;
InsightState state = connectionService.getState();
return state == InsightState.CONNECTING
|| state == InsightState.APP_CONNECT_MESSAGE

4
app/src/main/java/info/nightscout/androidaps/plugins/PumpInsightLocal/connection_service/InsightConnectionService.java
View file

@ -277,6 +277,10 @@ public class InsightConnectionService extends Service implements ConnectionEstab
}
}
public synchronized boolean hasRequestedConnection(Object lock) {
return connectionRequests.contains(lock);
}
private void cleanup() {
messageQueue.completeActiveRequest(new ConnectionLostException());
messageQueue.completePendingRequests(new ConnectionLostException());