From: lpsolit%gmail.com <>
Date: Thu, 28 Feb 2008 00:05:23 +0000 (+0000)
Subject: Bug 417146: Sanitycheck recurses into global/message.txt.tmpl - Patch by Frédéric... 
X-Git-Tag: bugzilla-3.1.3~298
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6e191c3b6af471d7b70027b510673d3190413db4;p=thirdparty%2Fbugzilla.git

Bug 417146: Sanitycheck recurses into global/message.txt.tmpl - Patch by Frédéric Buclin <LpSolit@gmail.com> r=justdave a=LpSolit
---

diff --git a/template/en/default/admin/sanitycheck/messages.html.tmpl b/template/en/default/admin/sanitycheck/messages.html.tmpl
index 0e44b21254..2ec043c4c3 100644
--- a/template/en/default/admin/sanitycheck/messages.html.tmpl
+++ b/template/en/default/admin/sanitycheck/messages.html.tmpl
@@ -281,7 +281,12 @@
 
 [% BLOCK bug_list %]
   [% FOREACH bug_id = badbugs %]
-    [%+ bug_id FILTER bug_link(bug_id) FILTER none %][% ", " IF !loop.last %]
+    [%# Do not use FILTER bug_link() here, because bug_link() calls get_text()
+     # which itself calls this template again, generating a recursion error.
+     # I doubt having a tooltip with the bug status and summary is so
+     # important here anyway, as you can click the "(as buglist)" link. %]
+    <a href="show_bug.cgi?id=[% bug_id FILTER url_quote %]">[% bug_id FILTER html %]</a>
+    [% ", " IF !loop.last %]
   [% END %]
   (<a href="buglist.cgi?bug_id=[% badbugs.join(",") FILTER url_quote %]">as buglist</a>).
 [% END %]