Backing out patch for bug 251596: found a problem with it after I committed. See the bug.

77c85a80 · justdave%bugzilla.org · e63b2d6e · 77c85a80 · 77c85a80
Commit 77c85a80 authored Oct 25, 2004 by justdave%bugzilla.org
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 11 deletions

show.html.tmpl template/en/default/bug/show.html.tmpl +6 -5

header.html.tmpl template/en/default/global/header.html.tmpl +2 -6

No files found.
--- a/template/en/default/bug/show.html.tmpl
+++ b/template/en/default/bug/show.html.tmpl
@@ -28,16 +28,17 @@

 [% filtered_desc = bug.short_desc FILTER html %]
 [% filtered_timestamp = bug.delta_ts FILTER time %]
+[% bodyattrs = BLOCK %]class='bz_bug bz_status_[% bug.bug_status 
+  FILTER css_class_quote %] bz_component_[% bug.component 
+  FILTER css_class_quote %] bz_bug_[% bug.bug_id 
+  FILTER css_class_quote %]' 
+[% END %]
 [% PROCESS global/header.html.tmpl
  title = "$terms.Bug $bug.bug_id - $bug.short_desc"
  h1 = "$terms.Bugzilla $terms.Bug $bug.bug_id"
  h2 = filtered_desc
  h3 = "Last modified: $filtered_timestamp"
-  bodyclasses = ['bz_bug',
-                 "bz_status_$bug.bug_status",
-                 "bz_component_$bug.component",
-                 "bz_bug_$bug.bug_id"
-                ]
+  bodyattrs = bodyattrs
 %]

 [% PROCESS bug/navigate.html.tmpl %]

--- a/template/en/default/global/header.html.tmpl
+++ b/template/en/default/global/header.html.tmpl
@@ -28,7 +28,6 @@
  # h3: string. Right-aligned subheader.
  # bgcolor: string. the page's background color ("#rrggbb").
  # bodyattrs: any extra attributes for the <body> tag
-  # bodyclasses: array of extra CSS classes for the <body>
  # onload: string. JavaScript code to run when the page finishes loading.
  # javascript: string. Javascript to go in the header.
  # javascript_urls: list. List of URLs to Javascript.
@@ -107,11 +106,8 @@
  #%]

  <body bgcolor="[% bgcolor %]" onload="[% onload %]"
-        class="[% Param('urlbase').replace('^https?://','').replace('/$','').replace('[-~@:/.]+','-') %]
-               [% FOREACH class = bodyclasses %]
-                 [% ' ' %][% class FILTER css_class_quote %]
-               [% END %]"
-        [% bodyattrs %]>
+   class="[% Param('urlbase').replace('^https?://','').replace('/$','').replace('[-~@:/.]+','-') %]" 
+   [%+ bodyattrs %]>

 [%# Migration note: the following file corresponds to the old Param
  # 'bannerhtml'