summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBilal Alsharifi <bilal.alsharifi@gmail.com>2020-02-19 16:44:32 -0500
committerBilal Alsharifi <bilal.alsharifi@gmail.com>2020-02-19 16:44:32 -0500
commitb239e724604fceb93581aa4bae1bb35bf83880c4 (patch)
tree6acb70aafb5f845260c91dd0eeafb2954f45d7c8
parent9ec33af2d4c5eef78ea1c6c4b3defaf801af6815 (diff)
downloadsdl_android-b239e724604fceb93581aa4bae1bb35bf83880c4.tar.gz
Update comments in retrieveCapability()
-rw-r--r--base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java8
1 files changed, 7 insertions, 1 deletions
diff --git a/base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java b/base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java
index 345a8da6e..149fd3960 100644
--- a/base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java
+++ b/base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java
@@ -550,7 +550,13 @@ public class SystemCapabilityManager {
}
final GetSystemCapability request = new GetSystemCapability();
request.setSystemCapabilityType(systemCapabilityType);
- // If subscribe is null, then don't change the current subscription status
+
+ /*
+ The subscription flag in the request should be set based on multiple variables:
+ - if subscribe is null (no change), shouldSubscribe = current subscription status
+ - if subscribe is false, then shouldSubscribe = false
+ - if subscribe is true and the HU supports subscriptions, then shouldSubscribe = true
+ */
final boolean shouldSubscribe = ((subscribe != null) ? subscribe : isSubscribedToSystemCapability(systemCapabilityType)) && supportsSubscriptions();
request.setSubscribe(shouldSubscribe);
request.setOnRPCResponseListener(new OnRPCResponseListener() {