html bugs in 2.0.13

Bug #265716 reported by Chuqui-users
2
Affects Status Importance Assigned to Milestone
GNU Mailman
New
Low
Unassigned

Bug Description

Ran into some HTML bugs in 2.0.13. For instance, going to
the listinfo/mailman-subscribers page and subscribing, it
returns the "confirmation is required" warning.

It looks good, but view source it. An unterminated "<BODY"
tag, followed by a second body tag with a bgcolor, followed
by a third body tag a couple of lines later.

There are also two /body tags, and the <title> is in the
body, not in the head.

While it seems to format okay in browsers, it's seriously
broken HTML, and it's driving mod_layout nuts, since it
depends on the body tags to do header/footer insertions
right. I think there's one other generated page with similar
problems, too, but I'm not sure which it is yet.

[http://sourceforge.net/tracker/index.php?func=detail&aid=596215&group_id=103&atid=100103]

Tags: web-cgi
Revision history for this message
Netik-users (netik-users) wrote :

This also breaks things in an interesting way when you're
attempting to change the background color of the subscribe
results page (or any page, for that matter.)

The doc object is created in main is built in htmlformat.py:

<code>
        if self.title:
            output.append('%s<TITLE>%s</TITLE>' % (tab,
self.title))
        output.append('%s</HEAD>' % tab)
        output.append('%s<BODY' % tab)
</code>

The worst part is that this bug occurs on EVERY SINGLE PAGE
that mailman generates!

You can fix this serious bug by commenting out the line:
<pre>
        output.append('%s<BODY' % tab)
</pre>
in htmlformat.py, line #281 (at least in my version, which is
2.0.9.

Also, there is a secondary issue associated with page
generation; lots of calls to doc.Format immediately set the
bgcolor to #ffffff; There needs to be a way to set the global
(or per-list) background color and not hardcode it in this way.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.