Fix for bug 51487: Now only checks for a valid QA Contact if it actually

asked you for one.  Thanks to bg.mahesh@team.indiainfo.com (B.G. Mahesh)
for catching this.

diff --git a/editcomponents.cgi b/editcomponents.cgi
index 1abe62253558a9fc4957ecb129f4d825453aa40c..4714a4d8a422de3a96f23532d0536bd47c6b1b6d 100755 (executable)
--- a/editcomponents.cgi
+++ b/editcomponents.cgi
@@ -397,14 +397,8 @@ if ($action eq 'new') {
         PutTrailer($localtrailer);
         exit;
     }
-    #+++
 
     my $initialqacontact = trim($::FORM{initialqacontact} || '');
-       #
-       # Now validating to make sure it's too an existing account
-       #
-       DBNameToIdAndCheck($initialqacontact);
-
     if (Param('useqacontact')) {
         if ($initialqacontact eq '') {
             print "You must enter an initial QA contact for the component '$component'. Please press\n";
@@ -412,8 +406,10 @@ if ($action eq 'new') {
             PutTrailer($localtrailer);
             exit;
         }
-        #+++
-        #DBNameToIdAndCheck($initialqacontact, 0);
+       #
+       # Now validating to make sure it's too an existing account
+       #
+       DBNameToIdAndCheck($initialqacontact);
     }
 
     # Add the new component