Bug#330257: awstats: wrong html tag prevents proper css decoration

[mike dentifrice]

Package: awstats
Severity: minor
Tags: patch


Awstats outputs wrong HTML tags (<td>) for the "summary" table headers. Other 
tables' headers are properly defined by <th> tags, thus allowing redinition of 
the hard-coded color scheme and further styling in a coherent fashion.

Following is a patch against awstats.pl, fixing those typos.

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (700, 'testing'), (650, 'unstable'), (600, 'stable')
Architecture: powerpc (ppc)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.12.5
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=ISO-8859-15)

*** awstats-th_fix.patch
--- awstats.pl.orig     2005-09-26 21:27:23.000000000 +0200
+++ awstats.pl  2005-09-26 21:33:17.000000000 +0200
@@ -8542,12 +8542,12 @@
                        
                        # Show main indicators title row
                        print "<tr>";
-                       if ($LogType eq 'W' || $LogType eq 'S') { print "<td 
bgcolor=\"#$color_TableBGTitle\">&nbsp;</td>"; }
-                       if ($ShowSummary =~ /U/i) { print "<td width=\"$w%\" 
bgcolor=\"#$color_u\"".Tooltip(2).">$Message[11]</td>"; } else { print "<td 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</td>"; }
-                       if ($ShowSummary =~ /V/i) { print "<td width=\"$w%\" 
bgcolor=\"#$color_v\"".Tooltip(1).">$Message[10]</td>"; } else { print "<td 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</td>"; }
-                       if ($ShowSummary =~ /P/i) { print "<td width=\"$w%\" 
bgcolor=\"#$color_p\"".Tooltip(3).">$Message[56]</td>"; } else { print "<td 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</td>"; }
-                       if ($ShowSummary =~ /H/i) { print "<td width=\"$w%\" 
bgcolor=\"#$color_h\"".Tooltip(4).">$Message[57]</td>"; } else { print "<td 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</td>"; }
-                       if ($ShowSummary =~ /B/i) { print "<td width=\"$w%\" 
bgcolor=\"#$color_k\"".Tooltip(5).">$Message[75]</td>"; } else { print "<td 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</td>"; }
+                       if ($LogType eq 'W' || $LogType eq 'S') { print "<th 
bgcolor=\"#$color_TableBGTitle\">&nbsp;</th>"; }
+                       if ($ShowSummary =~ /U/i) { print "<th width=\"$w%\" 
bgcolor=\"#$color_u\"".Tooltip(2).">$Message[11]</th>"; } else { print "<th 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</th>"; }
+                       if ($ShowSummary =~ /V/i) { print "<th width=\"$w%\" 
bgcolor=\"#$color_v\"".Tooltip(1).">$Message[10]</th>"; } else { print "<th 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</th>"; }
+                       if ($ShowSummary =~ /P/i) { print "<th width=\"$w%\" 
bgcolor=\"#$color_p\"".Tooltip(3).">$Message[56]</th>"; } else { print "<th 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</th>"; }
+                       if ($ShowSummary =~ /H/i) { print "<th width=\"$w%\" 
bgcolor=\"#$color_h\"".Tooltip(4).">$Message[57]</th>"; } else { print "<th 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</th>"; }
+                       if ($ShowSummary =~ /B/i) { print "<th width=\"$w%\" 
bgcolor=\"#$color_k\"".Tooltip(5).">$Message[75]</th>"; } else { print "<th 
bgcolor=\"#$color_TableBGTitle\" width=\"20%\">&nbsp;</th>"; }
                        print "</tr>\n";
                        # Show main indicators values for viewed traffic
                        print "<tr>";


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]