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
=== modified file 'plugins/ScreenGrabber/screengrabber.cpp'
--- plugins/ScreenGrabber/screengrabber.cpp 2014-10-15 15:26:04 +0000
+++ plugins/ScreenGrabber/screengrabber.cpp 2015-03-25 06:43:51 +0000
@@ -81,7 +81,7 @@
81QString ScreenGrabber::makeFileName()81QString ScreenGrabber::makeFileName()
82{82{
83 QString fileName(fileNamePrefix);83 QString fileName(fileNamePrefix);
84 fileName.append(QDateTime::currentDateTime().toString("yyyymmdd_hhmmsszzz"));84 fileName.append(QDateTime::currentDateTime().toString("yyyyMMdd_hhmmsszzz"));
85 fileName.append(".");85 fileName.append(".");
86 fileName.append(getFormat());86 fileName.append(getFormat());
87 return fileName;87 return fileName;

Subscribers

People subscribed via source and target branches