Bug 194394 - Internal error after turning useqacontact off

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

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

globals.pl

@@ -661,7 +661,7 @@ sub CanSeeBug {
                || (($userid > 0) && 
                   (
                        ($assigned_to == $userid) 
-                    || ($qa_contact == $userid)
+                    || (Param('useqacontact') && $qa_contact == $userid)
                     || (($reporter == $userid) && $rep_access) 
                     || ($found_cc && $cc_access) 
                     || ($found_groups == $found_members)

process_bug.cgi

@@ -410,7 +410,7 @@ sub CheckCanChangeField {
     }
     
     # Allow the QA contact to change anything.
-    if ($qacontactid eq $whoid) {
+    if (Param('useqacontact') && ($qacontactid eq $whoid)) {
         return 1;
     }