Bug 32567

Summary: Update plugin unimarc_field_110.pl 'Script of title' and 'Transliteration code'
Product: Koha Reporter: Thibaud Guillot <thibaud.guillot>
Component: CatalogingAssignee: Thibaud Guillot <thibaud.guillot>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: trivial    
Priority: P5 - low CC: lucas, m.de.rooy, pierre.genty
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.03
Attachments: Bug 32567: Update unimarc_field_110.pl with some values
Bug 32567: Update unimarc_field_110.pl with some values
Bug 32567: Update unimarc_field_110.pl with some values
Bug 32567: (QA follow-up) Fix some labelling inconsistencies

Description Thibaud Guillot 2023-01-04 16:09:03 UTC
Update value builder for unimarc_field_110 with some values
Comment 1 Thibaud Guillot 2023-01-04 16:13:28 UTC
Created attachment 145031 [details] [review]
Bug 32567: Update unimarc_field_110.pl with some values

'Script of title' values =>
e - b/c/d/e/f/g/h
i - d/e
j - f/g
l - f
n - b/c
o - a
p - a

'Translitteral code' values =>
'g' and 'h'
Comment 2 Thibaud Guillot 2023-01-04 16:16:07 UTC
I hope there is no translation error, if someone tells me I will correct it. I took the opportunity to reorder the different fields in alphabetical order, it is easier to find your way
Comment 3 Owen Leonard 2023-01-05 16:28:20 UTC
Created attachment 145065 [details] [review]
Bug 32567: Update unimarc_field_110.pl with some values

'Script of title' values =>
e - b/c/d/e/f/g/h
i - d/e
j - f/g
l - f
n - b/c
o - a
p - a

'Translitteral code' values =>
'g' and 'h'

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 4 Katrin Fischer 2023-02-05 13:07:38 UTC
Created attachment 146209 [details] [review]
Bug 32567: Update unimarc_field_110.pl with some values

'Script of title' values =>
e - b/c/d/e/f/g/h
i - d/e
j - f/g
l - f
n - b/c
o - a
p - a

'Translitteral code' values =>
'g' and 'h'

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2023-02-05 13:07:42 UTC
Created attachment 146210 [details] [review]
Bug 32567: (QA follow-up) Fix some labelling inconsistencies

This makes sure that the descriptions in the pull-downs always
match for selected/unselected.
This is not only for consistency, but will also ease translations
as simple things like an additional space will create a new
string to translate.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Thibaud Guillot 2023-02-06 08:26:51 UTC
(In reply to Katrin Fischer from comment #5)
> Created attachment 146210 [details] [review] [review]
> Bug 32567: (QA follow-up) Fix some labelling inconsistencies
> 
> This makes sure that the descriptions in the pull-downs always
> match for selected/unselected.
> This is not only for consistency, but will also ease translations
> as simple things like an additional space will create a new
> string to translate.
> 
> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Thanks Katrin
Comment 7 Tomás Cohen Arazi 2023-02-07 18:29:22 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 8 Jacob O'Mara 2023-02-08 16:53:13 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.
Comment 9 Lucas Gass 2023-02-15 21:39:11 UTC
Doesn't apply cleanly to 22.05.x, no backport