Review Request 117915: Fetch photo-taken Date/Time for Images and file created Date/Time for other files
Shantanu Tushar
shantanu at kde.org
Thu May 1 06:09:31 UTC 2014
> On May 1, 2014, 6:06 a.m., Commit Hook wrote:
> > This review has been submitted with commit 121e51d74ff50a3644f275d5cfb571326c7a7286 by Shantanu Tushar to branch master.
Had pushed this by mistake when trying to fix a bug, reverted it. The review is still open.
- Shantanu
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117915/#review57044
-----------------------------------------------------------
On May 1, 2014, 6:06 a.m., Shantanu Tushar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117915/
> -----------------------------------------------------------
>
> (Updated May 1, 2014, 6:06 a.m.)
>
>
> Review request for Plasma and Vishesh Handa.
>
>
> Repository: plasma-mediacenter
>
>
> Description
> -------
>
> Just like the Nepomuk media source, the Baloo media source needs to provide the date/time information for media. This is used to sort the media to show the more recent media first.
> For images, the date/time when the photo was actually taken is used, and the file creation date/time is used for other media.
>
>
> Diffs
> -----
>
> plugins/baloosearch/CMakeLists.txt 1ff81fb
> plugins/baloosearch/audiosearchresulthandler.h PRE-CREATION
> plugins/baloosearch/audiosearchresulthandler.cpp PRE-CREATION
> plugins/baloosearch/baloosearchmediasource.h e315de4
> plugins/baloosearch/baloosearchmediasource.cpp 7ebfa61
> plugins/baloosearch/imagesearchresulthandler.h PRE-CREATION
> plugins/baloosearch/imagesearchresulthandler.cpp PRE-CREATION
> plugins/baloosearch/searchresulthandler.h PRE-CREATION
> plugins/baloosearch/searchresulthandler.cpp PRE-CREATION
> plugins/baloosearch/videosearchresulthandler.h PRE-CREATION
> plugins/baloosearch/videosearchresulthandler.cpp PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/117915/diff/
>
>
> Testing
> -------
>
> Tested with all three types of media, works fine. Unit tests for the new code to follow.
>
>
> Thanks,
>
> Shantanu Tushar
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140501/e7c12d5d/attachment.html>
More information about the Plasma-devel
mailing list