Merge lp:~fboucault/qtubuntu-camera/save_videos_subfolder into lp:qtubuntu-camera

Proposed by Florian Boucault
Status: Merged
Approved by: Jim Hodapp
Approved revision: 101
Merged at revision: 102
Proposed branch: lp:~fboucault/qtubuntu-camera/save_videos_subfolder
Merge into: lp:qtubuntu-camera
Diff against target: 29 lines (+3/-2)
1 file modified
src/storagemanager.cpp (+3/-2)
To merge this branch: bzr merge lp:~fboucault/qtubuntu-camera/save_videos_subfolder
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+228741@code.launchpad.net

Commit message

Save pictures and videos in a subfolder of the standard paths that depends on the application name.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Looks good, thanks!

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/storagemanager.cpp'
2--- src/storagemanager.cpp 2013-05-24 11:25:15 +0000
3+++ src/storagemanager.cpp 2014-07-29 18:11:44 +0000
4@@ -21,6 +21,7 @@
5 #include <QFile>
6 #include <QFileInfo>
7 #include <QStandardPaths>
8+#include <QCoreApplication>
9
10 const QLatin1String photoBase = QLatin1String("image");
11 const QLatin1String videoBase = QLatin1String("video");
12@@ -36,7 +37,7 @@
13 {
14 m_directory = directoy;
15 if (m_directory.isEmpty())
16- m_directory = QStandardPaths::writableLocation(QStandardPaths::PicturesLocation);
17+ m_directory = QStandardPaths::writableLocation(QStandardPaths::PicturesLocation) + "/" + QCoreApplication::applicationName();
18
19 return nextMediaFileName(photoBase, photoExtension);
20 }
21@@ -45,7 +46,7 @@
22 {
23 m_directory = directoy;
24 if (m_directory.isEmpty())
25- m_directory = QStandardPaths::writableLocation(QStandardPaths::MoviesLocation);
26+ m_directory = QStandardPaths::writableLocation(QStandardPaths::MoviesLocation) + "/" + QCoreApplication::applicationName();
27
28 return nextMediaFileName(videoBase, videoExtension);
29 }

Subscribers

People subscribed via source and target branches