Tomas Cohen Arazi
b46418478e
This bug looked cool and safe, but tests highlighted that the
(introduced) newlines were translated into spaces, which is not correct
in the CSV format (i.e. q{"Column 1" , "Column 2"} is not really
correct).
Also, the double quotes were forcibly introduced (semi-correct) but the
tests weren't adjusted.
We should really stop using templates for generating CSV, and use a
library for the task instead of manually crafting them. But that's for
another bug report.
This patch:
* Removes extra spaces in TT-generated CSV headers
* Adjusts the tests to the new format introduced by this report
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
8 lines
887 B
Text
8 lines
887 B
Text
[% FILTER collapse %]
|
|
[%- USE Koha -%]
|
|
[%- PROCESS 'i18n.inc' -%]
|
|
[%- SET delimiter = Koha.CSVDelimiter() -%]
|
|
[%- BLOCK -%]
|
|
"[% t("Contract name") | html %]"[% delimiter | html %]"[% t("Order number") | html %]"[% delimiter | html %]"[% t("Entry date") | html %]"[% delimiter | html %]"[% t("ISBN") | html %]"[% delimiter | html %]"[% t("Author") | html %]"[% delimiter | html %]"[% t("Title") | html %]"[% delimiter | html %]"[% t("Publication year") | html %]"[% delimiter | html %]"[% t("Publisher") | html %]"[% delimiter | html %]"[% t("Collection title") | html %]"[% delimiter | html %]"[% t("Note for vendor") | html %]"[% delimiter | html %]"[% t("Quantity") | html %]"[% delimiter | html %]"[% t("RRP") | html %]"[% delimiter | html %]"[% t("Delivery place") | html %]"[% delimiter | html %]"[% t("Billing place") | html %]"
|
|
[%- END -%]
|
|
[%- END -%]
|