This commit has a dependency of wrt-commons.

Therefore, this commit is reverted.
Please merge wrt-commons commit before this commit.

Revert "Icon extension for package manager and manifest fix"

This reverts commit e11660bf9e6ad7fa8183ae7eecd435763a1bf027.

Change-Id: I08a01cbdecc0f3222bbae8522dd1ef9c29bc79d5

diff --git a/src/jobs/widget_install/task_manifest_file.cpp b/src/jobs/widget_install/task_manifest_file.cpp
index 8308125..37d856e 100644 (file)
--- a/src/jobs/widget_install/task_manifest_file.cpp
+++ b/src/jobs/widget_install/task_manifest_file.cpp
@@ -52,7 +52,6 @@
 #include <libxml_utils.h>
 #include <pkgmgr/pkgmgr_parser.h>
 #include <dpl/localization/LanguageTagsProvider.h>
-#include <dpl/utils/path.h>
 
 #define DEFAULT_ICON_NAME   "icon.png"
 #define DEFAULT_PREVIEW_NAME   "preview.png"
@@ -220,37 +219,53 @@ void TaskManifestFile::stepCopyIconFiles()
             if (std::find(generatedLocales.begin(), generatedLocales.end(),
                     *locale) != generatedLocales.end())
             {
+                if (icon->src == L"icon.jpg") {
+                    generatedLocales.push_back(*locale);
+                } else if (icon->src == L"icon.gif") {
+                    generatedLocales.push_back(*locale);
+                } else if (icon->src == L"icon.png") {
+                    generatedLocales.push_back(*locale);
+                } else if (icon->src == L"icon.ico") {
+                    generatedLocales.push_back(*locale);
+                } else if (icon->src == L"icon.svg") {
+                    generatedLocales.push_back(*locale);
+                }
                 LogDebug("Skipping - has that locale");
                 continue;
             } else {
                 generatedLocales.push_back(*locale);
             }
 
-            DPL::Utils::Path sourceFile(m_context.locations->getSourceDir());
+            std::ostringstream sourceFile;
+            std::ostringstream targetFile;
+
+            sourceFile << m_context.locations->getSourceDir() << "/";
+
             if (!locale->empty()) {
-                sourceFile /= "locales";
-                sourceFile /= *locale;
+                sourceFile << "locales/" << *locale << "/";
             }
-            sourceFile /= src;
 
-            DPL::Utils::Path targetFile(GlobalConfig::GetUserWidgetDesktopIconPath());
-            targetFile /= getIconTargetFilename(*locale, sourceFile.Extension());
+            sourceFile << src;
+
+            targetFile << GlobalConfig::GetUserWidgetDesktopIconPath() << "/";
+            targetFile << getIconTargetFilename(*locale);
 
             if (m_context.widgetConfig.packagingType ==
                 WrtDB::PKG_TYPE_HOSTED_WEB_APP)
             {
                 m_context.locations->setIconTargetFilenameForLocale(
-                    targetFile.Fullpath());
+                    targetFile.str());
             }
 
-            LogDebug("Copying icon: " << sourceFile << " -> " << targetFile);
+            LogDebug("Copying icon: " << sourceFile.str() <<
+                     " -> " << targetFile.str());
 
-            icon_list.push_back(targetFile.Fullpath());
+            icon_list.push_back(targetFile.str());
 
             Try
             {
-                DPL::FileInput input(sourceFile.Fullpath());
-                DPL::FileOutput output(targetFile.Fullpath());
+                DPL::FileInput input(sourceFile.str());
+                DPL::FileOutput output(targetFile.str());
                 DPL::Copy(&input, &output);
             }
 
@@ -427,7 +442,7 @@ void TaskManifestFile::stepAbortIconFiles()
 }
 
 DPL::String TaskManifestFile::getIconTargetFilename(
-    const DPL::String& languageTag, const std::string & ext) const
+    const DPL::String& languageTag) const
 {
     DPL::OStringStream filename;
     TizenAppId appid = m_context.widgetConfig.tzAppid;
@@ -450,10 +465,7 @@ DPL::String TaskManifestFile::getIconTargetFilename(
         }
     }
 
