Attachment 157923 Details for Bug 35043 – Bug 35043: Do not have \n or \t appear in PO files

[patch] Bug 35043: Do not have \n or \t appear in PO files

Bug-35043-Do-not-have-n-or-t-appear-in-PO-files.patch (text/plain), 1.84 KB, created by Marcel de Rooy on 2023-10-26 12:41:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-10-26 12:41:47 UTC

Size: 1.84 KB

patch

obsolete

>From b07984fca44c2328e0b734858bd100bb9d110e7a Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 12 Oct 2023 12:35:12 +0200
>Subject: [PATCH] Bug 35043: Do not have \n or \t appear in PO files
>Content-Type: text/plain; charset=utf-8
>
>This is a trick, so that we won't have to deal with \t in PO files.
>\n is not a problem, but it seems consistent to use the same method for
>both.
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> .../prog/en/modules/tools/csv-profiles.tt              | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/csv-profiles.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/csv-profiles.tt
>index b14ca1ec8e..4dcd7b7de2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/csv-profiles.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/csv-profiles.tt
>@@ -111,15 +111,17 @@
>     [% ELSE %]
>         <option value=" ">Space ( )</option>
>     [% END %]
>+    [% tab_char = '\t' %]
>     [% IF selected_separator == '\t' %]
>-        <option value="\t" selected="selected">Tabulation (\t)</option>
>+        <option value="\t" selected="selected">Tabulation ([% tab_char | html %])</option>
>     [% ELSE %]
>-        <option value="\t">Tabulation (\t)</option>
>+        <option value="\t">Tabulation ([% tab_char | html %])</option>
>     [% END %]
>+    [% new_line_char = '\n' %]
>     [% IF selected_separator == '\n' %]
>-        <option value="\n" selected="selected">New line (\n)</option>
>+        <option value="\n" selected="selected">New line ([% new_line_char | html %])</option>
>     [% ELSE %]
>-        <option value="\n">New line (\n)</option>
>+        <option value="\n">New line ([% new_line_char | html %])</option>
>     [% END %]
> [% END %]
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35043: 156937 | 156938 | 156950 | 156951 | 157922 | 157923