From 5323ab05caef4b7a5c44fb567b412a3917272f33 Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Sat, 25 Oct 2008 04:13:50 +0000
Subject: Bug 459064: Set FIELD_TYPE_SINGLE_SELECT on standard global select
 fields Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=bbaetz, a=mkanat

---
 Bugzilla/WebService/Bug.pm | 17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)

(limited to 'Bugzilla/WebService')

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index 8cfb6acde..b6411968c 100755
--- a/Bugzilla/WebService/Bug.pm
+++ b/Bugzilla/WebService/Bug.pm
@@ -45,15 +45,6 @@ use constant FIELD_MAP => {
     platform    => 'rep_platform',
 };
 
-use constant GLOBAL_SELECT_FIELDS => qw(
-    bug_severity
-    bug_status
-    op_sys
-    priority
-    rep_platform
-    resolution
-);
-
 use constant PRODUCT_SPECIFIC_FIELDS => qw(version target_milestone component);
 
 ######################################################
@@ -192,13 +183,11 @@ sub legal_values {
     my ($self, $params) = @_;
     my $field = FIELD_MAP->{$params->{field}} || $params->{field};
 
-    my @custom_select = Bugzilla->get_fields(
-        {custom => 1, type => [FIELD_TYPE_SINGLE_SELECT, FIELD_TYPE_MULTI_SELECT]});
-    # We only want field names.
-    @custom_select = map {$_->name} @custom_select;
+    my @global_selects = Bugzilla->get_fields(
+        {type => [FIELD_TYPE_SINGLE_SELECT, FIELD_TYPE_MULTI_SELECT]});
 
     my $values;
-    if (grep($_ eq $field, GLOBAL_SELECT_FIELDS, @custom_select)) {
+    if (grep($_->name eq $field, @global_selects)) {
         $values = get_legal_field_values($field);
     }
     elsif (grep($_ eq $field, PRODUCT_SPECIFIC_FIELDS)) {
-- 
cgit v1.2.3-24-g4f1b