Bug 2662 - "ag" quicksearch adds "1" to value
Summary: "ag" quicksearch adds "1" to value
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail
Classification: Unclassified
Component: Filtering (show other bugs)
Version: 3.8.0
Hardware: PC All
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2012-05-16 08:28 CEST by Abhay S. Kushwaha
Modified: 2012-05-20 11:02 CEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Abhay S. Kushwaha 2012-05-16 08:28:53 CEST
When using the quicksearch filter on messages, if you use "ag" filter, it adds "1" to the value.

So if you want to search for messages that arrived more than 1 day ago, one has to use "ag 0" instead of "ag 1" which actually returns message that arrived more than 2 days ago.
Comment 1 users 2012-05-16 09:26:34 CEST
Changes related to this bug have been committed.
Please check latest CVS and update the bug accordingly.
You can also get the patch from:
http://www.claws-mail.org/tracker/

2012-05-16 [paul]	3.8.0cvs43

	* src/matcher.c
		fix bug 2662, '"ag" quicksearch adds "1" to value'
Comment 2 Colin Leroy 2012-05-16 09:36:03 CEST
From my point of view, this was correct. "ag 0" gives me mails received more than 24 hours ago. This is "more than zero day", aka "one day or more".

With the patch in place, in my opinion it's now working as "age 0" (age greater or equal).
Comment 3 Paul 2012-05-16 09:53:54 CEST
I guess it is a point of view: does zero days mean 23h 59m 59s or does it mean 0h 0d 0s
Comment 4 Paul 2012-05-16 09:58:08 CEST
the thing is that, as Abhay pointed out, "ag 1 & al 2" was returning nothing, so ag al were working forwards and backwards from different points (end and start) in a 24 hour period
Comment 5 Colin Leroy 2012-05-16 10:10:34 CEST
True... No hard feelings either way, for me.