From c941f8937d1da35b65e99126856664d1d96feec6 Mon Sep 17 00:00:00 2001 From: Piotr Kosko Date: Mon, 27 Jun 2016 10:56:23 +0200 Subject: [PATCH] [Preference] Additional check for unsetChangeListener NotFoundErr [Feature] Handling NotFoundErr was added in unsetChangeListener() [Verification] TCT passrate is 97.73% (1 fail caused by TCT issue) Change-Id: I48be34dbac09ccfd119da2ff400911b7e8e4c0f5 Signed-off-by: Piotr Kosko --- src/preference/preference_manager.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/preference/preference_manager.cc b/src/preference/preference_manager.cc index b9760040..2461f3fb 100644 --- a/src/preference/preference_manager.cc +++ b/src/preference/preference_manager.cc @@ -285,14 +285,13 @@ common::TizenResult PreferenceManager::UnsetChangeListener(const std::string& ke if (PREFERENCE_ERROR_NONE != ret) { return MakeErrResult(ret, "preference_unset_changed_cb function error"); } - key_listeners_.erase(it); - break; + return common::TizenSuccess(); } } } - - return common::TizenSuccess(); + // key was not found + return common::NotFoundError(std::string("Listener for given key was not found")); } } // namespace preference -- 2.34.1