Bug 321198 - search by time does not work
Summary: search by time does not work
Alias: None
Product: dolphin
Classification: Applications
Component: search (show other bugs)
Version: 16.12.2
Platform: Fedora RPMs Linux
: NOR normal (vote)
Target Milestone: ---
Assignee: Vishesh Handa
: 316278 (view as bug list)
Depends on:
Reported: 2013-06-15 23:53 UTC by Dj YB
Modified: 2013-07-23 06:42 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.10.5

Fixes the issue (1.38 KB, patch)
2013-06-25 19:05 UTC, Vishesh Handa

Note You need to log in before you can comment on or make changes to this bug.
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
Actual Results:  

Expected Results:  
stuff from this year

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:
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

Comment 8 Frank Reininghaus 2013-07-23 06:42:44 UTC
*** Bug 316278 has been marked as a duplicate of this bug. ***