diff options
author | Bilal Alsharifi <bilal.alsharifi@gmail.com> | 2020-02-19 16:44:32 -0500 |
---|---|---|
committer | Bilal Alsharifi <bilal.alsharifi@gmail.com> | 2020-02-19 16:44:32 -0500 |
commit | b239e724604fceb93581aa4bae1bb35bf83880c4 (patch) | |
tree | 6acb70aafb5f845260c91dd0eeafb2954f45d7c8 | |
parent | 9ec33af2d4c5eef78ea1c6c4b3defaf801af6815 (diff) | |
download | sdl_android-b239e724604fceb93581aa4bae1bb35bf83880c4.tar.gz |
Update comments in retrieveCapability()
-rw-r--r-- | base/src/main/java/com/smartdevicelink/proxy/SystemCapabilityManager.java | 8 |
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() { |