From: Frank Lichtenheld <frank@lichtenheld.de>
Date: Mon, 19 Nov 2007 17:34:01 +0000 (+0100)
Subject: html/index.tmpl: Loops in TT and me, the neverending story
X-Git-Url: https://git.deb.at/?p=deb%2Fpackages.git;a=commitdiff_plain;h=b9f0c69092ef533663cd99fbd98459dfc7790313

html/index.tmpl: Loops in TT and me, the neverending story

Using implictely declared variables when looping over an array of
hashes remains a good way to shoot oneself in the foot. Fixes
non-English index pages if not all of the packages have translated
short descs (which is always the case).
---

diff --git a/templates/html/index.tmpl b/templates/html/index.tmpl
index 63d5f49..cc20983 100644
--- a/templates/html/index.tmpl
+++ b/templates/html/index.tmpl
@@ -25,30 +25,30 @@
 	navigation = nav_arr
 %]
 
-[% FOREACH packages %]
+[% FOREACH p IN packages %]
   [% '<dl>' IF loop.first %]
 
   [% BLOCK marker %] [<strong class="pmarker">[% text %]</strong>] [%- END %]
   [% BLOCK markers %]
-    [% PROCESS marker text=section IF section != main_section %]
-    [% PROCESS marker text=archive IF archive != main_archive %]
+    [% PROCESS marker text=p.section IF p.section != main_section %]
+    [% PROCESS marker text=p.archive IF p.archive != main_archive %]
   [% END %]
-  [%- IF trans_desc.$lang;
-	sdesc = trans_desc.$lang;
+  [%- IF p.trans_desc.$lang;
+	sdesc = p.trans_desc.$lang;
       ELSE;
-	sdesc = desc;
+	sdesc = p.desc;
       END -%]
-  [% IF providers.size %]
-    <dt><a href="[% name %]" id="[% name %]">[% name %]</a></dt>
+  [% IF p.providers.size %]
+    <dt><a href="[% p.name %]" id="[% p.name %]">[% p.name %]</a></dt>
        <dd>[% g('virtual package provided by') %]
-	[% FOREACH provider IN providers %]<a href="../[% provider | uri %]">[% provider %]</a>
+	[% FOREACH provider IN p.providers %]<a href="../[% provider | uri %]">[% provider %]</a>
 	[%- ', ' UNLESS loop.last %][% END %]</dd>
-    [% IF versions.length %]
-      <dt><a href="[% name %]">[% name %]</a> ([% versions %])[% PROCESS markers %]</dt>
+    [% IF p.versions.length %]
+      <dt><a href="[% p.name %]">[% p.name %]</a> ([% p.versions %])[% PROCESS markers %]</dt>
          <dd>[% sdesc | html %]</dd>
     [% END %]
   [% ELSE %]
-    <dt><a href="[% name %]" id="[% name %]">[% name %]</a> ([% versions %])[% PROCESS markers %]</dt>
+    <dt><a href="[% p.name %]" id="[% p.name %]">[% p.name %]</a> ([% p.versions %])[% PROCESS markers %]</dt>
        <dd>[% sdesc | html %]</dd>
   [% END %]