From: myk%mozilla.org <>
Date: Wed, 22 May 2002 00:53:03 +0000 (+0000)
Subject: Fix for bug 47251: Make HTML output HTML 4.01 Transitional compliant.
X-Git-Tag: bugzilla-2.17.1~355
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f3b6e880a28c2f7fbe62702f5557f8b32fc10ca0;p=thirdparty%2Fbugzilla.git

Fix for bug 47251: Make HTML output HTML 4.01 Transitional compliant.
Patch by mental <xor@ivwnet.com>.
r=justdave,myk
---

diff --git a/template/en/default/account/email/confirm.html.tmpl b/template/en/default/account/email/confirm.html.tmpl
index ebba5c76f0..140c54e0d8 100644
--- a/template/en/default/account/email/confirm.html.tmpl
+++ b/template/en/default/account/email/confirm.html.tmpl
@@ -30,7 +30,7 @@
 </p>
 
 <form method="post" action="token.cgi">
-  <input type="hidden" name="t" value=[% token FILTER html %]>
+  <input type="hidden" name="t" value="[% token FILTER html %]">
   <input type="hidden" name="a" value="chgem">
   <table>
     <tr>
diff --git a/template/en/default/account/exists.html.tmpl b/template/en/default/account/exists.html.tmpl
index 5281b20cf3..3516dc7ccb 100644
--- a/template/en/default/account/exists.html.tmpl
+++ b/template/en/default/account/exists.html.tmpl
@@ -28,11 +28,11 @@
 %]
 
 <form method="get" action="token.cgi">
-  <input type="hidden" name="a" value="reqpw" />
-  <input type="hidden" name="loginname" value="[% login FILTER html %]" />
+  <input type="hidden" name="a" value="reqpw">
+  <input type="hidden" name="loginname" value="[% login FILTER html %]">
   A Bugzilla account for <tt>[% login FILTER html %]</tt> already exists. If you
   are the account holder and have forgotten your password, 
-  <input type="submit" value="submit a request to change it" />.
+  <input type="submit" value="submit a request to change it">.
 </form>
 
 [% PROCESS global/footer.html.tmpl %]
diff --git a/template/en/default/account/password/set-forgotten-password.html.tmpl b/template/en/default/account/password/set-forgotten-password.html.tmpl
index eb1abe746a..eaabf7dc4b 100644
--- a/template/en/default/account/password/set-forgotten-password.html.tmpl
+++ b/template/en/default/account/password/set-forgotten-password.html.tmpl
@@ -26,27 +26,27 @@
 </p>
 
 <form method="post" action="token.cgi">
-  <input type="hidden" name="t" value="[% token FILTER html %]" />
-  <input type="hidden" name="a" value="chgpw" />
+  <input type="hidden" name="t" value="[% token FILTER html %]">
+  <input type="hidden" name="a" value="chgpw">
   <table>
     <tr>
       <th align="right">New Password:</th>
       <td>
-        <input type="password" name="password" size="16" maxlength="16" />
+        <input type="password" name="password" size="16" maxlength="16">
       </td>
     </tr>
     
     <tr>
       <th align="right">New Password Again:</th>
       <td>
-        <input type="password" name="matchpassword" size="16" maxlength="16" />
+        <input type="password" name="matchpassword" size="16" maxlength="16">
       </td>
     </tr>
     
     <tr>
       <th align="right">&nbsp;</th>
       <td>
-        <input type="submit" value="Submit" />
+        <input type="submit" value="Submit">
       </td>
     </tr>
   </table>
diff --git a/template/en/default/account/prefs/account.html.tmpl b/template/en/default/account/prefs/account.html.tmpl
index e1675c7fb3..798bb8e404 100644
--- a/template/en/default/account/prefs/account.html.tmpl
+++ b/template/en/default/account/prefs/account.html.tmpl
@@ -36,8 +36,8 @@
     <th align="right">Password:</th>
     <td>
       <input type="hidden" name="Bugzilla_login" 
-             value="[% login FILTER html %]" />
-      <input type="password" name="Bugzilla_password" />
+             value="[% login FILTER html %]">
+      <input type="password" name="Bugzilla_password">
     </td>
   </tr>              
   <tr>
@@ -47,21 +47,21 @@
   <tr>
     <th align="right">New password:</th>
     <td>
-      <input type="password" name="new_password1" />
+      <input type="password" name="new_password1">
     </td>
   </tr>              
 
   <tr>
     <th align="right">Re-enter new password:</th>
     <td>
-      <input type="password" name="new_password2" />
+      <input type="password" name="new_password2">
     </td>
   </tr>              
 
   <tr>
     <th align="right">Your real name (optional, but encouraged):</th>
     <td>
-      <input size="35" name="realname" value="[% realname FILTER html %]" />
+      <input size="35" name="realname" value="[% realname FILTER html %]">
     </td>
   </tr>  
 
diff --git a/template/en/default/account/prefs/email.html.tmpl b/template/en/default/account/prefs/email.html.tmpl
index f7d529c7c9..da57a8713e 100644
--- a/template/en/default/account/prefs/email.html.tmpl
+++ b/template/en/default/account/prefs/email.html.tmpl
@@ -35,7 +35,7 @@
   [% IF Param('supportwatchers') %]
     <tr>
       <td colspan="4">
-        <hr />
+        <hr>
       </td>
     </tr>
 
@@ -52,7 +52,7 @@
     <tr>
       <th align="right">Users to watch:</th>
       <td>
