From 012d45ae9579b0f1690a2daed8212f38e9c9e26a Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Fri, 13 Jul 2007 17:43:27 +0000 Subject: Bug 385849: Make Bugzilla::Bug do updating for op_sys, rep_platform, and other product-inspecific fields Patch By Max Kanat-Alexander r=LpSolit --- Bugzilla/Bug.pm | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) (limited to 'Bugzilla/Bug.pm') diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 321220194..3ace277dd 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -149,8 +149,15 @@ use constant UPDATE_VALIDATORS => { use constant UPDATE_COLUMNS => qw( everconfirmed + bug_file_loc + bug_severity bug_status + op_sys + priority + rep_platform resolution + short_desc + status_whiteboard ); # This is used by add_comment to know what we validate before putting in @@ -1145,6 +1152,19 @@ sub fields { # Mutators ##################################################################### +# To run check_can_change_field. +sub _set_global_validator { + my ($self, $value, $field) = @_; + my $current_value = $self->$field; + my $privs; + $self->check_can_change_field($field, $current_value, $value, \$privs) + || ThrowUserError('illegal_change', { field => $field, + oldvalue => $current_value, + newvalue => $value, + privs => $privs }); +} + + ################# # "Set" Methods # ################# @@ -1160,13 +1180,20 @@ sub set_dependencies { $self->{'blocked'} = $blocked; } sub _set_everconfirmed { $_[0]->set('everconfirmed', $_[1]); } +sub set_op_sys { $_[0]->set('op_sys', $_[1]); } +sub set_platform { $_[0]->set('rep_platform', $_[1]); } +sub set_priority { $_[0]->set('priority', $_[1]); } sub set_resolution { $_[0]->set('resolution', $_[1]); } +sub set_severity { $_[0]->set('bug_severity', $_[1]); } sub set_status { my ($self, $status) = @_; $self->set('bug_status', $status); # Check for the everconfirmed transition $self->_set_everconfirmed(1) if (is_open_state($status) && $status ne 'UNCONFIRMED'); } +sub set_status_whiteboard { $_[0]->set('status_whiteboard', $_[1]); } +sub set_summary { $_[0]->set('short_desc', $_[1]); } +sub set_url { $_[0]->set('bug_file_loc', $_[1]); } ######################## # "Add/Remove" Methods # -- cgit v1.2.3-24-g4f1b