https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40320

--- Comment #8 from Jonathan Druart <[email protected]> ---
(In reply to Marcel de Rooy from comment #7)
> No blocker just a comment:
> +                [%~ IF ( patron.city ) ~%][% ", " | $raw %][% END ~%]
> This looks overcomplicated to me somehow.
> Note that the second tilde in the IF should do nothing. And what is the
> bonus of pushing a comma thru raw anyway?

IIRC We want to keep the space after the comma.

If you have [% IF %], [% END %] then prettier will remove the space.
Use TT [% ", " %] makes sure the space will stay.
The filter is mandatory or find-missing-filters.t will fail (and better raw
than html, we know there is nothing to escape).

You can play with the code and the tests if you think there is a better
solution.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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