diff options
author | bbaetz%acm.org <> | 2003-02-23 16:31:07 +0100 |
---|---|---|
committer | bbaetz%acm.org <> | 2003-02-23 16:31:07 +0100 |
commit | c0266d1fcdc15f5ccfb5890492a42f4ea598437b (patch) | |
tree | 3302ae6af039953c20ece548fad40742572774cd /Bugzilla | |
parent | 9fc47ebee67937f2b309494457784d6ee8db9cfd (diff) | |
download | bugzilla-c0266d1fcdc15f5ccfb5890492a42f4ea598437b.tar.gz bugzilla-c0266d1fcdc15f5ccfb5890492a42f4ea598437b.tar.xz |
Bug 194394 - Internal error after turning useqacontact off
r,a=justdave
Diffstat (limited to 'Bugzilla')
-rwxr-xr-x | Bugzilla/Bug.pm | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 14876dbe3..b451190e2 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -50,9 +50,13 @@ sub fields { bug_file_loc status_whiteboard keywords priority bug_severity target_milestone dependson blocked votes - reporter assigned_to qa_contact cc + reporter assigned_to cc ); + if (Param('useqacontact')) { + push @fields, "qa_contact"; + } + if (Param('timetrackinggroup')) { push @fields, qw(estimated_time remaining_time actual_time); } @@ -182,6 +186,8 @@ sub initBug { if (Param('useqacontact') && $self->{'qa_contact'} > 0) { $self->{'qa_contact'} = new Bugzilla::User($self->{'qa_contact'}); + } else { + $self->{'qa_contact'} = undef; } my $ccSet = new RelationSet; @@ -376,7 +382,7 @@ sub user { # and actually try to make the change. $self->{'user'}->{'canedit'} = $::userid == 0 || $::userid == $self->{'reporter'}{'id'} - || ($self->{'qa_contact'} && $::userid == $self->{'qa_contact'}{'id'}) + || (Param('useqacontact') && $self->{'qa_contact'} && $::userid == $self->{'qa_contact'}{'id'}) || $::userid == $self->{'assigned_to'}{'id'} || &::UserInGroup("editbugs"); $self->{'user'}->{'canconfirm'} = $::userid == 0 |