[Koha-devel] [bug 6458]: need some help for a parsing problem in 'misc/translator/TTParser.pm'

Frère Sébastien Marie sebastien.marie at latrappe.fr
Tue Jul 19 12:02:48 CEST 2011


On Tue, Jul 19, 2011 at 11:38:28AM +0200, Paul Poulain wrote:
> Not a direct answer to your question, but we agreed, a long time ago,
> not to have template loops/if inside a html tag.
> So, the template should not be
> 
> <tr[% IF ( overdue ) %] class="overdue"[% END %]>
> but
> [% IF ( overdue ) %] 
>   <tr class="overdue" >
> [% ELSE %]
>   <tr>
> [% END %]
> 
> that does not break the translator and is *much* easy to read.
> 
> ah, and I just fixed http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6412 that is exactly related to this problem ;-)

but should we open bug report for each "miss-used" templates ? (warning, simple grep report 56 occurences)
and not all generate a problem (in my case, only when contains also a translated string: an invalid template is produced)

-- 
Frère Sébastien Marie
Abbaye Notre Dame de La Trappe
61380 Soligny-la-Trappe
Tél: 02.33.84.17.00
Fax: 02.33.34.98.57
Web: http://www.latrappe.fr/


More information about the Koha-devel mailing list