[Content] Adding comments in code and adding deprecation log 10/198310/2
authorSzymon Jastrzebski <s.jastrzebsk@partner.samsung.com>
Wed, 23 Jan 2019 14:06:31 +0000 (15:06 +0100)
committerMichal Michalski <m.michalski2@partner.samsung.com>
Thu, 28 Feb 2019 08:10:50 +0000 (08:10 +0000)
http://suprem.sec.samsung.net/jira/browse/ACR-1249

[Verification] Content TCT 100%

Change-Id: Ie3c2450e46c343b2db350475b0ec679d485e2d70
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>
src/content/content_manager.cc
src/content/js/datatypes.js

index e3c2ce2..782f2a3 100644 (file)
@@ -398,6 +398,10 @@ void ContentDirToJson(media_folder_h folder, picojson::object& o) {
 
   // storageType
   ret = media_folder_get_storage_type(folder, &storage_type);
+  // TODO: The function media_folder_get_storage_type is marked as deprecated since 5.0.
+  // As an alternative, it is recommended to use storage_get_type_dev function. However,
+  // this function does not work with internal storages. The media_folder_get_storage_type
+  // function should be kept or moved to internal header by Native API.
   if (ret == MEDIA_CONTENT_ERROR_NONE) {
     if (storage_type == MEDIA_CONTENT_STORAGE_INTERNAL) {
       o["storageType"] = picojson::value(std::string("INTERNAL"));
@@ -541,6 +545,10 @@ static void FolderToJson(media_folder_h folder, picojson::object* out) {
   date = stat_res.st_mtime;
 
   ret = media_folder_get_storage_type(folder, &storageType);
+  // TODO: The function media_folder_get_storage_type is marked as deprecated since 5.0.
+  // As an alternative, it is recommended to use storage_get_type_dev function. However,
+  // this function does not work with internal storages. The media_folder_get_storage_type
+  // function should be kept or moved to internal header by Native API.
   if (ret != MEDIA_CONTENT_ERROR_NONE) {
     LogAndReportError(ContentManager::convertError(ret), out,
                       ("Failed: media_folder_get_storage_type"));
index f632ded..60b95cc 100755 (executable)
@@ -85,6 +85,7 @@ function ContentDirectory(data) {
     },
     storageType: {
       get: function() {
+        utils_.printDeprecationWarningFor('ContentDirectoryStorageType');
         return storageType;
       },
       set: function(v) {