43821 - PutHeader has unbalanced <TABLE> tags

Status: VERIFIED FIXED

(Bugzilla :: Bugzilla-General, defect, P3)

Description

[tenthumbs]

Open any bug from your browser and look at the source. Line 32 duplicates line
31 like this:

</TD></TR></TABLE>
</TD></TR></TABLE>

Comment 1

[Zach Lipton [:zach]]

dup of 47251

*** This bug has been marked as a duplicate of 47251 ***

Comment 2

[tenthumbs]

I see nothing in bug 47251 that fixes this problem. Reopening.

Comment 3

[tenthumbs]

This really should be fixed.

Comment 4

[Dave Miller [:justdave]]

Although this could have been included in bug 47251, it wasn't.  Making the 
summary more specific so it doesn't get marked dupe again.  Adding 2.12 and 
copying Chris.

Comment 5

[Dave Miller [:justdave]]

I found this...  it's in CGI.pl rather than bug_form.pl.  The PutHeader() 
routine, to be exact.  Which means this affects every page on the site, not just 
the bug form.

I apologize for pasting the patch in here like this, but I'm away from my normal 
machine, and it's not easy to upload an attachment from this machine.

Index: CGI.pl
===================================================================
RCS file: /cvsroot/mozilla/webtools/bugzilla/CGI.pl,v
retrieving revision 1.68
diff -u -r1.68 CGI.pl
--- CGI.pl      2000/08/21 19:50:41     1.68
+++ CGI.pl      2000/09/04 03:40:01
@@ -795,8 +795,6 @@
     print "  <TD VALIGN=CENTER ALIGN=LEFT>\n";
 
     print "$h2\n";
-    print "</TD></TR></TABLE>\n";
-
     print "</TD></TR></TABLE>\n";
 
     if (Param("shutdownhtml")) {

Comment 6

[Dave Miller [:justdave]]

This has been checked in.

Comment 7

[Tara Hernandez]

Sorry for the spam, but I needed to be able to query for all of these correctly.

Comment 8

[Gervase Markham [:gerv]]

VERIFIED.

Gerv

Comment 9

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product