html/index.tmpl: Loops in TT and me, the neverending story
authorFrank Lichtenheld <frank@lichtenheld.de>
Mon, 19 Nov 2007 17:34:01 +0000 (18:34 +0100)
committerFrank Lichtenheld <frank@lichtenheld.de>
Mon, 19 Nov 2007 18:56:42 +0000 (19:56 +0100)
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).

templates/html/index.tmpl

index 63d5f49b48807bb793b829f7e81b0740ec0a5d58..cc209838c189b60d034bb2ac1a78acf741241f29 100644 (file)
        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 %]