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

--- Comment #9 from Marcel de Rooy <[email protected]> ---
(In reply to Jonathan Druart from comment #8)
> (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.

prettier was not yet on my radar. No play, just push

-- 
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