Merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-imported-folder into lp:ubuntu-docviewer-app

[Stefano Verzegnassi]

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

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

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

PASSED: Continuous integration, rev:334
https://core-apps-jenkins.ubuntu.com/job/docviewer-app-ci/275/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/864/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/docviewer-app-ci/275/rebuild

review: Approve (continuous-integration)

[Alan Pope 🍺🐧🐱 🦄 (popey)]

This doesn't work as expected.

Downloaded PDF from browser. Tap Open when done, get an error on Docviewer, and the file isn't saved. (no new files in ~/Documents on device)

Here's a video showing it.

https://youtu.be/R30dq-NeGe0

review: Needs Fixing

[Stefano Verzegnassi (verzegnassi-stefano)]

That happens also with the release currently available on the store (i.e. does not depend on this MP). If docviewer is already launched, everything works as expected.

In the log is there anything about Libertine containers? I started to see this problem when content-hub has added the support for xmir apps.
It happens only with docviewer though, I'll have a look (I suspect the QML Loader[1] causes the issue).

[1] http://bazaar.launchpad.net/~ubuntu-docviewer-dev/ubuntu-docviewer-app/lo-viewer/view/head:/src/app/qml/ubuntu-docviewer-app.qml#L208

Unmerged revisions

334. By Stefano Verzegnassi on 2016-04-14

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

=== 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