http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11508

--- Comment #2 from Owen Leonard <[email protected]> ---
The change to the template includes improperly nested [% IF %]s inside HTML
tags:

<option value= "0" [%- IF ( loo.ohidden.default ==  0 )
-%]selected="selected"[%- END -%]>Show all</option>

This should be written:

[%- IF ( loo.ohidden.default ==  0 ) -%]
    <option value= "0" selected="selected">Show all</option>
[%- ELSE -%
    <option value= "0">Show all</option>
[%- END -%]

This is because of problems the translation script has with parsing template
directives nested inside tags:

http://wiki.koha-community.org/wiki/Coding_Guidelines#HTML1:_Template_Toolkit_markup_inside_HTML

This should have been caught by the QA tool, but it looks like the "-" in "[%-
IF" isn't an expected pattern. It would be nice if the QA test tool were
updated to be aware of that construction.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to