Attachment 45092 Details for Bug 15231 – [SIGNED-OFF]Bug 15231 - Import patrons: Remove string splitting by html tags to avoid weird translations

[patch] [SIGNED-OFF]Bug 15231 - Import patrons: Remove string splitting by html tags to avoid weird translations

SIGNED-OFFBug-15231---Import-patrons-Remove-string.patch (text/plain), 5.45 KB, created by Héctor Eduardo Castro Avalos on 2015-11-23 03:04:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Héctor Eduardo Castro Avalos

Created: 2015-11-23 03:04:00 UTC

Size: 5.45 KB

patch

obsolete

>From b9b6a531aabeebb50c32a2c61e8460cbb899f7ac Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
>Date: Fri, 20 Nov 2015 22:58:40 +0100
>Subject: [PATCH] [SIGNED-OFF]Bug 15231 - Import patrons: Remove string
> splitting by html tags to avoid weird translations
>
>Some of the strings in Home > Tools > Import patrons contain html tags and therefore are
>splitted in the translation tool. This makes it nearly impossible to create correct translations
>from the partial strings.
>Additinally, the notes for Extended patron attributes never displayed (due to a bug).
>
>To test:
>- Apply patch
>- Enable Syspref 'ExtendedPAtronAttributes'
>- Go to Home > Tools > Import patrons and verify, that the Notes at the right of the
> screen make sense
>
>Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
>Split html tags removed, notes at right side it displayed correctly
>---
> .../prog/en/modules/tools/import_borrowers.tt | 36 ++++++++++++++--------
> 1 file changed, 23 insertions(+), 13 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
>index f8d9303..362d222 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
>@@ -96,8 +96,8 @@
> [% END %]
> [% ELSE %]
> <ul>
>-	<li>Select a file to import into the borrowers table</li>
>-	<li>If a cardnumber exists in the table, you can choose whether to ignore the new one or overwrite the old one.</li>
>+ <li>Select a file to import into the borrowers table.</li>
>+ <li>If a cardnumber exists in the table, you can choose whether to ignore the new one or overwrite the old one.</li>
> </ul>
> <form method="post" action="[% SCRIPT_NAME %]" enctype="multipart/form-data">
> <fieldset class="rows">
>@@ -193,19 +193,29 @@
> <div class="yui-u">
> <h2>Notes:</h2>
> <ul>
>-<li>The first line in the file must be a header row defining which columns you are supplying in the import file.</li>
>-<li>Download a starter CSV file with all the columns <a href="?sample=1">here</a>. Values are comma-separated.</li>
>-<li>OR choose which fields you want to supply from the following list:<ul><li>
>- [% FOREACH columnkey IN borrower_fields %]'[% columnkey.field %]', [% END %]
>-</li></ul></li>
>-[% IF ( ExtendedPatronAttributes ) %]
>-<li>If loading patron attributes, the 'patron_attributes' field should contain a comma-separated list of attribute types and values. The attribute type code and a colon should precede each value. For example: INSTID:12345,LANG:fr or STARTDATE:January 1 2010,TRACK:Day. If an input record has more than one attribute, the fields should either be entered as an unquoted string (previous examples), or with each field wrapped in separate double quotes and delimited by a comma: &quot;STARTDATE:January 1, 2010&quot;,&quot;TRACK:Day&quot;. The second syntax would be required if the data might have a comma in it, like a date string.
>+<li>Header: The first line in the file must be a header row defining which columns you are supplying in the import file.</li>
>+<li>Separator: Values are comma-separated.</li>
>+<li>Starter CSV: Koha provides a starter CSV with all the columns.
>+ <ul><li><a href="?sample=1">Download starter CSV</a></li></ul>
> </li>
>+<li>Field list: Alternatively, you can create your own CSV and choose which fields you want to supply from the following list:
>+ <ul><li>
>+ [% FOREACH columnkey IN borrower_fields %]'[% columnkey.field %]', [% END %]
>+ </li></ul>
>+</li>
>+[% IF ( Koha.Preference('ExtendedPatronAttributes') == 1 ) %]
>+<li>Extended patron attributes: If loading patron attributes, the 'patron_attributes' field should contain a comma-separated list of attribute types and values. The attribute type code and a colon should precede each value.
>+ <ul><li>Example 1: INSTID:12345,LANG:fr</li><li> Example 2: STARTDATE:January 1 2010,TRACK:Day</li></ul>
>+If an input record has more than one attribute, the fields should either be entered as an unquoted string (previous examples), or with each field wrapped in separate double quotes and delimited by a comma:
>+ <ul><li>Example 3: &quot;STARTDATE:January 1, 2010&quot;,&quot;TRACK:Day&quot;</li></ul>
>+The second syntax would be required if the data might have a comma in it, like a date string.</li>
> [% END %]
>-<li>The fields 'branchcode' and 'categorycode' are required and must match valid entries in your database.</li>
>-<li>'password' should be stored in plaintext, and will be converted to a Bcrypt hash (if your passwords are already encrypted, talk to your system administrator about options).</li>
>-<li>Date formats should match your system preference, and must be zero-padded, e.g. '01/02/2008'. Alternatively,
>-you can supply dates in ISO format (e.g., '2010-10-28').
>+<li>Required fields: The fields 'branchcode' and 'categorycode' are required and must match valid entries in your database.</li>
>+<li>Password: Values for the field 'password' should be stored in plaintext, and will be converted to a Bcrypt hash (if your passwords are already encrypted, talk to your system administrator about options).</li>
>+<li>Date formats: Date values should match your system preference, and must be zero-padded.
>+ <ul><li>Example: '01/02/2008'</li></ul>
>+Alternatively, you can supply dates in ISO format.
>+ <ul><li>Example: '2010-10-28'</li></ul>
> </li>
> </ul>
> 
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 15231: 45087 | 45092 | 45343 | 45346 | 45441 | 46037 | 46038 | 46039