aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMax Kanat-Alexander <mkanat@bugzilla.org>2010-05-16 15:21:29 -0700
committerMax Kanat-Alexander <mkanat@bugzilla.org>2010-05-16 15:21:29 -0700
commit003e49ae68c247448a5d9d18cb0110432668fdfd (patch)
treec735981eac17952cb445dc053c6fd3270eeb95a7 /process_bug.cgi
parentBug 556154: process_bug.cgi: move the stuff in the loop after $b->set_all (diff)
downloadbugzilla-003e49ae68c247448a5d9d18cb0110432668fdfd.tar.gz
bugzilla-003e49ae68c247448a5d9d18cb0110432668fdfd.tar.bz2
bugzilla-003e49ae68c247448a5d9d18cb0110432668fdfd.zip
Bug 428452: Move code to set/modify keywords from process_bug.cgi into
Bugzilla::Bug::set_all r=dkl, a=mkanat
Diffstat (limited to 'process_bug.cgi')
-rwxr-xr-xprocess_bug.cgi14
1 files changed, 3 insertions, 11 deletions
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;