Bug 321198

Summary:	search by time does not work
Product:	[Applications] dolphin	Reporter:	Dj YB <yehielb>
Component:	search	Assignee:	Vishesh Handa <me>
Status:	RESOLVED FIXED
Severity:	normal	CC:	frank78ac, me, nepomuk-bugs, rob.dyck
Priority:	NOR
Version:	16.12.2
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	http://commits.kde.org/kde-baseapps/25a2255eac8be1626258feaa4c74b7f503ad9f9a	Version Fixed In:	4.10.5
Sentry Crash Report:
Attachments:	Fixes the issue

Description Dj YB 2013-06-15 23:53:12 UTC

trying to filter the search using one of the options "Today" "Yesterday" "This week" "This Month" "this year"
produce no results, even though there are results using the place "This Month" for example,

Reproducible: Always

Steps to Reproduce:
1.open dolphin
2.search for something from this year
3.
Actual Results:  
nothing

Expected Results:  
stuff from this year

thanks

Comment 1 Frank Reininghaus 2013-06-17 07:25:26 UTC

Thanks for the bug report. If the files shown by the timeline kioslave and those found by a Nepomuk search are inconsistent, this sounds like a Nepomuk issue. Please correct me if I'm wrong, Vishesh!

Comment 2 Vishesh Handa 2013-06-20 18:53:33 UTC

To confirm - Today, This Week, and This Year do not work. Where as "This Month" works?

Comment 3 Dj YB 2013-06-21 06:51:29 UTC

in the places there are the following places:
Today
Yesterday
This Month
Last Month


those do work.

what is not working is using Ctrl+F and choosing time period.

Comment 4 Vishesh Handa 2013-06-21 19:23:35 UTC

Just tested it out. You're right it doesn't work.

Comment 5 Vishesh Handa 2013-06-25 19:05:19 UTC

Created attachment 80778 [details]
Fixes the issue

@Frank: May I commit?

Comment 6 Frank Reininghaus 2013-06-25 20:05:59 UTC

Thanks for the analysis,Vishesh!

Yes, please commit! Should be fine for the 4.10 branch, I think.

Comment 7 Vishesh Handa 2013-06-25 20:08:54 UTC

Git commit 25a2255eac8be1626258feaa4c74b7f503ad9f9a by Vishesh Handa.
Committed on 25/06/2013 at 18:56.
Pushed by vhanda into branch 'KDE/4.10'.

Dolphin Search Date: Set the proper date

QDate's api is not that intuitive and the addDays functions return a
QDate with those amount of days. They do not modify the original QDate.

M  +4    -4    dolphin/src/search/dolphinfacetswidget.cpp

http://commits.kde.org/kde-baseapps/25a2255eac8be1626258feaa4c74b7f503ad9f9a

Comment 8 Frank Reininghaus 2013-07-23 06:42:44 UTC

*** Bug 316278 has been marked as a duplicate of this bug. ***