-        <input size="35" name="watchedusers" value="[% watchedusers %]" />
+        <input size="35" name="watchedusers" value="[% watchedusers %]">
       </td>
     </tr>
   [% END %]
@@ -68,7 +68,7 @@
   </tr>
 </table>
 
-<hr />
+<hr>
 
 <table>
   <tr>
@@ -88,7 +88,7 @@
   </tr>
 </table>
 
-<hr />
+<hr>
 <b>Field/recipient specific options:</b>
 <br>
 <br>
diff --git a/template/en/default/account/prefs/footer.html.tmpl b/template/en/default/account/prefs/footer.html.tmpl
index ea5753f0e4..77ebd1e3ac 100644
--- a/template/en/default/account/prefs/footer.html.tmpl
+++ b/template/en/default/account/prefs/footer.html.tmpl
@@ -39,7 +39,7 @@
     </td>
   </tr>    
     
-  <input type="hidden" name="numqueries" value="[% queries.size %]" />
+  <input type="hidden" name="numqueries" value="[% queries.size %]">
   [% IF queries.size %]
     [% FOREACH query = queries %]
       <tr>
@@ -56,7 +56,7 @@
       </tr>
 
       <input type="hidden" name="name-[% loop.index %]" 
-             value="[% query.name FILTER html %]" />
+             value="[% query.name FILTER html %]">
     [% END %]
 
   [% ELSE %]
diff --git a/template/en/default/account/prefs/permissions.html.tmpl b/template/en/default/account/prefs/permissions.html.tmpl
index 7ab7f1df99..15bca6deb8 100644
--- a/template/en/default/account/prefs/permissions.html.tmpl
+++ b/template/en/default/account/prefs/permissions.html.tmpl
@@ -43,7 +43,7 @@
 
       [% IF set_bits.size %]
         And you can turn on or off the following bits for
-        <a HREF="editusers.cgi">other users</a>:
+        <a href="editusers.cgi">other users</a>:
         <p>
           <ul>
           [% FOREACH bit_description = set_bits %]
diff --git a/template/en/default/account/prefs/prefs.html.tmpl b/template/en/default/account/prefs/prefs.html.tmpl
index 93f004deaa..2a232f875a 100644
--- a/template/en/default/account/prefs/prefs.html.tmpl
+++ b/template/en/default/account/prefs/prefs.html.tmpl
@@ -71,7 +71,7 @@
           </td>
         [% ELSE %]
           <td align="center" bgcolor="#BBBBEE" class="unselected_tab">
-            <a HREF="userprefs.cgi?tab=[% tab.name %]">[% tab.description %]</a>
+            <a href="userprefs.cgi?tab=[% tab.name %]">[% tab.description %]</a>
           </td>
         [% END %]
        [% END %]
diff --git a/template/en/default/attachment/create.html.tmpl b/template/en/default/attachment/create.html.tmpl
index cb979b0c8e..332d0d5f02 100644
--- a/template/en/default/attachment/create.html.tmpl
+++ b/template/en/default/attachment/create.html.tmpl
@@ -83,7 +83,7 @@
         [% IF attachments.size %]
           [% FOREACH attachment = attachments %]
             <input type="checkbox" name="obsolete" value="[% attachment.id %]">
-            <a href="attachment.cgi?id=[% attachment.id %]&action=edit">[% attachment.id %]: [% attachment.description FILTER html %]</a><br>
+            <a href="attachment.cgi?id=[% attachment.id %]&amp;action=edit">[% attachment.id %]: [% attachment.description FILTER html %]</a><br>
           [% END %]
         [% ELSE %]
             [no attachments can be made obsolete]
diff --git a/template/en/default/attachment/created.html.tmpl b/template/en/default/attachment/created.html.tmpl
index 880630ba2c..56e71ccbac 100644
--- a/template/en/default/attachment/created.html.tmpl
+++ b/template/en/default/attachment/created.html.tmpl
@@ -38,7 +38,7 @@
   <tr>
     <td>
       <h2>
-        <a title="[% description FILTER html %]" href="attachment.cgi?id=[% attachid %]&action=edit">Attachment #[% attachid %]</a> 
+        <a title="[% description FILTER html %]" href="attachment.cgi?id=[% attachid %]&amp;action=edit">Attachment #[% attachid %]</a> 
         to <a href="show_bug.cgi?id=[% bugid %]">Bug #[% bugid %]</a> Created
       </h2>
 
@@ -49,7 +49,7 @@
           <b>Note:</b> Bugzilla automatically detected the content type
           <em>[% contenttype %]</em> for this attachment.  If this is 
           incorrect, correct the value by 
-          <a href="attachment.cgi?id=[% attachid %]&action=edit">editing the attachment</a>.
+          <a href="attachment.cgi?id=[% attachid %]&amp;action=edit">editing the attachment</a>.
         </p>
       [% END %]
     </td>
@@ -60,7 +60,7 @@
 </table>
 
 <p>
-<a href="attachment.cgi?bugid=[% bugid %]&action=enter">Create Another Attachment to Bug #[% bugid %]</a>
+<a href="attachment.cgi?bugid=[% bugid %]&amp;action=enter">Create Another Attachment to Bug #[% bugid %]</a>
 </p>
 
 [% PROCESS global/footer.html.tmpl %]
diff --git a/template/en/default/attachment/edit.html.tmpl b/template/en/default/attachment/edit.html.tmpl
index 6d29d6f03a..64346a0f5b 100644
--- a/template/en/default/attachment/edit.html.tmpl
+++ b/template/en/default/attachment/edit.html.tmpl
@@ -182,9 +182,9 @@
       [% IF isviewable %]
         <td width="75%">
           <textarea id="editFrame" name="comment" style="height: 400px; width: 100%; display: none;" cols="80" wrap="soft"></textarea>
