From 003e49ae68c247448a5d9d18cb0110432668fdfd Mon Sep 17 00:00:00 2001
From: Max Kanat-Alexander <mkanat@bugzilla.org>
Date: Sun, 16 May 2010 15:21:29 -0700
Subject: Bug 428452: Move code to set/modify keywords from process_bug.cgi
 into Bugzilla::Bug::set_all r=dkl, a=mkanat

---
 process_bug.cgi | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

(limited to 'process_bug.cgi')

diff --git a/process_bug.cgi b/process_bug.cgi
index ef8f41070..0753773df 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -306,16 +306,6 @@ elsif (should_set('dependson') || should_set('blocked')) {
     }
 }
 
-my $any_keyword_changes;
-if (defined $cgi->param('keywords')) {
-    foreach my $b (@bug_objects) {
-        my $return =
-            $b->modify_keywords(scalar $cgi->param('keywords'),
-                                scalar $cgi->param('keywordaction'));
-        $any_keyword_changes ||= $return;
-    }
-}
-
 # Component, target_milestone, and version are in here just in case
 # the 'product' field wasn't defined in the CGI. It doesn't hurt to set
 # them twice.
@@ -323,7 +313,8 @@ my @set_fields = qw(op_sys rep_platform priority bug_severity
                     component target_milestone version
                     bug_file_loc status_whiteboard short_desc
                     deadline remaining_time estimated_time
-                    work_time set_default_assignee set_default_qa_contact);
+                    work_time set_default_assignee set_default_qa_contact
+                    keywords keywordaction);
 push(@set_fields, 'assigned_to') if !$cgi->param('set_default_assignee');
 push(@set_fields, 'qa_contact')  if !$cgi->param('set_default_qa_contact');
 my %field_translation = (
@@ -333,6 +324,7 @@ my %field_translation = (
     bug_file_loc => 'url',
     set_default_assignee   => 'reset_assigned_to',
     set_default_qa_contact => 'reset_qa_contact',
+    keywordaction => 'keywords_action',
 );
 
 my %set_all_fields;
-- 
cgit v1.2.3-24-g4f1b