diff options
author | dave%intrec.com <> | 2000-09-11 01:54:04 +0200 |
---|---|---|
committer | dave%intrec.com <> | 2000-09-11 01:54:04 +0200 |
commit | 8ea64d14a889e8ab31b15d488d9e6d16c0f6a534 (patch) | |
tree | 591979fe2ba08805ef7d07de3177d71f4e4480b1 | |
parent | ef9c84bc9f388a6f15b39465f0a5c769496db311 (diff) | |
download | bugzilla-8ea64d14a889e8ab31b15d488d9e6d16c0f6a534.tar.gz bugzilla-8ea64d14a889e8ab31b15d488d9e6d16c0f6a534.tar.xz |
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.
-rwxr-xr-x | editcomponents.cgi | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/editcomponents.cgi b/editcomponents.cgi index 1abe62253..4714a4d8a 100755 --- 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 |