-          <iframe id="viewFrame" src="attachment.cgi?id=[% attachid %]&action=view" style="height: 400px; width: 100%;">
+          <iframe id="viewFrame" src="attachment.cgi?id=[% attachid %]&amp;action=view" style="height: 400px; width: 100%;">
             <b>You cannot view the attachment while editing it because your browser does not support IFRAMEs.
-            <a href="attachment.cgi?id=[% attachid %]&action=view">View the attachment on a separate page</a>.</b>
+            <a href="attachment.cgi?id=[% attachid %]&amp;action=view">View the attachment on a separate page</a>.</b>
           </iframe>
           <script type="application/x-javascript" language="JavaScript">
             <!--
@@ -200,7 +200,7 @@
         <td id="noview" width="50%">
           <p><b>
             Attachment cannot be viewed because its MIME type is not either text/*, image/*, or application/vnd.mozilla.*.
-            <a href="attachment.cgi?id=[% attachid %]&action=view">Download the attachment instead</a>.
+            <a href="attachment.cgi?id=[% attachid %]&amp;action=view">Download the attachment instead</a>.
           </b></p>
         </td>
       [% END %]
@@ -214,7 +214,7 @@
     [% IF a == attachid %]
       #[% a %]
     [% ELSE %]
-      <a href="attachment.cgi?id=[% a %]&action=edit">#[% a %]</a>
+      <a href="attachment.cgi?id=[% a %]&amp;action=edit">#[% a %]</a>
     [% END %]
     [% "|" UNLESS loop.last() %]
   [% END %]
diff --git a/template/en/default/attachment/show-multiple.html.tmpl b/template/en/default/attachment/show-multiple.html.tmpl
index 83a0642a7c..b3c05f2e2c 100644
--- a/template/en/default/attachment/show-multiple.html.tmpl
+++ b/template/en/default/attachment/show-multiple.html.tmpl
@@ -73,15 +73,15 @@
       </td>
 
       <td valign="top">
-        <a href="attachment.cgi?id=[% a.attachid %]&action=edit">Edit</a> 
+        <a href="attachment.cgi?id=[% a.attachid %]&amp;action=edit">Edit</a> 
       </td>
     </tr>
   </table>
 
   [% IF a.isviewable %]
-    <iframe src="attachment.cgi?id=[% a.attachid %]&action=view" width="75%" height="350">
+    <iframe src="attachment.cgi?id=[% a.attachid %]&amp;action=view" width="75%" height="350">
       <b>You cannot view the attachment on this page because your browser does not support IFRAMEs.
-      <a href="attachment.cgi?id=[% a.attachid %]&action=view">View the attachment on a separate page</a>.</b>
+      <a href="attachment.cgi?id=[% a.attachid %]&amp;action=view">View the attachment on a separate page</a>.</b>
     </iframe>
   [% ELSE %]
     <p><b>
diff --git a/template/en/default/attachment/updated.html.tmpl b/template/en/default/attachment/updated.html.tmpl
index 5c2ffd10fe..2c7ceb24dc 100644
--- a/template/en/default/attachment/updated.html.tmpl
+++ b/template/en/default/attachment/updated.html.tmpl
@@ -37,7 +37,7 @@
   <tr>
     <td>
       <h2>Changes to 
-        <a href="attachment.cgi?id=[% attachid %]&action=edit">attachment [% attachid %]</a>
+        <a href="attachment.cgi?id=[% attachid %]&amp;action=edit">attachment [% attachid %]</a>
         of bug [% bugid %] submitted
       </h2>
       [% mailresults %]
diff --git a/template/en/default/bug/activity/show.html.tmpl b/template/en/default/bug/activity/show.html.tmpl
index 9bd0e0b379..2e44f89fd0 100644
--- a/template/en/default/bug/activity/show.html.tmpl
+++ b/template/en/default/bug/activity/show.html.tmpl
@@ -29,7 +29,7 @@
 [% PROCESS global/header.html.tmpl 
   title = "Changes made to bug $bug_id"
   h1 = "Activity log"
-  h2 = "Bug <a href='show_bug.cgi?id=$bug_id'>$bug_id</a>"
+  h2 = "Bug <a href=\"show_bug.cgi?id=$bug_id\">$bug_id</a>"
  %]
 
 <br>
diff --git a/template/en/default/bug/choose-xml.html.tmpl b/template/en/default/bug/choose-xml.html.tmpl
index 341df9414c..045ad7c58e 100644
--- a/template/en/default/bug/choose-xml.html.tmpl
+++ b/template/en/default/bug/choose-xml.html.tmpl
@@ -34,8 +34,8 @@
         Display bugs as XML by entering a list of bug numbers here:
       </td>
       <td>
-        <input name="id" size="30" />
-        <input type="submit" value="Display as XML" />
+        <input name="id" size="30">
+        <input type="submit" value="Display as XML">
       </td>
     </tr>
     
diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl
index a885f95583..c6ab1fdb0a 100644
--- a/template/en/default/bug/create/create.html.tmpl
+++ b/template/en/default/bug/create/create.html.tmpl
@@ -26,7 +26,7 @@
 %]
 
 <form method="post" action="post_bug.cgi">
-<input type="hidden" name="product" value="[% product FILTER html %]" />
+<input type="hidden" name="product" value="[% product FILTER html %]">
 
 <table cellspacing="2" cellpadding="0" border="0">
 
@@ -105,7 +105,7 @@
     [% sel = { description => 'Priority', name => 'priority' } %]
     [% INCLUDE select %]
   [% ELSE %]
-    <input type="hidden" name="priority" value="[% default.priority %]" />
+    <input type="hidden" name="priority" value="[% default.priority %]">
   [% END %]
 
   [% sel = { description => 'Severity', name => 'bug_severity' } %]
@@ -125,7 +125,7 @@
     <td colspan="2"></td>
   </tr>
 [% ELSE %]
-  <input type="hidden" name="bug_status" value="[% default.bug_status %]" />  
+  <input type="hidden" name="bug_status" value="[% default.bug_status %]">  
 [% END %]
 
   <tr>
@@ -136,7 +136,7 @@
     </td>
     <td colspan="3">
       <input name="assigned_to" size="32" 
-             value="[% assigned_to FILTER html %]" />
+             value="[% assigned_to FILTER html %]">
       (Leave blank to assign to default component owner)
     </td>
   </tr>
@@ -144,7 +144,7 @@
   <tr>
     <td align="right"><strong>Cc:</strong></td>
     <td colspan="3">
-      <input name="cc" size="45" value="[% cc FILTER html %]" />
+      <input name="cc" size="45" value="[% cc FILTER html %]">
     </td>
   </tr>
   
@@ -157,14 +157,14 @@
     <td align="right"<strong>URL:</strong></td>
     <td colspan="3">
       <input name="bug_file_loc" size="60" 
-             value="[% bug_file_loc FILTER html %]" />
+             value="[% bug_file_loc FILTER html %]">
     </td>
   </tr>
   
   <tr>
     <td align="right"><strong>Summary:</strong></td>
     <td colspan="3">
-      <input name="short_desc" size="60" value="[% short_desc FILTER html %]" />
+      <input name="short_desc" size="60" value="[% short_desc FILTER html %]">
     </td>
   </tr>
   
@@ -195,7 +195,7 @@
       [% FOREACH g = group %]
         &nbsp;&nbsp;&nbsp;&nbsp;
         <input type="checkbox" name="bit-[% g.bit %]" value="1"
-          [% " checked=\"checked\"" IF g.checked %] />[% g.description %]<br />
+          [% " checked=\"checked\"" IF g.checked %]>[% g.description %]<br>
       [% END %]
       <br>
     [% END %]
@@ -211,7 +211,7 @@
                return false; } return true;">
       &nbsp;&nbsp;&nbsp;&nbsp;
       <input type="submit" name="maketemplate" 
-             value="Remember values as bookmarkable template" />
+             value="Remember values as bookmarkable template">
     </td>
   </tr>
 
diff --git a/template/en/default/bug/dependency-graph.html.tmpl b/template/en/default/bug/dependency-graph.html.tmpl
index 3266f5fd79..8206a847bf 100644
--- a/template/en/default/bug/dependency-graph.html.tmpl
+++ b/template/en/default/bug/dependency-graph.html.tmpl
@@ -38,7 +38,7 @@
  
 [% IF NOT multiple_bugs AND NOT doall %]
   [% title = "$title for bug $bug_id"
-     h1 = "$h1 for bug <a href='show_bug.cgi?id=$bug_id'>$bug_id</a>" 
+     h1 = "$h1 for bug <a href=\"show_bug.cgi?id=$bug_id\">$bug_id</a>" 
    %]
 [% END %]
 
@@ -54,7 +54,7 @@
   <img src="[% image_url %]" usemap="#imagemap"> 
 [% ELSE %]
   <a href="[% map_url %]">
-    <img src="[% image_url %]" ismap> 
+    <img src="[% image_url %]" ismap="ismap"> 
   </a>
 [% END %]
 
diff --git a/template/en/default/bug/edit.html.tmpl b/template/en/default/bug/edit.html.tmpl
index 926b8415b8..e20b54121e 100644
--- a/template/en/default/bug/edit.html.tmpl
+++ b/template/en/default/bug/edit.html.tmpl
@@ -116,7 +116,7 @@
       </td>
       <td rowspan="4" valign="top"> 
       [% IF bug.cc %]
-        <select name="cc" multiple size="5">
+        <select name="cc" multiple="multiple" size="5">
         [% FOREACH c = bug.cc %]
           <option value="[% c FILTER html %]">[% c FILTER html %]</option>
         [% END %]
@@ -288,9 +288,9 @@
       </th>
       <td>
         [% bug.votes %]&nbsp;&nbsp;&nbsp;
-        <a href="votes.cgi?action=show_bug&bug_id=[% bug.bug_id %]">Show 
+        <a href="votes.cgi?action=show_bug&amp;bug_id=[% bug.bug_id %]">Show 
         votes for this bug</a>&nbsp;&nbsp;&nbsp;
-        <a href="votes.cgi?action=show_user&bug_id=[% bug.bug_id %]">Vote 
+        <a href="votes.cgi?action=show_user&amp;bug_id=[% bug.bug_id %]">Vote 
         for this bug</a>
       </td>
     </tr>
@@ -317,8 +317,8 @@
     [% FOREACH group = groups %]
       &nbsp;&nbsp;&nbsp;&nbsp;
       <input type="checkbox" name="bit-[% group.bit %]" value="1" 
-        [% " checked='checked'" IF group.ison %]
-        [% " disabled='disabled'" IF NOT group.ingroup %]>
+        [% " checked=\"checked\"" IF group.ison %]
+        [% " disabled=\"disabled\"" IF NOT group.ingroup %]>
       [% group.description %]
       <br>
     [% END %]
@@ -357,7 +357,7 @@
 [%# *** Knob *** %]
 
   <br>
-  <input type="radio" name="knob" value="none" checked>
+  <input type="radio" name="knob" value="none" checked="checked">
   Leave as <b>[% bug.bug_status FILTER html %]&nbsp;
               [% bug.resolution FILTER html %]</b>
   <br>
@@ -410,7 +410,7 @@
       [% knum = knum + 1 %]
 
       <input type="radio" name="knob" value="reassign"> 
-      <a href="bug_status.html#assigned_to">Reassign</A> bug to
+      <a href="bug_status.html#assigned_to">Reassign</a> bug to
       <input name="assigned_to" size="32" 
              onchange="if ((this.value != '[% bug.assigned_to_email FILTER js %]') && 
                             (this.value != '')) {
diff --git a/template/en/default/bug/process/verify-new-product.html.tmpl b/template/en/default/bug/process/verify-new-product.html.tmpl
index 8569158a74..4facc45c89 100644
--- a/template/en/default/bug/process/verify-new-product.html.tmpl
+++ b/template/en/default/bug/process/verify-new-product.html.tmpl
@@ -86,7 +86,7 @@
   <p>
     <input type="radio" name="addtonewgroup" value="no"><b>no</b><br>
     <input type="radio" name="addtonewgroup" value="yes"><b>yes</b><br>
-    <input type="radio" name="addtonewgroup" value="yesifinold" checked>
+    <input type="radio" name="addtonewgroup" value="yesifinold" checked="checked">
       <b>yes, but only if the bug was in its old product's group</b><br>
   </p>
 [% END %]
diff --git a/template/en/default/bug/show-multiple.html.tmpl b/template/en/default/bug/show-multiple.html.tmpl
index 0290cb6d44..0e73f4ad72 100644
--- a/template/en/default/bug/show-multiple.html.tmpl
+++ b/template/en/default/bug/show-multiple.html.tmpl
@@ -104,7 +104,7 @@
     <tr>
       <td colspan="4">
         <b>URL:</b>&nbsp;
-        <A HREF="[% bug.bug_file_loc %]">[% bug.bug_file_loc FILTER html %]</a> 
+        <a href="[% bug.bug_file_loc %]">[% bug.bug_file_loc FILTER html %]</a> 
     </tr>
     
     <tr>
diff --git a/template/en/default/bug/votes/list-for-bug.html.tmpl b/template/en/default/bug/votes/list-for-bug.html.tmpl
index cb68a2a474..379727f70e 100644
--- a/template/en/default/bug/votes/list-for-bug.html.tmpl
+++ b/template/en/default/bug/votes/list-for-bug.html.tmpl
@@ -29,7 +29,7 @@
   
 [% PROCESS global/header.html.tmpl
            title = "Show Votes"
-           h2 = "Bug <a href='show_bug.cgi?id=$bug_id'>$bug_id</a>"
+           h2 = "Bug <a href=\"show_bug.cgi?id=$bug_id\">$bug_id</a>"
  %]
  
 <table cellspacing="4">
@@ -41,7 +41,7 @@
   [% FOREACH user = users %]      
     <tr>
       <td>
-        <a href="votes.cgi?action=show_user&user=[% user.name %]">
+        <a href="votes.cgi?action=show_user&amp;user=[% user.name %]">
           [% user.name %]
         </a>
       </td>
diff --git a/template/en/default/bug/votes/list-for-user.html.tmpl b/template/en/default/bug/votes/list-for-user.html.tmpl
index 4797d71b75..5f442e0bc4 100644
--- a/template/en/default/bug/votes/list-for-user.html.tmpl
+++ b/template/en/default/bug/votes/list-for-user.html.tmpl
@@ -73,7 +73,7 @@
               [% "</strike>" IF NOT bug.opened %]
             </td>
             <td>
-              <a href="votes.cgi?action=show_bug&bug_id=[% bug.id %]">
+              <a href="votes.cgi?action=show_bug&amp;bug_id=[% bug.id %]">
                 [% bug.summary FILTER html %]
               </a>
             </td>
diff --git a/template/en/default/global/header.html.tmpl b/template/en/default/global/header.html.tmpl
index 3c79e22d13..8ae85f83df 100644
--- a/template/en/default/global/header.html.tmpl
+++ b/template/en/default/global/header.html.tmpl
@@ -91,7 +91,7 @@
         <td valign="top" align="left">
           <table border="0" cellpadding="0" cellspacing="2">
             <tr>
-              <td valign="top" align="left" nowrap>
+              <td valign="top" align="left" nowrap="nowrap">
                 <font size="+1"><b>[% h1 %]</b></font>
               </td>
             </tr>
diff --git a/template/en/default/index.html.tmpl b/template/en/default/index.html.tmpl
index 4a28442551..08804d4146 100644
--- a/template/en/default/index.html.tmpl
+++ b/template/en/default/index.html.tmpl
@@ -78,7 +78,7 @@ function addSidebar() {
   </p>
   </form>
  </td>
- <td align="right"><img src="ant.jpg" width=329 height=220 border=2 alt="ant.jpg [8.5k]"></td>
+ <td align="right"><img src="ant.jpg" width="329" height="220" border="2" alt="ant.jpg [8.5k]"></td>
 </tr></table>
 
 <script type="text/javascript" language="JavaScript" src="localconfig.js"></script>
diff --git a/template/en/default/list/edit-multiple.html.tmpl b/template/en/default/list/edit-multiple.html.tmpl
index fb28845c51..eb8e759bed 100644
--- a/template/en/default/list/edit-multiple.html.tmpl
+++ b/template/en/default/list/edit-multiple.html.tmpl
@@ -28,11 +28,11 @@
           item.checked = value;
       }
   }
-  document.write(' <input type="button" value="Uncheck All" onclick="SetCheckboxes(false);">');
-  document.write(' <input type="button" value="Check All" onclick="SetCheckboxes(true);">');
+  document.write(' <input type="button" name="uncheck_all" value="Uncheck All" onclick="SetCheckboxes(false);">');
+  document.write(' <input type="button" name="check_all" value="Check All" onclick="SetCheckboxes(true);">');
 </script>
 
-<hr />
+<hr>
 
 <p><font size="-1">
   To change multiple bugs:
@@ -162,24 +162,24 @@
 
 <input type="hidden" name="multiupdate" value="Y">
 
-<label for="comment"><b>Additional Comments:</b></label><br />
-<textarea id="comment" name="comment" rows="5" cols="80" wrap="hard"></textarea><br />
+<label for="comment"><b>Additional Comments:</b></label><br>
+<textarea id="comment" name="comment" rows="5" cols="80" wrap="hard"></textarea><br>
 
 [% IF groups.size > 0 %]
 
-  <b>Groupset:</b><br />
+  <b>Groupset:</b><br>
   <table border="1">
     <tr>
-      <th>Don't<br />change<br />this group<br />restriction</td>
-      <th>Remove<br />bugs<br />from this<br />group</td>
-      <th>Add<br />bugs<br />to this<br />group</td>
+      <th>Don&apos;t<br>change<br>this group<br>restriction</td>
+      <th>Remove<br>bugs<br>from this<br>group</td>
+      <th>Add<br>bugs<br>to this<br>group</td>
       <th>Group Name:</td>
     </tr>
 
     [% FOREACH group = groups %]
     <tr>
       <td align="center">
-        <input type="radio" name="bit-[% group.bit %]" value="-1" checked>
+        <input type="radio" name="bit-[% group.bit %]" value="-1" checked="checked">
       </td>
       <td align="center">
         <input type="radio" name="bit-[% group.bit %]" value="0">
@@ -208,7 +208,7 @@
 
   [% IF foundinactive %]
     <font size="-1">(Note: Bugs may not be added to <strike>inactive 
-    groups</strike>, only removed.)</font><br />
+    groups</strike>, only removed.)</font><br>
   [% END %]
 
 [% END %]
@@ -216,33 +216,33 @@
 
 
 [% knum = 0 %]
-<input id="knob-none" type="radio" name="knob" value="none" CHECKED>
-<label for="knob-none">Do nothing else</label><br />
+<input id="knob-none" type="radio" name="knob" value="none" checked="checked">
+<label for="knob-none">Do nothing else</label><br>
 
 [% IF bugstatuses.size == 1 && bugstatuses.0 == unconfirmedstate %]
   [% knum = knum + 1 %]
   <input id="knob-confirm" type="radio" name="knob" value="confirm">
   <label for="knob-confirm">
     Confirm bugs (change status to <b>NEW</b>)
-  </label><br />
+  </label><br>
 [% END %]
 
 [% knum = knum + 1 %]
 <input id="knob-accept" type="radio" name="knob" value="accept">
 <label for="knob-accept">
   Accept bugs (change status to <b>ASSIGNED</b>)
-</label><br />
+</label><br>
 
 [%# If all the bugs being changed are open, allow the user to close them. %]
 [% IF !bugstatuses.containsany(closedstates) %]
   [% knum = knum + 1 %]
   <input id="knob-clearresolution" type="radio" name="knob" value="clearresolution">
-  <label for="knob-clearresolution">Clear the resolution</label><br />
+  <label for="knob-clearresolution">Clear the resolution</label><br>
 
   [% knum = knum + 1 %]
   <input id="knob-resolve" type="radio" name="knob" value="resolve">
   <label for="knob-resolve">
-    Resolve bugs, changing <A HREF="bug_status.html">resolution</A> to
+    Resolve bugs, changing <a href="bug_status.html">resolution</a> to
   </label>
   <select name="resolution" onchange="document.forms.changeform.knob[[% knum %]].checked=true">
     [% FOREACH resolution = resolutions %]
@@ -251,7 +251,7 @@
         [% resolution FILTER html %]
       </option>
     [% END %]
-  </select><br />
+  </select><br>
 
 [% END %]
 
@@ -259,30 +259,30 @@
 [% IF !bugstatuses.containsany(openstates) %]
   [% knum = knum + 1 %]
   <input id="knob-reopen" type="radio" name="knob" value="reopen">
-  <label for="knob-reopen">Reopen bugs</label><br />
+  <label for="knob-reopen">Reopen bugs</label><br>
 [% END %]
 
 [% IF bugstatuses.size == 1 %]
   [% IF bugstatuses.contains('RESOLVED') %]
     [% knum = knum + 1 %]
     <input id="knob-verify" type="radio" name="knob" value="verify">
-    <label for="knob-verify">Mark bugs as <b>VERIFIED</b></label><br />
+    <label for="knob-verify">Mark bugs as <b>VERIFIED</b></label><br>
   [% ELSIF bugstatuses.contains('VERIFIED') %]
     [% knum = knum + 1 %]
     <input id="knob-close" type="radio" name="knob" value="close">
-    <label for="knob-close">Mark bugs as <b>CLOSED</b></label><br />
+    <label for="knob-close">Mark bugs as <b>CLOSED</b></label><br>
   [% END %]
 [% END %]
 
 [% knum = knum + 1 %]
 <input id="knob-reassign" type="radio" name="knob" value="reassign">
 <label for="knob-reassign"><a href="bug_status.html#assigned_to">
-  Reassign</A> bugs to
+  Reassign</a> bugs to
 </label>
 <input name="assigned_to" 
        value="[% user.login FILTER html %]"
        onchange="document.forms.changeform.knob[[% knum %]].checked = true;"
-       size="32"><br />
+       size="32"><br>
 
 [% knum = knum + 1 %]
 <input id="knob-reassignbycomponent" 
@@ -291,7 +291,7 @@
        value="reassignbycomponent">
 <label for="knob-reassignbycomponent">
   Reassign bugs to owner of selected component
-</label><br />
+</label><br>
 
 <input type="submit" value="Commit">
 
@@ -306,7 +306,7 @@
 
 [% BLOCK selectmenu %]
   <select id="[% menuname %]" name="[% menuname %]">
-    <option value="[% dontchange FILTER html %]" selected>
+    <option value="[% dontchange FILTER html %]" selected="selected">
       [% dontchange FILTER html %]
     </option>
     [% FOREACH menuitem = menuitems %]
diff --git a/template/en/default/list/list-simple.html.tmpl b/template/en/default/list/list-simple.html.tmpl
index b86f697edd..25c8c82ec4 100644
--- a/template/en/default/list/list-simple.html.tmpl
+++ b/template/en/default/list/list-simple.html.tmpl
@@ -39,7 +39,7 @@
 
   <head>
     <title>[% title %]</title>
-    <link href="css/buglist.css" rel="stylesheet" type="text/css" />
+    <link href="css/buglist.css" rel="stylesheet" type="text/css">
   </head>
 
   <body>
diff --git a/template/en/default/list/list.html.tmpl b/template/en/default/list/list.html.tmpl
index 15d5fc0a05..f6a4df3895 100644
--- a/template/en/default/list/list.html.tmpl
+++ b/template/en/default/list/list.html.tmpl
@@ -38,7 +38,7 @@
 %]
 
 <div align="center">
-  <b>[% currenttime %]</b><br />
+  <b>[% currenttime %]</b><br>
 
   [% IF debug %]
     <p>[% query FILTER html %]</p>
@@ -57,7 +57,7 @@
   </h2>
 [% END %]
 
-<hr />
+<hr>
 
 
 [%############################################################################%]
@@ -105,7 +105,7 @@
 
 [% END %]
 
-<br />
+<br>
 
 
 [%############################################################################%]
@@ -118,7 +118,7 @@
   
   </form>
 
-  <hr />
+  <hr>
 
 [% END %]
 
diff --git a/template/en/default/list/quips.html.tmpl b/template/en/default/list/quips.html.tmpl
index 032f35fcfc..c874b14d79 100644
--- a/template/en/default/list/quips.html.tmpl
+++ b/template/en/default/list/quips.html.tmpl
@@ -46,10 +46,10 @@
 </p>
 
 <form method="post" action="quips.cgi">
-  <input type="hidden" name="action" value="add" />
-  <input size="80" name="quip" />
+  <input type="hidden" name="action" value="add">
+  <input size="80" name="quip">
   <p>
-    <input type="submit" value="Add This Quip" />
+    <input type="submit" value="Add This Quip">
   </p>
 </form>
 
diff --git a/template/en/default/reports/components.html.tmpl b/template/en/default/reports/components.html.tmpl
index 8bc549509d..7190883b28 100644
--- a/template/en/default/reports/components.html.tmpl
+++ b/template/en/default/reports/components.html.tmpl
@@ -55,7 +55,7 @@
       [% INCLUDE describe_comp %]
     [% END %]
     <tr>
-      <td colspan='[% numcols %]'>
+      <td colspan="[% numcols %]">
         <hr>
       </td>
     </tr>
@@ -75,7 +75,7 @@
     </td>
   </tr>
   <tr>
-    <td rowspan='2'>
+    <td rowspan="2">
       <a name="[% comp.name FILTER html %]">[% comp.name FILTER html %]</a>
     </td>
     <td>
diff --git a/template/en/default/reports/duplicates-table.html.tmpl b/template/en/default/reports/duplicates-table.html.tmpl
index f7b8ec1738..24eb4246e4 100644
--- a/template/en/default/reports/duplicates-table.html.tmpl
+++ b/template/en/default/reports/duplicates-table.html.tmpl
@@ -44,7 +44,7 @@
 [%# *** Column Headers *** %]
 
 <table border>
-  <tr BGCOLOR="#CCCCCC">
+  <tr bgcolor="#CCCCCC">
     [% FOREACH column = [ { name => "id", description => "Bug #" },
                           { name => "count", description => "Dupe<br>Count" },
                           { name => "delta", 
@@ -111,7 +111,7 @@
     <td>
       <center>
         [% "<strike>" IF bug.resolution != "" %]
-        <A HREF="show_bug.cgi?id=[% bug.id %]">[% bug.id %]</a>
+        <a href="show_bug.cgi?id=[% bug.id %]">[% bug.id %]</a>
         [% "</strike>" IF bug.resolution != "" %]
       </center>
     </td>
diff --git a/template/en/default/reports/duplicates.html.tmpl b/template/en/default/reports/duplicates.html.tmpl
index 203776cd3d..64ba5e120d 100644
--- a/template/en/default/reports/duplicates.html.tmpl
+++ b/template/en/default/reports/duplicates.html.tmpl
@@ -59,8 +59,8 @@
 <h3><a name="params">Change Parameters</a></h3>
 
 <form method="get" action="duplicates.cgi">
-  <input type="hidden" name="sortby" value="[% sortby %]" />
-  <input type="hidden" name="reverse" value="[% reverse %]" />
+  <input type="hidden" name="sortby" value="[% sortby %]">
+  <input type="hidden" name="reverse" value="[% reverse %]">
   <input type="hidden" name="bug_id" value="[% bug_ids_string %]">
   <table>
     <tr>
@@ -68,20 +68,20 @@
           work with:</td>
       <td>
         <input type="radio" name="sortvisible" id="entirelist" value="0" 
-          [% "checked" IF NOT sortvisible %] />         
+          [% "checked" IF NOT sortvisible %]>         
         <label for="entirelist">
           entire list
         </label>
-        <br />        
+        <br>        
         <input type="radio" name="sortvisible" id="visiblelist" value="1" 
-          [% "checked" IF sortvisible %] /> 
+          [% "checked" IF sortvisible %]> 
         <label for="visiblelist">
           currently visible list
         </label>
       </td>
       <td rowspan="4" valign="top">Restrict to products:</td>
       <td rowspan="4" valign="top">
-        <select name="product" size="5" multiple>
+        <select name="product" size="5" multiple="multiple">
           [% FOREACH p = products %]
             <option name="[% p %]"
             [% " selected" IF product == p %]>[% p %]</option>
@@ -93,14 +93,14 @@
     <tr>
       <td>Max rows:</td>
       <td>
-        <input size="4" name="maxrows" value="[% maxrows %]" />
+        <input size="4" name="maxrows" value="[% maxrows %]">
       </td>
     </tr>
 
     <tr>
       <td>Change column is change in the last:</td>
       <td>
-        <input size="4" name="changedsince" value="[% changedsince %]" /> days
+        <input size="4" name="changedsince" value="[% changedsince %]"> days
       </td>
     </tr>
 
@@ -112,13 +112,13 @@
       </td>
       <td>
         <input type="checkbox" name="openonly" id="openonly" value="1" 
-          [% "checked" IF openonly %] />      
+          [% "checked" IF openonly %]>      
       </td>
     </tr>
 
   </table>
 
-  <input type="submit" value="Change" /> 
+  <input type="submit" value="Change"> 
 </form>
 
 <form method="post" action="buglist.cgi">
@@ -128,7 +128,7 @@
   (Note: the order may not be the same.)
 </form>
 
-<hr />
+<hr>
 
 <b>
   <a name="explanation">What are "Most Frequently Reported Bugs"?</a>
diff --git a/template/en/default/search/form.html.tmpl b/template/en/default/search/form.html.tmpl
index 790feee640..cd81d7cac8 100644
--- a/template/en/default/search/form.html.tmpl
+++ b/template/en/default/search/form.html.tmpl
@@ -368,7 +368,7 @@ function selectProduct(f) {
           [%# Can't use the select block here because of onChange and the fact that
               'component' is a toolkit reserved word - we use 'component_' instead. %]
           <td align="left">
-            <select name="product" multiple size="5" onChange="selectProduct(this.form);">
+            <select name="product" multiple="multiple" size="5" onchange="selectProduct(this.form);">
             [% FOREACH p = product %]
               <option value="[% p FILTER html %]"
                 [% " selected" IF lsearch(default.product, p) != -1 %]>
@@ -378,7 +378,7 @@ function selectProduct(f) {
           </td>
 
           <td align="left">
-            <select name="component" multiple size="5">
+            <select name="component" multiple="multiple" size="5">
             [% FOREACH c = component_ %]
               <option value="[% c FILTER html %]"
                 [% " selected" IF lsearch(default.component, c) != -1 %]>
@@ -618,13 +618,13 @@ function selectProduct(f) {
 
 <dl>
   <dt>Only bugs changed in the last </dt>
-  <dd><input name=changedin size=3 value="[% default.changedin.0 FILTER html %]"> days</dd>
+  <dd><input name="changedin" size="3" value="[% default.changedin.0 FILTER html %]"> days</dd>
 </dl>
 
 <dl>
   <dt>Only bugs where any of the fields</dt>
   <dd>
-    <select name="chfield" multiple size="4">
+    <select name="chfield" multiple="multiple" size="4">
     [% FOREACH field = chfield %]
       <option value="[% field FILTER html %]"
         [% " selected" IF lsearch(default.chfield, field) != -1 %]>
@@ -659,7 +659,7 @@ function selectProduct(f) {
 
 [% BLOCK select %]
   <td align="left">
-    <select name="[% sel.name %]" multiple size="[% sel.size %]">
+    <select name="[% sel.name %]" multiple="multiple" size="[% sel.size %]">
     [% FOREACH name = ${sel.name} %]
       <option value="[% name FILTER html %]"
         [% " selected" IF lsearch(default.${sel.name}, name) != -1 %]>
diff --git a/template/en/default/search/knob.html.tmpl b/template/en/default/search/knob.html.tmpl
index fd3abcbb07..1553fe7da1 100644
--- a/template/en/default/search/knob.html.tmpl
+++ b/template/en/default/search/knob.html.tmpl
@@ -36,7 +36,7 @@
       <input type="hidden" name="cmdtype" value="doit">
     [% ELSE %]
       <br>
-      <input type="radio" name="cmdtype" value="doit" checked> Run this query
+      <input type="radio" name="cmdtype" value="doit" checked="checked"> Run this query
       <br>
 
       [% IF namedqueries.size > 0 %]