Frédéric Buclin Assignee
	Description • 15 years ago

Search.pm looks at the number of bug statuses, respectively resolutions, passed to it to determine if you selected them all. If yes, then it ignores the bug status or resolution list as this criteria is non-restrictive (to make the SQL query faster). The problem is that if you deactivated N bug statuses or resolutions (see bug 456743) and you select all but N bug statuses or resolutions in query.cgi, it thinks that you selected them all because the counts are equal: same number of selected items and active values. As query.cgi correctly lists inactive values together with active ones, Search.pm should also look at active *and* inactive values before deciding that you selected them all.

	Max Kanat-Alexander
	Comment 1 • 15 years ago

Comment on attachment 417943 [details] [diff] [review]
patch, v1

I think Bugzilla::Field::Choice->type('resolution')->get_all might be more reliable, since who knows what we'll do with legal_values in the future in terms of disabled choices.

But otherwise this generally looks fine.

	Frédéric Buclin Assignee
	Comment 2 • 15 years ago

(In reply to comment #1)
> I think Bugzilla::Field::Choice->type('resolution')->get_all might be more
> reliable, since who knows what we'll do with legal_values in the future

->legal_values is what is used in query.cgi. So I use it here too for consistency, and this way we are sure to compare comparable lists.


Checking in Bugzilla/Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.179; previous revision: 1.178
done