Merge lp:~feng-kylin/unity8/fixlp1436006 into lp:unity8

Proposed by handsome_feng
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 1699
Merged at revision: 1719
Proposed branch: lp:~feng-kylin/unity8/fixlp1436006
Merge into: lp:unity8
Diff against target: 12 lines (+1/-1)
1 file modified
plugins/ScreenGrabber/screengrabber.cpp (+1/-1)
To merge this branch: bzr merge lp:~feng-kylin/unity8/fixlp1436006
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
Review via email: mp+254036@code.launchpad.net

Commit message

Modified the wrong time format in ScreenGrabber

Description of the change

Modified the wrong time format in ScreenGrabber to fix the bug lp:1436006.

To post a comment you must log in.
Revision history for this message
Albert Astals Cid (aacid) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass? If not, please explain why.
Waiting before top approval

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve
Revision history for this message
Albert Astals Cid (aacid) wrote :

Meh, CI didn't run for some reason, anyway the fix is obvious enough.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/ScreenGrabber/screengrabber.cpp'
2--- plugins/ScreenGrabber/screengrabber.cpp 2014-10-15 15:26:04 +0000
3+++ plugins/ScreenGrabber/screengrabber.cpp 2015-03-25 06:43:51 +0000
4@@ -81,7 +81,7 @@
5 QString ScreenGrabber::makeFileName()
6 {
7 QString fileName(fileNamePrefix);
8- fileName.append(QDateTime::currentDateTime().toString("yyyymmdd_hhmmsszzz"));
9+ fileName.append(QDateTime::currentDateTime().toString("yyyyMMdd_hhmmsszzz"));
10 fileName.append(".");
11 fileName.append(getFormat());
12 return fileName;

Subscribers

People subscribed via source and target branches