diff options
author | lpsolit%gmail.com <> | 2009-12-17 23:10:33 +0000 |
---|---|---|
committer | lpsolit%gmail.com <> | 2009-12-17 23:10:33 +0000 |
commit | c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5 (patch) | |
tree | 62b2810a1ea02edbfa2a83ef3cfb5e3744a048d0 /Bugzilla | |
parent | 95b16ee99af37bb2d3e5bd15fa63c6197c399dbe (diff) | |
download | bugs-c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5.tar bugs-c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5.tar.gz bugs-c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5.tar.bz2 bugs-c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5.tar.xz bugs-c5612a1fdcbdfd6aacf9e85e1f74c399ba021fa5.zip |
Bug 535309: The bug_status and resolution fields in query.cgi can be ignored by Search.pm if you deactivated some resolutions or bug statuses - Patch by Frédéric Buclin <LpSolit@gmail.com> r/a=mkanat
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Search.pm | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm index 7b8ac10e2..bb23f6fdc 100644 --- a/Bugzilla/Search.pm +++ b/Bugzilla/Search.pm @@ -304,7 +304,8 @@ sub init { # into their equivalent lists of open and closed statuses. if ($params->param('bug_status')) { my @bug_statuses = $params->param('bug_status'); - my @legal_statuses = @{get_legal_field_values('bug_status')}; + # Also include inactive bug statuses, as you can query them. + my @legal_statuses = @{Bugzilla::Field->new({name => 'bug_status'})->legal_values}; if (scalar(@bug_statuses) == scalar(@legal_statuses) || $bug_statuses[0] eq "__all__") { @@ -322,7 +323,8 @@ sub init { if ($params->param('resolution')) { my @resolutions = $params->param('resolution'); - my $legal_resolutions = get_legal_field_values('resolution'); + # Also include inactive resolutions, as you can query them. + my $legal_resolutions = Bugzilla::Field->new({name => 'resolution'})->legal_values; if (scalar(@resolutions) == scalar(@$legal_resolutions)) { $params->delete('resolution'); } |