584414 - In Bugzilla 3.6, searching for bugs with at least X votes is broken

Status: VERIFIED FIXED

(Bugzilla :: Query/Bug List, defect)

Description

[karl155]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.2.8) Gecko/20100722 Firefox/3.6.8
Build Identifier: 

Now it is possible to enter every valid field in the serarch query. But that is the problem:
1. The serach mask says: "Only bugs with at least: ____ votes".
2. If you enter 50 you will get a query with "votes=50".
3. This ends with that result filter: "Votes: (is greater than) 49, Votes: 50"


Reproducible: Always

Comment 1

[karl155]

Version: 3.6.1:

http://www.bugzilla.org/releases/3.6.1/release-notes.html
"Search: For those who like to make their own buglist.cgi URLs (and for people working on customizations), buglist.cgi now accepts nearly every valid field in Bugzilla as a direct URL parameter, like &field=value"

Comment 2

[Max Kanat-Alexander]

This is really blocking 3.6.3, but I'll change that when we have the flag.

Comment 3

[Frédéric Buclin]

The SQL query has:

WHERE ((bugs.votes > '3') AND ( bugs.votes IN ('4') ))


Bugzilla 3.4 doesn't have this 2nd condition.

Comment 4

[Frédéric Buclin]

For the record, Bugzilla 4.0 and 4.1 are not affected. So this is a 3.6 only regression.

Comment 5

[Thomas D. (:thomas8)]

Is there any workaround for this in 3.6? Any query that I could use to show all bugs of a given product with more than 50 votes?

Comment 6

[Thomas D. (:thomas8)]

https://bugzilla.mozilla.org/buglist.cgi?query_format=advanced&field0-0-0=votes&type0-0-0=greaterthan&value0-0-0=50&resolution=---&product=Thunderbird

Comment 7

[Max Kanat-Alexander]

Attachment: v1

Here's a patch to fix it in 3.6.

Comment 8

[Frédéric Buclin]

Comment on attachment 472589 [details] [diff] [review]
v1

Cool, thanks for the fix. :) r=LpSolit

Comment 9

[Max Kanat-Alexander]

Hey, you're welcome! :-) Thanks for the review!

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/                         
modified Bugzilla/Search.pm
Committed revision 7169.