[fix][mediacontroller] fix error with new metadata features 92/214992/1
authorDawid Juszczak <d.juszczak@samsung.com>
Mon, 30 Sep 2019 14:01:35 +0000 (16:01 +0200)
committerDawid Juszczak <d.juszczak@samsung.com>
Mon, 30 Sep 2019 14:01:35 +0000 (16:01 +0200)
Change-Id: I957a41a84d1a8382c060ea615485c0c0f4d18602
Signed-off-by: Dawid Juszczak <d.juszczak@samsung.com>
src/mediacontroller/mediacontroller_utils.cc
src/mediacontroller/mediacontroller_utils.h

index 7c5507e..2af7090 100644 (file)
@@ -332,7 +332,7 @@ ErrorCode utils::ConvertMediaControllerError(int e) {
   return error;
 }
 
-bool isEncodable(std::string name) {
+bool utils::isEncodable(const std::string &name) {
   if (name == "season" || name == "episode" || name == "resolution") {
     return true;
   }
@@ -445,7 +445,8 @@ PlatformResult utils::DecodeMetadata(const picojson::value &to_decode, picojson:
   }
 
   (*decoded)["seasonNumber"] = picojson::value((double)number);
-  (*decoded)["episodeTitle"] = season_title ? picojson::value(std::string(season_title)) : picojson::value();
+  (*decoded)["episodeTitle"] =
+      season_title ? picojson::value(std::string(season_title)) : picojson::value();
 
   // decode episode
   val = to_decode.get("episode").get<std::string>();
@@ -458,7 +459,8 @@ PlatformResult utils::DecodeMetadata(const picojson::value &to_decode, picojson:
   }
 
   (*decoded)["episodeNumber"] = picojson::value((double)number);
-  (*decoded)["episodeTitle"] = episode_title ? picojson::value(std::string(episode_title)) : picojson::value();
+  (*decoded)["episodeTitle"] =
+      episode_title ? picojson::value(std::string(episode_title)) : picojson::value();
 
   // decode resolution
   val = to_decode.get("resolution").get<std::string>();
index 3922c43..47926f4 100644 (file)
@@ -59,7 +59,7 @@ extern const common::PlatformEnum<mc_display_rotation_e> MediaControllerDisplayR
 namespace utils {
 common::PlatformResult GetAllPlaylists(const std::string& app_id, picojson::array* playlists);
 common::ErrorCode ConvertMediaControllerError(int e);
-bool isEncodable(std::string name);
+bool isEncodable(const std::string& name);
 common::PlatformResult EncodeMetadata(const picojson::object& to_encode, picojson::object* encoded);
 common::PlatformResult DecodeMetadata(const picojson::value& to_decode, picojson::object* decoded);
 }  // namespace utils