aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--CGI.pl21
-rw-r--r--bug_form.pl2
-rwxr-xr-xbuglist.cgi137
-rwxr-xr-xchangepassword.cgi20
-rwxr-xr-xcolchange.cgi22
-rwxr-xr-xdoeditowners.cgi4
-rwxr-xr-xdoeditparams.cgi4
-rwxr-xr-xeditowners.cgi12
-rwxr-xr-xeditparams.cgi26
-rwxr-xr-xenter_bug.cgi8
-rwxr-xr-xpost_bug.cgi15
-rwxr-xr-xprocess_bug.cgi69
-rwxr-xr-xquery.cgi135
-rwxr-xr-xrelogin.cgi6
-rwxr-xr-xreports.cgi20
-rwxr-xr-xsanitycheck.cgi3
-rwxr-xr-xshow_bug.cgi15
17 files changed, 243 insertions, 276 deletions
diff --git a/CGI.pl b/CGI.pl
index 89768775a..f16640bc1 100644
--- a/CGI.pl
+++ b/CGI.pl
@@ -39,8 +39,8 @@ sub GeneratePersonInput {
}
sub GeneratePeopleInput {
- my ($field, $size, $def_value) = (@_);
- return "<INPUT NAME=\"$field\" SIZE=\"$size\" VALUE=\"$def_value\">";
+ my ($field, $def_value) = (@_);
+ return "<INPUT NAME=\"$field\" SIZE=45 VALUE=\"$def_value\">";
}
@@ -187,15 +187,15 @@ sub make_options {
}
$last = $item;
if ($isregexp ? $item =~ $default : $default eq $item) {
- $popup .= " <OPTION SELECTED VALUE=\"" . url_quote($item) . "\">" . url_decode($item) . "\n";
+ $popup .= "<OPTION SELECTED VALUE=\"$item\">$item";
$found = 1;
} else {
- $popup .= " <OPTION VALUE=\"" . url_quote($item) . "\">" . url_decode($item) . "\n";
+ $popup .= "<OPTION VALUE=\"$item\">$item";
}
}
}
if (!$found && $default ne "") {
- $popup .= " <OPTION SELECTED>$default\n";
+ $popup .= "<OPTION SELECTED>$default";
}
return $popup;
}
@@ -203,9 +203,9 @@ sub make_options {
sub make_popup {
my ($name,$src,$default,$listtype,$onchange) = (@_);
- my $popup = "<SELECT NAME=\"$name\"";
+ my $popup = "<SELECT NAME=$name";
if ($listtype > 0) {
- $popup .= " SIZE=\"5\"";
+ $popup .= " SIZE=5";
if ($listtype == 2) {
$popup .= " MULTIPLE";
}
@@ -246,8 +246,7 @@ sub confirm_login {
if ($enteredlogin !~ /^[^@, ]*@[^@, ]*\.[^@, ]*$/) {
print "Content-type: text/html\n\n";
- PutHeader("Invalid e-mail address entered");
-
+ print "<H1>Invalid e-mail address entered.</H1>\n";
print "The e-mail address you entered\n";
print "(<b>$enteredlogin</b>) didn't match our minimal\n";
print "syntax checking for a legal email address. A legal\n";
@@ -303,7 +302,7 @@ To use the wonders of bugzilla, you can use the following:
my $enteredcryptpwd = crypt($enteredpwd, substr($realcryptpwd, 0, 2));
if ($realcryptpwd eq "" || $enteredcryptpwd ne $realcryptpwd) {
print "Content-type: text/html\n\n";
- PutHeader("Login failed.");
+ print "<H1>Login failed.</H1>\n";
print "The username or password you entered is not valid.\n";
print "Please click <b>Back</b> and try again.\n";
exit;
@@ -345,7 +344,7 @@ To use the wonders of bugzilla, you can use the following:
if ($loginok ne "1") {
print "Content-type: text/html\n\n";
- PutHeader("Please log in.");
+ print "<H1>Please log in.</H1>\n";
print "I need a legitimate e-mail address and password to continue.\n";
if (!defined $nexturl || $nexturl eq "") {
# Sets nexturl to be argv0, stripping everything up to and
diff --git a/bug_form.pl b/bug_form.pl
index 287b6ad12..ed6aead8a 100644
--- a/bug_form.pl
+++ b/bug_form.pl
@@ -39,7 +39,7 @@ select
short_desc,
date_format(creation_ts,'Y-m-d')
from bugs
-where bug_id = '" . $::FORM{'id'} . "'";
+where bug_id = $::FORM{'id'}";
SendSQL($query);
my %bug;
diff --git a/buglist.cgi b/buglist.cgi
index c0cf44d47..8439ccbbd 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -24,18 +24,13 @@ use strict;
require "CGI.pl";
-my $serverpush = 0;
+my $serverpush = 1;
-# Internet explorer and Lynx don't seem to understand server push. What fun.
-# Carefully coded to cope with the fact that MSIE puts both "MSIE" and
-# "Mozilla" in its user agent string.
-
-$_ = $ENV{'HTTP_USER_AGENT'};
-if ($_ =~/Mozilla/ && $_ !~ /MSIE/) {
- $serverpush = 1;
+if ($ENV{'HTTP_USER_AGENT'} =~ /MSIE/) {
+ # Internet explorer doesn't seem to understand server push. What fun.
+ $serverpush = 0;
}
-
if ($serverpush) {
print "Content-type: multipart/x-mixed-replace;boundary=thisrandomstring\n";
print "\n";
@@ -68,47 +63,46 @@ if (!defined $::FORM{'cmdtype'}) {
CMD: for ($::FORM{'cmdtype'}) {
/^runnamed$/ && do {
$::buffer = $::COOKIE{"QUERY_" . $::FORM{"namedcmd"}};
- ProcessFormFields($::buffer);
+ ProcessFormFields $::buffer;
last CMD;
};
/^editnamed$/ && do {
my $url = "query.cgi?" . $::COOKIE{"QUERY_" . $::FORM{"namedcmd"}};
print "Content-type: text/html
-Refresh: 0; URL=$url\n\n";
+Refresh: 0; URL=$url
-PutHeader("What a hack.");
-print "
+<TITLE>What a hack.</TITLE>
Loading your query named <B>$::FORM{'namedcmd'}</B>...";
exit;
};
/^forgetnamed$/ && do {
print "Set-Cookie: QUERY_" . $::FORM{'namedcmd'} . "= ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT
-Content-type: text/html\n\n";
+Content-type: text/html
-PutHeader("Forget what?");
-print "
+<HTML>
+<TITLE>Forget what?</TITLE>
OK, the <B>$::FORM{'namedcmd'}</B> query is gone.
<P>
-<A HREF=\"query.cgi\">Go back to the query page.</A>
-";
+<A HREF=query.cgi>Go back to the query page.</A>";
exit;
};
/^asnamed$/ && do {
if ($::FORM{'newqueryname'} =~ /^[a-zA-Z0-9_ ]+$/) {
print "Set-Cookie: QUERY_" . $::FORM{'newqueryname'} . "=$::buffer ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT
-Content-type: text/html\n\n";
+Content-type: text/html
-PutHeader("OK, done.");
-print "
+<HTML>
+<TITLE>OK, done.</TITLE>
OK, you now have a new query named <B>$::FORM{'newqueryname'}</B>.
+
<P>
-<A HREF=\"query.cgi\">Go back to the query page.</A>
-";
+
+<A HREF=query.cgi>Go back to the query page.</A>";
} else {
- print "Content-type: text/html\n\n";
+ print "Content-type: text/html
-PutHeader("Picky, picky.");
-print "
+<HTML>
+<TITLE>Picky, picky.</TITLE>
Query names can only have letters, digits, spaces, or underbars. You entered
\"<B>$::FORM{'newqueryname'}</B>\", which doesn't cut it.
<P>
@@ -118,15 +112,15 @@ Click the <B>Back</B> button and type in a valid name for this query.";
};
/^asdefault$/ && do {
print "Set-Cookie: DEFAULTQUERY=$::buffer ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT
-Content-type: text/html\n\n";
+Content-type: text/html
-PutHeader("OK, default is set.");
-print "
+<HTML>
+<TITLE>OK, default is set.</TITLE>
OK, you now have a new default query.
<P>
-<A HREF=\"query.cgi\">Go back to the query page, using the new default.</A>";
+<A HREF=query.cgi>Go back to the query page, using the new default.</A>";
exit;
};
}
@@ -231,8 +225,7 @@ if (defined $::FORM{'sql'}) {
}
} else {
my $ref = $::MFORM{$field};
- foreach my $vv (@$ref) {
- my $v = url_decode($vv);
+ foreach my $v (@$ref) {
if ($v eq "(empty)") {
$query .= "\t\t${or}bugs.$field is null\n";
} else {
@@ -335,20 +328,20 @@ if (defined $::FORM{'order'}) {
}
if ($dotweak) {
- pnl "<FORM NAME=\"changeform\" METHOD=\"POST\" ACTION=\"process_bug.cgi\">";
+ pnl "<FORM NAME=changeform METHOD=POST ACTION=\"process_bug.cgi\">";
}
-my $tablestart = "<TABLE CELLSPACING=\"0\" CELLPADDING=\"2\">
-<TR ALIGN=\"LEFT\"><TH>
+my $tablestart = "<TABLE CELLSPACING=0 CELLPADDING=2>
+<TR ALIGN=LEFT><TH>
<A HREF=\"buglist.cgi?$fields&order=bugs.bug_id\">ID</A>";
foreach my $c (@collist) {
if (exists $::needquote{$c}) {
if ($::needquote{$c}) {
- $tablestart .= "<TH WIDTH=\"100%\" valigh=\"left\">";
+ $tablestart .= "<TH WIDTH=100% valigh=left>";
} else {
- $tablestart .= "<TH valign=\"left\">";
+ $tablestart .= "<TH valign=left>";
}
if (defined $::sortkey{$c}) {
$tablestart .= "<A HREF=\"buglist.cgi?$fields&order=$::sortkey{$c}$oldorder\">$::title{$c}</A>";
@@ -377,9 +370,9 @@ while (@row = FetchSQLData()) {
pnl "</TABLE>$tablestart";
}
push @bugarray, $bug_id;
- pnl "<TR VALIGN=\"TOP\" ALIGN=\"LEFT\"><TD>";
+ pnl "<TR VALIGN=TOP ALIGN=LEFT><TD>";
if ($dotweak) {
- pnl "<input type=\"checkbox\" name=\"id_$bug_id\">";
+ pnl "<input type=checkbox name=id_$bug_id>";
}
pnl "<A HREF=\"show_bug.cgi?id=$bug_id\">";
pnl "$bug_id</A> ";
@@ -455,9 +448,9 @@ if (open (COMMENTS, "<data/comments")) {
}
-print "<HR><I><A HREF=\"newquip.html\">$quip\n";
+print "<HR><I><A HREF=newquip.html>$quip\n";
print "</I></A></CENTER>\n";
-print "<HR SIZE=\"10\">$tablestart\n";
+print "<HR SIZE=10>$tablestart\n";
print $::bugl;
print "</TABLE>\n";
@@ -480,7 +473,7 @@ function SetCheckboxes(value) {
item.checked = value;
}
}
-document.write(\" <input type=\\\"button\\\" value=\\\"Uncheck All\\\" onclick=\\\"SetCheckboxes(false);\\\"> <input type=\\\"button\\\" value=\\\"Check All\\\" onclick=\\\"SetCheckboxes(true);\\\">\");
+document.write(\" <input type=button value=\\\"Uncheck All\\\" onclick=\\\"SetCheckboxes(false);\\\"> <input type=button value=\\\"Check All\\\" onclick=\\\"SetCheckboxes(true);\\\">\");
</SCRIPT>";
my $resolution_popup = make_options(\@::legal_resolution_no_dup, "FIXED");
my @prod_list = keys %prodhash;
@@ -504,52 +497,52 @@ document.write(\" <input type=\\\"button\\\" value=\\\"Uncheck All\\\" onclick=\
<hr>
<TABLE>
<TR>
- <TD ALIGN=\"RIGHT\"><B>Product:</B></TD>
- <TD><SELECT NAME=\"product\">$product_popup</SELECT></TD>
- <TD ALIGN=\"RIGHT\"><B>Version:</B></TD>
- <TD><SELECT NAME=\"version\">$version_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B>Product:</B></TD>
+ <TD><SELECT NAME=product>$product_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B>Version:</B></TD>
+ <TD><SELECT NAME=version>$version_popup</SELECT></TD>
<TR>
- <TD ALIGN=\"RIGHT\"><B><A HREF=\"bug_status.html#rep_platform\">Platform:</A></B></TD>
- <TD><SELECT NAME=\"rep_platform\">$platform_popup</SELECT></TD>
- <TD ALIGN=\"RIGHT\"><B><A HREF=\"bug_status.html#priority\">Priority:</A></B></TD>
- <TD><SELECT NAME=\"priority\">$priority_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B><A HREF=\"bug_status.html#rep_platform\">Platform:</A></B></TD>
+ <TD><SELECT NAME=rep_platform>$platform_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B><A HREF=\"bug_status.html#priority\">Priority:</A></B></TD>
+ <TD><SELECT NAME=priority>$priority_popup</SELECT></TD>
</TR>
<TR>
- <TD ALIGN=\"RIGHT\"><B>Component:</B></TD>
- <TD><SELECT NAME=\"component\">$component_popup</SELECT></TD>
- <TD ALIGN=\"RIGHT\"><B><A HREF=\"bug_status.html#severity\">Severity:</A></B></TD>
- <TD><SELECT NAME=\"bug_severity\">$sev_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B>Component:</B></TD>
+ <TD><SELECT NAME=component>$component_popup</SELECT></TD>
+ <TD ALIGN=RIGHT><B><A HREF=\"bug_status.html#severity\">Severity:</A></B></TD>
+ <TD><SELECT NAME=bug_severity>$sev_popup</SELECT></TD>
</TR>
</TABLE>
-<INPUT NAME=\"multiupdate\" value=\"Y\" TYPE=\"hidden\">
+<INPUT NAME=multiupdate value=Y TYPE=hidden>
<B>Additional Comments:</B>
<BR>
-<TEXTAREA WRAP=\"HARD\" NAME=\"comment\" ROWS=\"5\" COLS=\"80\"></TEXTAREA><BR>";
+<TEXTAREA WRAP=HARD NAME=comment ROWS=5 COLS=80></TEXTAREA><BR>";
# knum is which knob number we're generating, in javascript terms.
my $knum = 0;
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"none\" CHECKED>
+<INPUT TYPE=radio NAME=knob VALUE=none CHECKED>
Do nothing else<br>";
$knum++;
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"accept\">
+<INPUT TYPE=radio NAME=knob VALUE=accept>
Accept bugs (change status to <b>ASSIGNED</b>)<br>";
$knum++;
if (!defined $statushash{'CLOSED'} &&
!defined $statushash{'VERIFIED'} &&
!defined $statushash{'RESOLVED'}) {
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"clearresolution\">
+<INPUT TYPE=radio NAME=knob VALUE=clearresolution>
Clear the resolution<br>";
$knum++;
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"resolve\">
+<INPUT TYPE=radio NAME=knob VALUE=resolve>
Resolve bugs, changing <A HREF=\"bug_status.html\">resolution</A> to
- <SELECT NAME=\"resolution\"
+ <SELECT NAME=resolution
ONCHANGE=\"document.changeform.knob\[$knum\].checked=true\">
$resolution_popup</SELECT><br>";
$knum++;
@@ -558,38 +551,38 @@ document.write(\" <input type=\\\"button\\\" value=\\\"Uncheck All\\\" onclick=\
!defined $statushash{'ASSIGNED'} &&
!defined $statushash{'REOPENED'}) {
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"reopen\"> Reopen bugs<br>";
+<INPUT TYPE=radio NAME=knob VALUE=reopen> Reopen bugs<br>";
$knum++;
}
my @statuskeys = keys %statushash;
if ($#statuskeys == 1) {
if (defined $statushash{'RESOLVED'}) {
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"verify\">
+<INPUT TYPE=radio NAME=knob VALUE=verify>
Mark bugs as <b>VERIFIED</b><br>";
$knum++;
}
if (defined $statushash{'VERIFIED'}) {
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"close\">
+<INPUT TYPE=radio NAME=knob VALUE=close>
Mark bugs as <b>CLOSED</b><br>";
$knum++;
}
}
print "
-<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"reassign\">
+<INPUT TYPE=radio NAME=knob VALUE=reassign>
<A HREF=\"bug_status.html#assigned_to\">Reassign</A> bugs to
- <INPUT NAME=\"assigned_to\" SIZE=\"32\"
+ <INPUT NAME=assigned_to SIZE=32
ONCHANGE=\"document.changeform.knob\[$knum\].checked=true\"
VALUE=\"$::COOKIE{'Bugzilla_login'}\"><br>";
$knum++;
- print "<INPUT TYPE=\"radio\" NAME=\"knob\" VALUE=\"reassignbycomponent\">
+ print "<INPUT TYPE=radio NAME=knob VALUE=reassignbycomponent>
Reassign bugs to owner of selected component<br>";
$knum++;
print "
<p>
-<font size=\"-1\">
+<font size=-1>
To make changes to a bunch of bugs at once:
<ol>
<li> Put check boxes next to the bugs you want to change.
@@ -597,15 +590,15 @@ To make changes to a bunch of bugs at once:
comment explaining what you're doing.)
<li> Click the below \"Commit\" button.
</ol></font>
-<INPUT TYPE=\"SUBMIT\" VALUE=\"Commit\">
+<INPUT TYPE=SUBMIT VALUE=Commit>
</FORM><hr>\n";
}
if ($count > 0) {
- print "<FORM METHOD=\"POST\" ACTION=\"long_list.cgi\">
-<INPUT TYPE=\"HIDDEN\" NAME=\"buglist\" VALUE=\"$buglist\">
-<INPUT TYPE=\"SUBMIT\" VALUE=\"Long Format\">
+ print "<FORM METHOD=POST ACTION=\"long_list.cgi\">
+<INPUT TYPE=HIDDEN NAME=buglist VALUE=$buglist>
+<INPUT TYPE=SUBMIT VALUE=\"Long Format\">
<A HREF=\"query.cgi\">Query Page</A>
<A HREF=\"colchange.cgi?$::buffer\">Change columns</A>
</FORM>";
diff --git a/changepassword.cgi b/changepassword.cgi
index 37543768b..a3a17e39a 100755
--- a/changepassword.cgi
+++ b/changepassword.cgi
@@ -24,20 +24,20 @@ require "CGI.pl";
confirm_login();
if (! defined $::FORM{'pwd1'}) {
- print "Content-type: text/html\n\n";
- PutHeader("Change your password");
- print "
-<FORM METHOD=\"post\">
+ print "Content-type: text/html
+
+<H1>Change your password</H1>
+<form method=post>
<table>
<tr>
-<TD ALIGN=\"right\">Please enter the new password for <b>$::COOKIE{'Bugzilla_login'}</b>:</td>
-<TD><INPUT TYPE=\"password\" NAME=\"pwd1\"></td>
+<td align=right>Please enter the new password for <b>$::COOKIE{'Bugzilla_login'}</b>:</td>
+<td><input type=password name=pwd1></td>
</tr>
<tr>
-<TD ALIGN=\"right\">Re-enter your new password:</td>
-<TD><INPUT TYPE=\"password\" name=\"pwd2\"></td>
+<td align=right>Re-enter your new password:</td>
+<td><input type=password name=pwd2></td>
</table>
-<INPUT TYPE=\"submit\" VALUE=\"Submit\">\n";
+<input type=submit value=Submit>\n";
exit;
}
@@ -85,4 +85,4 @@ SendSQL("update logincookies set cryptpassword = '$encrypted' where cookie = $::
print "<H1>OK, done.</H1>
Your new password has been set.
<p>
-<A HREF=\"query.cgi\">Back to query page.</A>\n";
+<a href=query.cgi>Back to query page.</a>\n";
diff --git a/colchange.cgi b/colchange.cgi
index 32fe1bb43..bd3b23d6f 100755
--- a/colchange.cgi
+++ b/colchange.cgi
@@ -24,6 +24,8 @@ use strict;
require "CGI.pl";
+print "Content-type: text/html\n";
+
# The master list not only says what fields are possible, but what order
# they get displayed in.
@@ -47,8 +49,8 @@ if (defined $::FORM{'rememberedquery'}) {
my $list = join(" ", @collist);
print "Set-Cookie: COLUMNLIST=$list ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT\n";
print "Refresh: 0; URL=buglist.cgi?$::FORM{'rememberedquery'}\n";
- print "Content-type: text/html\n\n";
- PutHeader("What a hack");
+ print "\n";
+ print "<TITLE>What a hack.</TITLE>\n";
print "Resubmitting your query with new columns...\n";
exit;
}
@@ -59,9 +61,6 @@ if (defined $::COOKIE{'COLUMNLIST'}) {
@collist = @::default_column_list;
}
-print "Content-type: text/html\n\n";
-
-PutHeader("Column Change");
my %desc;
foreach my $i (@masterlist) {
@@ -72,11 +71,12 @@ $desc{'summary'} = "Summary (first 60 characters)";
$desc{'summaryfull'} = "Full Summary";
+print "\n";
print "Check which columns you wish to appear on the list, and then click\n";
print "on submit.\n";
print "<p>\n";
-print "<FORM ACTION=\"colchange.cgi\">\n";
-print "<INPUT TYPE=\"HIDDEN\" NAME=\"rememberedquery\" VALUE=\"$::buffer\">\n";
+print "<FORM ACTION=colchange.cgi>\n";
+print "<INPUT TYPE=HIDDEN NAME=rememberedquery VALUE=$::buffer>\n";
foreach my $i (@masterlist) {
my $c;
@@ -85,13 +85,13 @@ foreach my $i (@masterlist) {
} else {
$c = '';
}
- print "<INPUT TYPE=\"checkbox\" NAME=\"column_$i\" $c>$desc{$i}<br>\n";
+ print "<INPUT TYPE=checkbox NAME=column_$i $c>$desc{$i}<br>\n";
}
print "<P>\n";
print "<INPUT TYPE=\"submit\" VALUE=\"Submit\">\n";
print "</FORM>\n";
-print "<FORM ACTION=\"colchange.cgi\">\n";
-print "<INPUT TYPE=\"HIDDEN\" NAME=\"rememberedquery\" VALUE=\"$::buffer\">\n";
-print "<INPUT TYPE=\"HIDDEN\" NAME=\"resetit\" VALUE=\"1\">\n";
+print "<FORM ACTION=colchange.cgi>\n";
+print "<INPUT TYPE=HIDDEN NAME=rememberedquery VALUE=$::buffer>\n";
+print "<INPUT TYPE=HIDDEN NAME=resetit VALUE=1>\n";
print "<INPUT TYPE=\"submit\" VALUE=\"Reset to Bugzilla default\">\n";
print "</FORM>\n";
diff --git a/doeditowners.cgi b/doeditowners.cgi
index 88bfdeb65..b09d7298b 100755
--- a/doeditowners.cgi
+++ b/doeditowners.cgi
@@ -68,5 +68,5 @@ foreach my $update (@updates) {
}
print "OK, done.<p>\n";
-print "<A HREF=\"editowners.cgi\">Edit the owners some more.</A>\n<P>\n";
-print "<A HREF=\"query.cgi\">Go back to the query page.</A>\n";
+print "<a href=editowners.cgi>Edit the owners some more.</a><p>\n";
+print "<a href=query.cgi>Go back to the query page.</a>\n";
diff --git a/doeditparams.cgi b/doeditparams.cgi
index 514fdbd7d..e43fd73ce 100755
--- a/doeditparams.cgi
+++ b/doeditparams.cgi
@@ -70,6 +70,6 @@ foreach my $i (@::param_list) {
WriteParams();
print "OK, done.<p>\n";
-print "<A HREF=\"editparams.cgi\">Edit the params some more.</A>\n<P>\n";
-print "<A HREF=\"query.cgi\">Go back to the query page.</A>\n";
+print "<a href=editparams.cgi>Edit the params some more.</a><p>\n";
+print "<a href=query.cgi>Go back to the query page.</a>\n";
diff --git a/editowners.cgi b/editowners.cgi
index 647bdeda0..1bfb6ac13 100755
--- a/editowners.cgi
+++ b/editowners.cgi
@@ -44,9 +44,9 @@ PutHeader("Edit Component Owners");
print "This lets you edit the owners of the program components of bugzilla.\n";
-print "<FORM METHOD=\"POST\" ACTION=\"doeditowners.cgi\">\n<TABLE>\n";
+print "<form method=post action=doeditowners.cgi><table>\n";
-my $rowbreak = "<TR><TD COLSPAN=\"2\"><HR></TD></TR>";
+my $rowbreak = "<tr><td colspan=2><hr></td></tr>";
SendSQL("select program, value, initialowner from components order by program, value");
@@ -56,17 +56,17 @@ my $curProgram = "";
while (@line = FetchSQLData()) {
if ($line[0] ne $curProgram) {
print $rowbreak;
- print "<TR><TH ALIGN=\"RIGHT\" VALIGN=\"TOP\">$line[0]:</TH><TD></TD></TR>\n";
+ print "<tr><th align=right valign=top>$line[0]:</th><td></td></tr>\n";
$curProgram = $line[0];
}
- print "<TR><TD VALIGN=\"TOP\">$line[1]</TD><TD><INPUT SIZE=\"80\" ";
+ print "<tr><td valign = top>$line[1]</td><td><input size=80 ";
print "name=\"$line[0]_$line[1]\" value=\"$line[2]\"></td></tr>\n";
}
print "</table>\n";
-print "<INPUT TYPE=\"submit\" VALUE=\"Submit changes\">\n";
+print "<input type=submit value=\"Submit changes\">\n";
print "</form>\n";
-print "<P><A HREF=\"query.cgi\">Skip all this, and go back to the query page</A>\n";
+print "<p><a href=query.cgi>Skip all this, and go back to the query page</a>\n";
diff --git a/editparams.cgi b/editparams.cgi
index b00071956..f926c0ae9 100755
--- a/editparams.cgi
+++ b/editparams.cgi
@@ -49,23 +49,23 @@ print "Be careful!\n";
print "<p>\n";
print "Any item you check Reset on will get reset to its default value.\n";
-print "<FORM METHOD=\"POST\" ACTION=\"doeditparams.cgi\">\n<TABLE>\n";
+print "<form method=post action=doeditparams.cgi><table>\n";
-my $rowbreak = "<TR><TD COLSPAN=\"2\"><HR></TD></TR>";
+my $rowbreak = "<tr><td colspan=2><hr></td></tr>";
print $rowbreak;
foreach my $i (@::param_list) {
- print "<TR><TH ALIGN=\"RIGHT\" VALIGN=\"TOP\">$i:</TH><TD>$::param_desc{$i}</TD></TR>\n";
- print "<TR><TD VALIGN=\"TOP\"><INPUT TYPE=\"checkbox\" NAME=\"reset-$i\">Reset</TD><TD>\n";
+ print "<tr><th align=right valign=top>$i:</th><td>$::param_desc{$i}</td></tr>\n";
+ print "<tr><td valign=top><input type=checkbox name=reset-$i>Reset</td><td>\n";
my $value = Param($i);
SWITCH: for ($::param_type{$i}) {
/^t$/ && do {
- print "<INPUT SIZE=\"80\" NAME=\"$i\" VALUE=\"" .
+ print "<input size=80 name=$i value=\"" .
value_quote($value) . '">\n';
last SWITCH;
};
/^l$/ && do {
- print "<TEXTAREA WRAP=\"HARD\" NAME=\"$i\" ROWS=\"10\" COLS=\"80\">" .
+ print "<textarea wrap=hard name=$i rows=10 cols=80>" .
value_quote($value) . "</textarea>\n";
last SWITCH;
};
@@ -79,18 +79,18 @@ foreach my $i (@::param_list) {
$on = "";
$off = "checked";
}
- print "<INPUT TYPE=\"radio\" NAME=\"$i\" VALUE=\"1\" $on>On\n";
- print "<INPUT TYPE=\"radio\" NAME=\"$i\" VALUE=\"0\" $off>Off\n";
+ print "<input type=radio name=$i value=1 $on>On\n";
+ print "<input type=radio name=$i value=0 $off>Off\n";
last SWITCH;
};
# DEFAULT
- print "<FONT COLOR=\"red\"><BLINK>Unknown param type $::param_type{$i}!!!</BLINK></FONT>\n";
+ print "<font color=red><blink>Unknown param type $::param_type{$i}!!!</blink></font>\n";
}
print "</td></tr>\n";
print $rowbreak;
}
-print "<TR><TH ALIGN=\"RIGHT\" VALIGN=\"TOP\">version:</TH><TD>
+print "<tr><th align=right valign=top>version:</th><td>
What version of Bugzilla this is. This can't be modified here, but
<tt>%version%</tt> can be used as a parameter in places that understand
such parameters</td></tr>
@@ -98,9 +98,9 @@ such parameters</td></tr>
print "</table>\n";
-print "<INPUT TYPE=\"reset\" VALUE=\"Reset form\"><BR>\n";
-print "<INPUT TYPE=\"submit\" VALUE=\"Submit changes\">\n";
+print "<input type=reset value=\"Reset form\"><br>\n";
+print "<input type=submit value=\"Submit changes\">\n";
print "</form>\n";
-print "<P><A HREF=\"query.cgi\">Skip all this, and go back to the query page</A>\n";
+print "<p><a href=query.cgi>Skip all this, and go back to the query page</a>\n";
diff --git a/enter_bug.cgi b/enter_bug.cgi
index 893a992da..529c394b8 100755
--- a/enter_bug.cgi
+++ b/enter_bug.cgi
@@ -43,9 +43,9 @@ if (!defined $::FORM{'product'}) {
print "a bug.</H2>\n";
print "<table>";
foreach my $p (sort (@prodlist)) {
- print "<TR><TH ALIGN=\"RIGHT\" VALIGN=\"TOP\"><A HREF=\"enter_bug.cgi?product=" . url_quote($p) . "\"&$::buffer>$p</A>:</TH>\n";
+ print "<tr><th align=right valign=top><a href=\"enter_bug.cgi?product=" . url_quote($p) . "\"&$::buffer>$p</a>:</th>\n";
if (defined $::proddesc{$p}) {
- print "<TD VALIGN=\"TOP\">$::proddesc{$p}</TD>\n";
+ print "<td valign=top>$::proddesc{$p}</td>\n";
}
print "</tr>";
}
@@ -55,7 +55,7 @@ if (!defined $::FORM{'product'}) {
$::FORM{'product'} = $prodlist[0];
}
-my $product = url_decode($::FORM{'product'});
+my $product = $::FORM{'product'};
confirm_login();
@@ -149,7 +149,7 @@ GetVersionTable();
my $assign_element = GeneratePersonInput('assigned_to', 1,
formvalue('assigned_to'));
-my $cc_element = GeneratePeopleInput('cc', 45, formvalue('cc'));
+my $cc_element = GeneratePeopleInput('cc', formvalue('cc'));
my $priority_popup = make_popup('priority', \@::legal_priority,
diff --git a/post_bug.cgi b/post_bug.cgi
index 52885a535..3ea3b070f 100755
--- a/post_bug.cgi
+++ b/post_bug.cgi
@@ -18,7 +18,6 @@
# Netscape Communications Corporation. All Rights Reserved.
#
# Contributor(s): Terry Weissman <terry@mozilla.org>
-# Andrew Anderson <andrew@redhat.com>
use diagnostics;
@@ -35,19 +34,17 @@ $zz = $zz . $zz;
confirm_login();
-my $platform = url_quote($::FORM{'product'});
-my $version = url_quote($::FORM{'version'});
-
-print "Set-Cookie: PLATFORM=$platform ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT\n";
-print "Set-Cookie: VERSION-$platform=$version ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT\n";
+print "Set-Cookie: PLATFORM=$::FORM{'product'} ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT\n";
+print "Set-Cookie: VERSION-$::FORM{'product'}=$::FORM{'version'} ; path=/ ; expires=Sun, 30-Jun-2029 00:00:00 GMT\n";
print "Content-type: text/html\n\n";
if (defined $::FORM{'maketemplate'}) {
- PutHeader("Bookmarks are your friend.", "Template constructed.");
+ print "<TITLE>Bookmarks are your friend.</TITLE>\n";
+ print "<H1>Template constructed.</H1>\n";
my $url = "enter_bug.cgi?$::buffer";
- print "If you put a bookmark <A HREF=\"$url\">to this link</A>, it will\n";
+ print "If you put a bookmark <a href=\"$url\">to this link</a>, it will\n";
print "bring up the submit-a-new-bug page with the fields initialized\n";
print "as you've requested.\n";
exit;
@@ -121,8 +118,6 @@ foreach my $person (keys %ccids) {
print "<H2>Changes Submitted</H2>\n";
print "<A HREF=\"show_bug.cgi?id=$id\">Show BUG# $id</A>\n";
print "<BR><A HREF=\"query.cgi\">Back To Query Page</A>\n";
-print "<BR><A HREF=\"enter_bug.cgi?product=" . url_quote($::FORM{'product'}). "\">Enter a new bug</A>\n";
-
system("./processmail $id < /dev/null > /dev/null 2> /dev/null &");
exit;
diff --git a/process_bug.cgi b/process_bug.cgi
index 84750e0e6..6755ce91c 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -37,14 +37,11 @@ print "Content-type: text/html\n\n";
GetVersionTable();
if ($::FORM{'product'} ne $::dontchange) {
- my $prod = url_decode($::FORM{'product'});
- my $version = url_decode($::FORM{'version'});
- my $component = url_decode($::FORM{'component'});
- my $vok = lsearch($::versions{$prod}, $version) >= 0;
- my $cok = lsearch($::components{$prod}, $component) >= 0;
+ my $prod = $::FORM{'product'};
+ my $vok = lsearch($::versions{$prod}, $::FORM{'version'}) >= 0;
+ my $cok = lsearch($::components{$prod}, $::FORM{'component'}) >= 0;
if (!$vok || !$cok) {
print "<H1>Changing product means changing version and component.</H1>\n";
-
print "You have chosen a new product, and now the version and/or\n";
print "component fields are not correct. (Or, possibly, the bug did\n";
print "not have a valid component or version field in the first place.)\n";
@@ -52,26 +49,26 @@ if ($::FORM{'product'} ne $::dontchange) {
print "<form>\n";
print "<table>\n";
print "<tr>\n";
- print "<td align=\"right\"><b>Product:</b></td>\n";
+ print "<td align=right><b>Product:</b></td>\n";
print "<td>$prod</td>\n";
print "</tr><tr>\n";
- print "<td align=\"right\"><b>Version:</b></td>\n";
- print "<td>" . Version_element($version, $prod) . "</td>\n";
+ print "<td align=right><b>Version:</b></td>\n";
+ print "<td>" . Version_element($::FORM{'version'}, $prod) . "</td>\n";
print "</tr><tr>\n";
- print "<td align=\"right\"><b>Component:</b></td>\n";
- print "<td>" . Component_element($component, $prod) . "</td>\n";
+ print "<td align=right><b>Component:</b></td>\n";
+ print "<td>" . Component_element($::FORM{'component'}, $prod) . "</td>\n";
print "</tr>\n";
print "</table>\n";
foreach my $i (keys %::FORM) {
if ($i ne 'version' && $i ne 'component') {
- print "<input type=\"hidden\" name=\"$i\" value=\"" .
+ print "<input type=hidden name=$i value=\"" .
value_quote($::FORM{$i}) . "\">\n";
}
}
- print "<input type=\"submit\" value=\"Commit\">\n";
+ print "<input type=submit value=Commit>\n";
print "</form>\n";
print "</hr>\n";
- print "<A HREF=\"query.cgi\">Cancel all this and go back to the query page.</A>\n";
+ print "<a href=query.cgi>Cancel all this and go back to the query page.</a>\n";
exit;
}
}
@@ -92,6 +89,11 @@ if (!defined $::FORM{'who'}) {
$::FORM{'who'} = $::COOKIE{'Bugzilla_login'};
}
+print "<TITLE>Update Bug " . join(" ", @idlist) . "</TITLE>\n";
+if (defined $::FORM{'id'}) {
+ navigation_header();
+}
+print "<HR>\n";
$::query = "update bugs\nset";
$::comma = "";
umask(0);
@@ -117,7 +119,6 @@ sub ChangeResolution {
}
}
-ConnectToDatabase();
foreach my $field ("rep_platform", "priority", "bug_severity", "url",
"summary", "component", "bug_file_loc", "short_desc",
@@ -125,11 +126,15 @@ foreach my $field ("rep_platform", "priority", "bug_severity", "url",
if (defined $::FORM{$field}) {
if ($::FORM{$field} ne $::dontchange) {
DoComma();
- $::query .= "$field = " . SqlQuote(url_decode($::FORM{$field}));
+ $::query .= "$field = " . SqlQuote($::FORM{$field});
}
}
}
+
+
+ConnectToDatabase();
+
SWITCH: for ($::FORM{'knob'}) {
/^none$/ && do {
last SWITCH;
@@ -162,8 +167,8 @@ SWITCH: for ($::FORM{'knob'}) {
}
ChangeStatus('NEW');
SendSQL("select initialowner from components where program=" .
- SqlQuote(url_decode($::FORM{'product'})) . " and value=" .
- SqlQuote(url_decode($::FORM{'component'})));
+ SqlQuote($::FORM{'product'}) . " and value=" .
+ SqlQuote($::FORM{'component'}));
my $newname = FetchOneColumn();
my $newid = DBNameToIdAndCheck($newname, 1);
DoComma();
@@ -172,7 +177,6 @@ SWITCH: for ($::FORM{'knob'}) {
};
/^reopen$/ && do {
ChangeStatus('REOPENED');
- ChangeResolution('');
last SWITCH;
};
/^verify$/ && do {
@@ -193,8 +197,7 @@ SWITCH: for ($::FORM{'knob'}) {
exit;
}
if ($::FORM{'dup_id'} == $::FORM{'id'}) {
- PutHeader("Nice try.");
- print "But it doesn't really make sense to mark a\n";
+ print "Nice try. But it doesn't really make sense to mark a\n";
print "bug as a duplicate of itself, does it?\n";
exit;
}
@@ -210,7 +213,6 @@ SWITCH: for ($::FORM{'knob'}) {
if ($#idlist < 0) {
- PutHeader("Nothing to modify");
print "You apparently didn't choose any bugs to modify.\n";
print "<p>Click <b>Back</b> and try again.\n";
exit;
@@ -229,7 +231,7 @@ my $basequery = $::query;
sub SnapShotBug {
my ($id) = (@_);
SendSQL("select " . join(',', @::log_columns) .
- " from bugs where bug_id = '" . $id . "'");
+ " from bugs where bug_id = $id");
return FetchSQLData();
}
@@ -238,7 +240,7 @@ foreach my $id (@idlist) {
SendSQL("lock tables bugs write, bugs_activity write, cc write, profiles write");
my @oldvalues = SnapShotBug($id);
- my $query = "$basequery\nwhere bug_id = '" . $id . "'";
+ my $query = "$basequery\nwhere bug_id = $id";
# print "<PRE>$query</PRE>\n";
@@ -259,7 +261,7 @@ foreach my $id (@idlist) {
}
}
- SendSQL("delete from cc where bug_id = '" . $id . "'");
+ SendSQL("delete from cc where bug_id = $id");
foreach my $ccid (keys %ccids) {
SendSQL("insert into cc (bug_id, who) values ($id, $ccid)");
}
@@ -274,8 +276,7 @@ foreach my $id (@idlist) {
if ($old ne $new) {
if (!defined $whoid) {
$whoid = DBNameToIdAndCheck($::FORM{'who'});
- $query = "select delta_ts from bugs where bug_id = '" . $id . "'";
- SendSQL($query);
+ SendSQL("select delta_ts from bugs where bug_id = $id");
$timestamp = FetchOneColumn();
}
if ($col eq 'assigned_to') {
@@ -286,21 +287,13 @@ foreach my $id (@idlist) {
$old = SqlQuote($old);
$new = SqlQuote($new);
my $q = "insert into bugs_activity (bug_id,who,when,field,oldvalue,newvalue) values ($id,$whoid,$timestamp,$col,$old,$new)";
- # print "<pre>$q</pre>";
+ # puts "<pre>$q</pre>"
SendSQL($q);
}
}
-
- PutHeader("Changes submitted for bug $::FORM{'id'}",
- "Changes Submitted", $::FORM{'id'});
- if (defined $::FORM{'id'}) {
- navigation_header();
- }
- print "<HR>\n<P>\n";
- print "<A HREF=\"show_bug.cgi?id=$id\">Back To BUG# $id</A>\n";
- print "<BR><A HREF=\"query.cgi\">Back To Query Page</A>\n";
- print "<BR><A HREF=\"enter_bug.cgi\">Enter a new bug</A>\n";
+ print "<TABLE BORDER=1><TD><H1>Changes Submitted</H1>\n";
+ print "<TD><A HREF=\"show_bug.cgi?id=$id\">Back To BUG# $id</A></TABLE>\n";
SendSQL("unlock tables");
diff --git a/query.cgi b/query.cgi
index ccac365de..b24dee0f3 100755
--- a/query.cgi
+++ b/query.cgi
@@ -99,8 +99,8 @@ print "Set-Cookie: BUGLIST=
Content-type: text/html\n\n";
GetVersionTable();
-my $who = GeneratePeopleInput("assigned_to", 45, $default{"assigned_to"});
-my $reporter = GeneratePeopleInput("reporter", 45, $default{"reporter"});
+my $who = GeneratePeopleInput("assigned_to", $default{"assigned_to"});
+my $reporter = GeneratePeopleInput("reporter", $default{"reporter"});
# Muck the "legal product" list so that the default one is always first (and
@@ -118,102 +118,97 @@ PutHeader("Bugzilla Query Page", "Query Page");
push @::legal_resolution, "---"; # Oy, what a hack.
print "
-<FORM NAME=\"queryForm\" METHOD=\"GET\" ACTION=\"buglist.cgi\">
+<FORM NAME=queryForm METHOD=GET ACTION=\"buglist.cgi\">
<table>
<tr>
-<th align=\"left\"><A HREF=\"bug_status.html\">Status</a>:</th>
-<th align=\"left\"><A HREF=\"bug_status.html\">Resolution</a>:</th>
-<th align=\"left\"><A HREF=\"bug_status.html#rep_platform\">Platform</a>:</th>
-<th align=\"left\"><A HREF=\"bug_status.html#priority\">Priority</a>:</th>
-<th align=\"left\"><A HREF=\"bug_status.html#severity\">Severity</a>:</th>
+<th align=left><A HREF=\"bug_status.html\">Status</a>:</th>
+<th align=left><A HREF=\"bug_status.html\">Resolution</a>:</th>
+<th align=left><A HREF=\"bug_status.html#rep_platform\">Platform</a>:</th>
+<th align=left><A HREF=\"bug_status.html#priority\">Priority</a>:</th>
+<th align=left><A HREF=\"bug_status.html#severity\">Severity</a>:</th>
</tr>
<tr>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"bug_status\" MULTIPLE SIZE=\"7\">
+<td align=left valign=top>
+<SELECT NAME=\"bug_status\" MULTIPLE SIZE=7>
@{[make_options(\@::legal_bug_status, $default{'bug_status'}, $type{'bug_status'})]}
-</SELECT><P>
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"resolution\" MULTIPLE SIZE=\"7\">
+<td align=left valign=top>
+<SELECT NAME=\"resolution\" MULTIPLE SIZE=7>
@{[make_options(\@::legal_resolution, $default{'resolution'}, $type{'resolution'})]}
-</SELECT><P>
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"rep_platform\" MULTIPLE SIZE=\"7\">
+<td align=left valign=top>
+<SELECT NAME=\"rep_platform\" MULTIPLE SIZE=7>
@{[make_options(\@::legal_platform, $default{'rep_platform'}, $type{'rep_platform'})]}
-</SELECT><P>
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"priority\" MULTIPLE SIZE=\"7\">
+<td align=left valign=top>
+<SELECT NAME=\"priority\" MULTIPLE SIZE=7>
@{[make_options(\@::legal_priority, $default{'priority'}, $type{'priority'})]}
-</SELECT><P>
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"bug_severity\" MULTIPLE SIZE=\"7\">
+<td align=left valign=top>
+<SELECT NAME=\"bug_severity\" MULTIPLE SIZE=7>
@{[make_options(\@::legal_severity, $default{'bug_severity'}, $type{'bug_severity'})]}
-</SELECT><P>
+</SELECT>
</tr>
</table>
<p>
<TABLE>
- <TR>
- <TD ALIGN=\"RIGHT\">
- <A HREF=\"bug_status.html#assigned_to\"><B>Assigned To:</B></A>
- <TD>$who
+<TR><TD ALIGN=RIGHT><B><A HREF=\"bug_status.html#assigned_to\">Assigned To:</a></B><TD>$who
+
<p>
- <TR>
- <TD ALIGN=\"RIGHT\">
- <A HREF=\"bug_status.html#reporter\"><B>Reporter:</B></A>
- <TD>$reporter
+<TR><TD ALIGN=RIGHT><B>Reporter:</B><TD>$reporter
</TABLE>
-<NOBR>Changed in the last <INPUT NAME=\"changedin\" SIZE=\"2\"> days.</NOBR>
+<NOBR>Changed in the last <INPUT NAME=changedin SIZE=2> days.</NOBR>
<P>
<table>
<tr>
-<TH ALIGN=\"LEFT\">Program:</th>
-<TH ALIGN=\"LEFT\">Version:</th>
-<TH ALIGN=\"LEFT\">Component:</th>
+<TH ALIGN=LEFT>Program:</th>
+<TH ALIGN=LEFT>Version:</th>
+<TH ALIGN=LEFT>Component:</th>
</tr>
<tr>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"product\" MULTIPLE SIZE=\"5\">
-@{[make_options(\@::legal_product, url_decode($default{'product'}), $type{'product'})]}
-</SELECT><P>
+<td align=left valign=top>
+<SELECT NAME=\"product\" MULTIPLE SIZE=5>
+@{[make_options(\@::legal_product, $default{'product'}, $type{'product'})]}
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"version\" MULTIPLE SIZE=\"5\">
+<td align=left valign=top>
+<SELECT NAME=\"version\" MULTIPLE SIZE=5>
@{[make_options(\@::legal_versions, $default{'version'}, $type{'version'})]}
-</SELECT><P>
+</SELECT>
</td>
-<td align=\"left\" valign=\"top\">
-<SELECT NAME=\"component\" MULTIPLE SIZE=\"5\">
+<td align=left valign=top>
+<SELECT NAME=\"component\" MULTIPLE SIZE=5>
@{[make_options(\@::legal_components, $default{'component'}, $type{'component'})]}
-</SELECT><P>
+</SELECT>
</td>
</tr>
</table>
-<table border=\"0\">
+<table border=0>
<tr>
-<td align=\"right\">Summary:</td>
-<td><input name=\"short_desc\" size=\"30\"></td>
-<td><input type=\"radio\" name=\"short_desc_type\" value=\"substr\" checked>Substring</td>
-<td><input type=\"radio\" name=\"short_desc_type\" value=\"regexp\">Regexp</td>
+<td align=right>Summary:</td>
+<td><input name=short_desc size=30></td>
+<td><input type=radio name=short_desc_type value=substr checked>Substring</td>
+<td><input type=radio name=short_desc_type value=regexp>Regexp</td>
</tr>
<tr>
-<td align=\"right\">Description:</td>
-<td><input name=\"long_desc\" size=\"30\"></td>
-<td><input type=\"radio\" name=\"long_desc_type\" value=\"substr\" checked>Substring</td>
-<td><input type=\"radio\" name=\"long_desc_type\" value=\"regexp\">Regexp</td>
+<td align=right>Description:</td>
+<td><input name=long_desc size=30></td>
+<td><input type=radio name=long_desc_type value=substr checked>Substring</td>
+<td><input type=radio name=long_desc_type value=regexp>Regexp</td>
</tr>
</table>
<p>
@@ -221,27 +216,27 @@ print "
<BR>
-<INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"doit\" CHECKED> Run this query
+<INPUT TYPE=radio NAME=cmdtype VALUE=doit CHECKED> Run this query
<BR>
";
if ($namelist ne "") {
print "
-<table cellspacing=\"0\" cellpadding=\"0\"><tr>
-<td><INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"editnamed\"> Load the remembered query:</td>
-<td rowspan=\"3\"><select name=\"namedcmd\">$namelist</select>
+<table cellspacing=0 cellpadding=0><tr>
+<td><INPUT TYPE=radio NAME=cmdtype VALUE=editnamed> Load the remembered query:</td>
+<td rowspan=3><select name=namedcmd>$namelist</select>
</tr><tr>
-<td><INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"runnamed\"> Run the remembered query:</td>
+<td><INPUT TYPE=radio NAME=cmdtype VALUE=runnamed> Run the remembered query:</td>
</tr><tr>
-<td><INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"forgetnamed\"> Forget the remembered query:</td>
+<td><INPUT TYPE=radio NAME=cmdtype VALUE=forgetnamed> Forget the remembered query:</td>
</tr></table>"
}
print "
-<INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"asdefault\"> Remember this as the default query
+<INPUT TYPE=radio NAME=cmdtype VALUE=asdefault> Remember this as the default query
<BR>
-<INPUT TYPE=\"radio\" NAME=\"cmdtype\" VALUE=\"asnamed\"> Remember this query, and name it:
-<INPUT TYPE=\"text\" NAME=\"newqueryname\">
+<INPUT TYPE=radio NAME=cmdtype VALUE=asnamed> Remember this query, and name it:
+<INPUT TYPE=text NAME=newqueryname>
<BR>
<NOBR><B>Sort By:</B>
@@ -252,7 +247,7 @@ print "
</SELECT></NOBR>
<INPUT TYPE=\"submit\" VALUE=\"Submit\">
<INPUT TYPE=\"reset\" VALUE=\"Reset back to the default query\">
-<INPUT TYPE=\"hidden\" name=\"form_name\" VALUE=\"query\">
+<INPUT TYPE=hidden name=form_name VALUE=query>
<BR>Give me a <A HREF=\"help.html\">clue</A> about how to use this form.
</CENTER>
</FORM>
@@ -262,14 +257,14 @@ print "
if (defined $::COOKIE{"Bugzilla_login"}) {
if ($::COOKIE{"Bugzilla_login"} eq Param("maintainer")) {
- print "<a href=\"editparams.cgi\">Edit Bugzilla operating parameters</a><br>\n";
- print "<a href=\"editowners.cgi\">Edit Bugzilla component owners</a><br>\n";
+ print "<a href=editparams.cgi>Edit Bugzilla operating parameters</a><br>\n";
+ print "<a href=editowners.cgi>Edit Bugzilla component owners</a><br>\n";
}
- print "<a href=\"relogin.cgi\">Log in as someone besides <b>$::COOKIE{'Bugzilla_login'}</b></a><br>\n";
+ print "<a href=relogin.cgi>Log in as someone besides <b>$::COOKIE{'Bugzilla_login'}</b></a><br>\n";
}
-print "<a href=\"changepassword.cgi\">Change your password.</a><br>\n";
-print "<a href=\"enter_bug.cgi\">Enter a new bug.</a><br>\n";
-print "<a href=\"reports.cgi\">Bug reports.</a><br>\n";
+print "<a href=changepassword.cgi>Change your password.</a><br>\n";
+print "<a href=\"enter_bug.cgi\">Create a new bug.</a><br>\n";
+print "<a href=\"reports.cgi\">Bug reports</a><br>\n";
diff --git a/relogin.cgi b/relogin.cgi
index 0c74319a5..5ef523945 100755
--- a/relogin.cgi
+++ b/relogin.cgi
@@ -30,13 +30,11 @@ Set-Cookie: Bugzilla_logincookie= ; path=/; expires=Sun, 30-Jun-80 00:00:00 GMT
Set-Cookie: Bugzilla_password= ; path=/; expires=Sun, 30-Jun-80 00:00:00 GMT
Content-type: text/html
-";
-PutHeader("Your login has been forgotten");
-print "
+<H1>Your login has been forgotten.</H1>
The cookie that was remembering your login is now gone. The next time you
do an action that requires a login, you will be prompted for it.
<p>
-<A HREF=\"query.cgi\">Back to the query page.</A>
+<a href=query.cgi>Back to the query page.</a>
";
exit;
diff --git a/reports.cgi b/reports.cgi
index 12ff00c7f..7456c054e 100755
--- a/reports.cgi
+++ b/reports.cgi
@@ -156,12 +156,11 @@ FIN
sub most_doomed
{
my $when = localtime (time);
- my $product = url_decode($::FORM{'product'});
print <<FIN;
<center>
<h1>
-Bug Report for $product
+Bug Report for $::FORM{'product'}
</h1>
$when<p>
FIN
@@ -180,7 +179,7 @@ from bugs,
versions projector
where bugs.assigned_to = assign.userid
and bugs.reporter = report.userid
-and bugs.product='$product'
+and bugs.product='$::FORM{'product'}'
and
(
bugs.bug_status = 'NEW' or
@@ -394,13 +393,13 @@ sub header
FIN
}
-sub show_chart {
- my $when = localtime (time);
- my $product = url_decode($::FORM{'product'});
+sub show_chart
+ {
+ my $when = localtime (time);
- if (! is_legal_product($product))
+ if (! is_legal_product ($::FORM{'product'}))
{
- &die_politely ("Unknown product: $product");
+ &die_politely ("Unknown product: $::FORM{'product'}");
}
print <<FIN;
@@ -455,7 +454,7 @@ FIN
my %settings =
(
- "title" => "Bug Charts for $product",
+ "title" => "Bug Charts for $::FORM{'product'}",
"x_label" => "Dates",
"y_label" => "Bug Count",
"legend_labels" => \@labels,
@@ -477,7 +476,6 @@ FIN
sub die_politely
{
my $msg = shift;
- my $product = url_decode($::FORM{'product'});
print <<FIN;
<p>
@@ -486,7 +484,7 @@ sub die_politely
<td align=center>
<font color=blue>Sorry, but ...</font>
<p>
-There is no graph available for <b>$product</b><p>
+There is no graph available for <b>$::FORM{'product'}</b><p>
<font size=-1>
$msg
diff --git a/sanitycheck.cgi b/sanitycheck.cgi
index 102e017b7..814a1f031 100755
--- a/sanitycheck.cgi
+++ b/sanitycheck.cgi
@@ -18,7 +18,6 @@
# Netscape Communications Corporation. All Rights Reserved.
#
# Contributor(s): Terry Weissman <terry@mozilla.org>
-# Andrew Anderson <andrew@redhat.com>
use diagnostics;
use strict;
@@ -43,7 +42,7 @@ sub Alert {
sub BugLink {
my ($id) = (@_);
- return "<a href=\"show_bug.cgi?id=$id\">$id</a>";
+ return "<a href='show_bug.cgi?id=$id'>$id</a>";
}
diff --git a/show_bug.cgi b/show_bug.cgi
index 06e02da2b..98829e356 100755
--- a/show_bug.cgi
+++ b/show_bug.cgi
@@ -22,19 +22,16 @@
use diagnostics;
use strict;
-use vars @::FORM;
+print "Content-type: text/html\n";
+print "\n";
require "CGI.pl";
-confirm_login();
-
-print "Content-type: text/html\n\n";
-
-if (!defined $::FORM{'id'} || $::FORM{'id'} eq "") {
- PutHeader("Search By Bug Number", "Search By Bug Number", "");
- print "<FORM METHOD=\"GET\" ACTION=\"show_bug.cgi\">\n";
+if (!defined $::FORM{'id'}) {
+ print "<H2>Search By Bug Number</H2>\n";
+ print "<FORM METHOD=GET ACTION=\"show_bug.cgi\">\n";
print "You may find a single bug by entering its bug id here: \n";
- print "<INPUT NAME=\"id\">\n";
+ print "<INPUT NAME=id>\n";
print "<INPUT TYPE=\"submit\" VALUE=\"Show Me This Bug\">\n";
print "</FORM>\n";
exit;