-    if(!ext.empty())
-    {
-        filename << L"." + DPL::FromUTF8String(ext);
-    }
+    filename << L".png";
     return filename.str();
 }
 
@@ -814,13 +826,12 @@ void TaskManifestFile::setWidgetIcons(UiApplication & uiApp)
                 tag = *locale;
             }
 
-            generateWidgetIcon(uiApp, tag, *locale, DPL::Utils::Path(icon->src).Extension(), defaultIconSaved);
+            generateWidgetIcon(uiApp, tag, *locale, defaultIconSaved);
         }
     }
     if (!!defaultLocale && !defaultIconSaved) {
         generateWidgetIcon(uiApp, DPL::OptionalString::Null,
                            DPL::String(),
-                           std::string(),
                            defaultIconSaved);
     }
 }
@@ -828,7 +839,6 @@ void TaskManifestFile::setWidgetIcons(UiApplication & uiApp)
 void TaskManifestFile::generateWidgetIcon(UiApplication & uiApp,
                                           const DPL::OptionalString& tag,
                                           const DPL::String& language,
-                                          const std::string & extension,
                                           bool & defaultIconSaved)
 {
     DPL::String locale;
@@ -839,7 +849,7 @@ void TaskManifestFile::generateWidgetIcon(UiApplication & uiApp,
     }
 
     DPL::String iconText;
-    iconText += getIconTargetFilename(language, extension);
+    iconText += getIconTargetFilename(language);
 
     if (!locale.empty()) {
         uiApp.addIcon(IconType(iconText, locale));
@@ -848,8 +858,8 @@ void TaskManifestFile::generateWidgetIcon(UiApplication & uiApp,
     }
     std::ostringstream iconPath;
     iconPath << GlobalConfig::GetUserWidgetDesktopIconPath() << "/";
-    iconPath << getIconTargetFilename(locale, extension);
-    m_context.job->SendProgressIconPath(iconPath.str());
+    iconPath << getIconTargetFilename(locale);
+     m_context.job->SendProgressIconPath(iconPath.str());
 }
 
 void TaskManifestFile::setWidgetDescription(Manifest & manifest)

diff --git a/src/jobs/widget_install/task_manifest_file.h b/src/jobs/widget_install/task_manifest_file.h
index 98c3d23..a318fc8 100644 (file)
--- a/src/jobs/widget_install/task_manifest_file.h
+++ b/src/jobs/widget_install/task_manifest_file.h
@@ -118,7 +118,7 @@ class TaskManifestFile :
                                    DPL::OptionalString description);
     void generateWidgetIcon(UiApplication & uiApp,
                             const DPL::OptionalString& tag,
-                            const DPL::String& language, const std::string &extension,
+                            const DPL::String& language,
                             bool & defaultIconSaved);
     void DynamicBoxFileCopy(const std::string& sourceFile,
                             const std::string& targetFile);
@@ -126,8 +126,7 @@ class TaskManifestFile :
     //for widget update
     void backupIconFiles();
     void getFileList(const char* path, std::list<std::string> &list);
-    DPL::String getIconTargetFilename(const DPL::String& languageTag,
-                                      const std::string & ext) const;
+    DPL::String getIconTargetFilename(const DPL::String& languageTag) const;
 
     static void saveLocalizedKey(std::ofstream &file,
                                  const DPL::String& key,

diff --git a/src/jobs/widget_install/task_widget_config.cpp b/src/jobs/widget_install/task_widget_config.cpp
index c837279..221fd1c 100644 (file)
--- a/src/jobs/widget_install/task_widget_config.cpp
+++ b/src/jobs/widget_install/task_widget_config.cpp
@@ -293,7 +293,7 @@ void TaskWidgetConfig::ProcessBackgroundPageFile()
 void TaskWidgetConfig::ProcessLocalizedIcons()
 {
     using namespace WrtDB;
-    FOREACH(i, m_installContext.widgetConfig.configInfo.iconsList)
+        FOREACH(i, m_installContext.widgetConfig.configInfo.iconsList)
     {
         ProcessIcon(*i);
     }