Bug 12299 - SQL setup (MARC21), translation of relators codes - Italian
Summary: SQL setup (MARC21), translation of relators codes - Italian
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: I18N/L10N (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-21 09:03 UTC by Zeno Tajoli
Modified: 2015-06-04 23:30 UTC (History)
4 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
translation of marc21_relatorterms.sql (52.31 KB, patch)
2014-05-21 09:16 UTC, Zeno Tajoli
Details | Diff | Splinter Review
translation of marc21_relatorterms.sql (51.25 KB, patch)
2014-06-20 10:00 UTC, Zeno Tajoli
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 12299: Translate marc21_relatorterms.sql into Italian (51.51 KB, patch)
2014-06-20 18:43 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 12299: Translate marc21_relatorterms.sql into Italian (51.42 KB, patch)
2014-06-20 18:57 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
[PASSED QA] Bug 12299: Translate marc21_relatorterms.sql into Italian (51.54 KB, patch)
2014-06-21 15:44 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Zeno Tajoli 2014-05-21 09:03:24 UTC
This the transaltion of file marc21_relatorterms.sql for Italian setup.
It is the transaltion of relator codes of MARC21. It use the authorised value 'RELTERMS'
Comment 1 Zeno Tajoli 2014-05-21 09:16:39 UTC Comment hidden (obsolete)
Comment 2 Zeno Tajoli 2014-05-21 09:17:20 UTC
To test:
- Apply this patch.
- Reset your DB (DROP DATABASE, CREATE DATABASE -- whatever method you
  prefer)
- Install it-IT templates on command line
- Run the webinstaller using it-IT
- Follow the steps and choose all under 'opzionali'
=> SUCCESS: no errors
=> Sign this
=> Extra: see the authorised value 'RELTERMS' and check if you like the
   translation
Comment 3 Bernardo Gonzalez Kriegel 2014-06-18 14:14:36 UTC
Hi Zeno,
there is a problem, your insert lines have an '=' where they must have a ','
e.g.
...VALUES ('RELTERMS','abr' = 'Riduttore');

Change that to
...VALUES ('RELTERMS','abr', 'Riduttore');

Doing that change, file loads ok.
Comment 4 Zeno Tajoli 2014-06-20 10:00:58 UTC Comment hidden (obsolete)
Comment 5 Zeno Tajoli 2014-06-20 10:02:03 UTC
To test:
- Apply this patch.
- Reset your DB (DROP DATABASE, CREATE DATABASE -- whatever method you
  prefer)
- Install it-IT templates on command line
- Run the webinstaller using it-IT
- Follow the steps and choose all under 'opzionali'
=> SUCCESS: no errors
=> Sign this
=> Extra: see the authorised value 'RELTERMS' and check if you like the
   translation
Comment 6 Bernardo Gonzalez Kriegel 2014-06-20 18:43:41 UTC Comment hidden (obsolete)
Comment 7 Bernardo Gonzalez Kriegel 2014-06-20 18:57:34 UTC Comment hidden (obsolete)
Comment 8 Katrin Fischer 2014-06-21 15:44:31 UTC
Created attachment 29131 [details] [review]
[PASSED QA] Bug 12299: Translate marc21_relatorterms.sql into Italian

To test:
- Apply this patch.
- Reset your DB (DROP DATABASE, CREATE DATABASE -- whatever method you
  prefer)
- Install it-IT templates on command line
- Run the webinstaller using it-IT
- Follow the steps and choose all under 'opzionali'
=> SUCCESS: no errors
=> Sign this
=> Extra: see the authorised value 'RELTERMS' and check if you like the
   translation

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Tested loading file.
Two errors fixed:
a) duplicate line ... ('RELTERMS','con', 'Conservatore')
b) missing semicolon ...  ('RELTERMS','voc','Cantante')

With that file loads Ok
No koha-qa errors

Ooops, forget the fix. Resubmited

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Adds an optional SQL file to the Italian installer - works as expected.
Comment 9 Tomás Cohen Arazi 2014-06-21 22:04:35 UTC
Patch pushed to master.

Thanks Zeno!

Note: I removed BOM from file.
Comment 10 Galen Charlton 2014-06-24 15:44:27 UTC
Pushed to 3.16.x for inclusion in 3.16.1.