From d4d800adf927df80e1f84c5e1c31d0ffb7c23de1 Mon Sep 17 00:00:00 2001 From: Dawid Juszczak Date: Mon, 23 Sep 2019 11:58:34 +0200 Subject: [PATCH] [mediacontroller][fix] fixed error with subscribe method before that commit, if subscribed method was executed on already subscribed server then unknown error was thrown Now it is fixed. [verification] tested manually on chrome console Change-Id: I7d5fe3d0d09eb95d6542ebfb2e163ca32c5d01a0 Signed-off-by: Dawid Juszczak --- src/mediacontroller/mediacontroller_client.cc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/mediacontroller/mediacontroller_client.cc b/src/mediacontroller/mediacontroller_client.cc index 9f773509..0fee47b6 100644 --- a/src/mediacontroller/mediacontroller_client.cc +++ b/src/mediacontroller/mediacontroller_client.cc @@ -705,6 +705,12 @@ PlatformResult MediaControllerClient::SubscribeServer(const std::string& server_ "Unable to subscribe server, AbilityChangeListener is not set"); } + auto srv = std::find(subscribed_servers.begin(), subscribed_servers.end(), server_name); + if (srv != subscribed_servers.end()) { + LoggerD("server %s is already subscribed", server_name.c_str()); + return PlatformResult(ErrorCode::NO_ERROR); + } + SCOPE_EXIT { if (to_unsubscribe > 0) { for (auto type : types::MediaControllerSubscriptionTypeEnum) { -- 2.34.1