Fix for bug 76714: correctly nests the FORM and TABLE elements.

Patch by Gerv Markham <gerv@mozilla.org>.
r=myk@mozilla.org, no second review needed.

diff --git a/CGI.pl b/CGI.pl
index 67cca6f330590f97e8bceeecc1238e06a2d990a9..ee6dbb4c76764b6dac0428eab499fcde10b42858 100644 (file)
--- a/CGI.pl
+++ b/CGI.pl
@@ -1357,7 +1357,7 @@ Actions:
         my $mybugstemplate = Param("mybugstemplate");
         my %substs;
         $substs{'userid'} = url_quote($::COOKIE{"Bugzilla_login"});
-        $html .= "</TR><TR>";
+        $html .= "<TR>";
         $html .= "<TD>Preset Queries: </TD>";
         $html .= "<TD colspan=3>\n";
         if ($mybugslink) {
@@ -1374,7 +1374,7 @@ Actions:
             $html .= "<A HREF=\"buglist.cgi?&cmdtype=runnamed&namedcmd=" .
                      url_quote($name) . "\"><NOBR>$name</NOBR></A>";
         }
-        $html .= "</TR>\n<TR>";
+        $html .= "</TD></TR>\n";
     } else {
     $html .= "</TD><TD>&nbsp;</TD><TD valign=middle align=right>\n";
         $html .=
@@ -1383,8 +1383,8 @@ Actions:
             " | <NOBR><a href=query.cgi?GoAheadAndLogIn=1>Log in</a></NOBR>";
         $html .= "</TD></TR>";
     }
-    $html .= "</FORM>\n";
     $html .= "</TABLE>";                
+    $html .= "</FORM>\n";
     return $html;
 }