summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordave%intrec.com <>2000-09-11 01:54:04 +0200
committerdave%intrec.com <>2000-09-11 01:54:04 +0200
commit8ea64d14a889e8ab31b15d488d9e6d16c0f6a534 (patch)
tree591979fe2ba08805ef7d07de3177d71f4e4480b1
parentef9c84bc9f388a6f15b39465f0a5c769496db311 (diff)
downloadbugzilla-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-xeditcomponents.cgi12
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