ubuntu-touch-coreapps-reviewers team mailing list archive

[Stefano Verzegnassi <stefano92.100@xxxxxxxxx>]

Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-imported-folder into lp:ubuntu-docviewer-app.

Commit message:
Save imported files from content-hub in a separate folder.

Requested reviews:
  Ubuntu Document Viewer Developers (ubuntu-docviewer-dev)
Related bugs:
  Bug #1570507 in Ubuntu Document Viewer App: "Imported files from content-hub should be saved in a specific folder"
  https://bugs.launchpad.net/ubuntu-docviewer-app/+bug/1570507

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-imported-folder/+merge/291923

Save imported files from content-hub in a separate folder.
-- 
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-imported-folder into lp:ubuntu-docviewer-app.

=== modified file 'src/plugin/file-qml-plugin/docviewerutils.cpp'
--- src/plugin/file-qml-plugin/docviewerutils.cpp	2015-12-27 12:10:06 +0000
+++ src/plugin/file-qml-plugin/docviewerutils.cpp	2016-04-14 17:29:33 +0000
@@ -91,7 +91,7 @@
     if (suffix.isEmpty())
         suffix = mt.preferredSuffix();
 
-    QString dir = QStandardPaths::writableLocation(QStandardPaths::DocumentsLocation) + QDir::separator();
+    QString dir = QStandardPaths::writableLocation(QStandardPaths::DocumentsLocation) + QDir::separator() + "Imported" + QDir::separator();
     QString destination = QString("%1.%2").arg(dir + filenameWithoutSuffix, suffix);
 
     // If there's already a file of this name, reformat it to