summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rwxr-xr-xenter_bug.cgi37
1 files changed, 9 insertions, 28 deletions
diff --git a/enter_bug.cgi b/enter_bug.cgi
index 2df9ca247..493dcadd3 100755
--- a/enter_bug.cgi
+++ b/enter_bug.cgi
@@ -79,13 +79,13 @@ if (!defined $::FORM{'product'}) {
}
push(@prodlist, $p);
}
-
if (1 != @prodlist) {
print "Content-type: text/html\n\n";
-
- my $bProdFlag = 0;
- my $tableContents = "";
-
+ PutHeader("Enter Bug");
+
+ print "<H2>First, you must pick a product on which to enter\n";
+ print "a bug.</H2>\n";
+ print "<table>";
foreach my $p (@prodlist) {
if (defined $::proddesc{$p} && $::proddesc{$p} eq '0') {
# Special hack. If we stuffed a "0" into proddesc, that means
@@ -93,7 +93,6 @@ if (!defined $::FORM{'product'}) {
# to allow people to specify that product here.
next;
}
-
if(Param("usebuggroupsentry")
&& GroupExists($p)
&& !UserInGroup($p)) {
@@ -102,34 +101,16 @@ if (!defined $::FORM{'product'}) {
# group, we don't want to include that product in this list.
next;
}
-
- $bProdFlag = 1;
- $tableContents .= "<tr><th align=right valign=top><a href=\"enter_bug.cgi?product=" . url_quote($p) . "\">$p</a>:</th>\n";
-
+ print "<tr><th align=right valign=top><a href=\"enter_bug.cgi?product=" . url_quote($p) . "\">$p</a>:</th>\n";
if (defined $::proddesc{$p}) {
- $tableContents .= "<td valign=top>$::proddesc{$p}</td>\n";
+ print "<td valign=top>$::proddesc{$p}</td>\n";
}
-
- $tableContents .= "</tr>";
+ print "</tr>";
}
-
- # display sensible message if no products were found for the user to enter bugs against
- if ($bProdFlag) {
- PutHeader("Enter Bug");
- print "<H2>First, you must pick a product on which to enter a bug.</H2>\n";
- print "<table>\n";
- print $tableContents;
- print "</table>\n";
- } else {
- PutHeader("Permission Denied");
- print "Sorry. There are no products for which you may enter bugs\n";
- print "<P>\n";
- }
-
+ print "</table>\n";
PutFooter();
exit;
}
-
$::FORM{'product'} = $prodlist[0];
}