https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #2 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to Jonathan Druart from comment #1)

Bernardo, could we imagine having the sysprefs' values translatable?

It's in my TODO list, but to translate explanations column. In this case it's the values that must be translated. Proposal: * Split sysprefs.sql into localizable and not localizable parts eg. sysprefs localsysprefs, our choice * Put them in /en/ dir * rewrite them as yaml, on syspref only translate explanation, on localsysprefs explanation and value * add strings from sysprefs in xx-YY-installer.po * add strings from localsysprefs in it's own po file. This file should contain useful comments so that the translators know what values are valid. What do you think? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #4 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Katrin Fischer from comment #0)

In the de-DE installer a sysprefs.sql file is used to make some changes to the system preferences for a smoother installation experience. The file can be found here:

http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=installer/data/ mysql/de-DE/mandatory/system_preferences.sql; h=2692f02b02da9100630a92c4f89fe7caba8ac786; hb=7ce4122131a6c3d7f0fe7c953dc95fd5d831bf7f

It's a mix of: - Translating prefs content (HTML prefs, they might go away to news sometime) - Localization settings (date format, currency format, time format) - Translating pref options (borrower titles and relationships)

It would be great if there was a way to keep this kind of modifications when switching to the new yaml based installer files.

We must split this bug report, there are 2 different requests: 1. Translate the default value of the sysprefs 2. Set predefined settings depending on the region/country/etc Then it seems we need to define some options, but we need to list exhaustively which ones are valid. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #6 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 103071 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103071&action=edit Bug 24973: rewrite subset of system preferences as YAML This patch removes a subset of system preferences from sysprefs.sql and rewrite them as YAML to SET their value for different languages using the translation process. The selected subset is an arbitrary, at this moment is similar to the one present in de-DE sysprefs file. Other languages update a different subset of sysprefs. I think that what the final subset will be falls outside the scope of this report. It can be enlarged or shortened at will. Test plan in following patch. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #8 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 103073 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103073&action=edit Bug 24973: add locale-sysprefs.yml to localized dirs This patch put a copy of en/locale-sysprefs.yml on each localized installation directory. Each file complements general sysprefs.sql file to enable a normal install. To test: 1) Apply the patch 2) Do a clean install for any of current localized dirs: de-DE,es-ES,fr-CA,fr-FR,it-IT,nb-NO,pl-PL,ru-RU and uk-UA 3) Check systempreferecences table content No changes expected. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #10 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 103075 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103075&action=edit Bug 24973: Add *-locale-prefs.po translation files This patch adds the new translation file for all current languages. To test: 1) Check update for a language (with no install dir) cd misc/translator ./translate update pt-BR 2) Check install ./translate install pt-BR verify creation of install dir ( installer/data/mysql/pt-BR) and locale_prefs.yml file 3) Do a clean install for your selected language, no problems expected -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #11 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Hi, I've uploaded my proposal for this problem. I create a new translation file, needed because it must show a message/advice to the translator. Then I need a new way to build the translation file, duplicating some of the code, can't found a better way for the moment, and release time is approaching, hope this pass. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #13 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to Martin Renvoize from comment #12)

Struggling to wrap my head around this one at the moment.. do we basically end up with still having to maintain a set of distinct sysprefs.yaml files within the koha repo?

Hi Martin, I'm not entirely happy with this approach too, and I'm open to suggestions. As of now we can only do inserts using yaml, perhaps we can leave syspres.sql as is and then add a yaml file to 'update' some selected subset of sysprefs, using this last one to translate.

If that's the case.. I wonder if we could instead split localisation into a distinct step in the installer.. so prefs are translated 'as is'.. but if you want to add/change defaults those changes go into their own distinct area and files.. something akin to the atomic updates and we run those after the rest of install as a localisation step.

I like the idea :) It's simpler, I'll try to write an alternative patch. Thanks -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #14 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Thanks for all your work here Bernardo. I never realise quite how involved translation and localisation gets. Your ongoing work here is greatly appreciated both by myself and the wider community I'm sure. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #16 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- My turn to thank you "All" for your work :) (In reply to Katrin Fischer from comment #15)

Just a thought: Why not keep it simple for now? We could keep the SQL files for any additional changes needed after the default installer files are loaded.

If I correctly interpret what you say is to leave all or part of the directory structure and at least one sql file with customizations, then the translation process populates those directories and then installation applies them. Is that correct? Something like this can be used for system preferences and frameworks, since they are loaded before the other files. Or if we agree in a hardcoded name then we could guarantee that it loads last.

At the moment it's very easy to customize the installer, as you just need to add your own directory (optional) and a text and sql file. If we could keep that ability at the same time, it would be a nice benefit. I have to admit, we run some changes on the frameworks and other standard settings.

It's easy, if you know how to do it :) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #18 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to Jonathan Druart from comment #17)

We could have a "localization" directory with common files (from Koha codebase) to have some localization stuffs set by default. localization/es-ES de-DE, etc. Then an empty localization/custom directory that we would look up to find .sql files.

You mean something like: 1) Before "./translate install xx-YY" en/ mandatory/ marcflavour/ optional/ localization/ de-DE/ fr-CA/ xx-YY/ 2) After "translate install xx-YY", new dir with translations xx-YY/ mandatory/ marcflavour/ optional/ and we load first from 'xx-YY/' and last from 'localization/xx-YY/', with xx-YY lang maintainer responsible for xx-YY/ content (mostly updates) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103071|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103073|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103075|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_20_11_target --- Comment #21 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Bernardo Gonzalez Kriegel from comment #18)

(In reply to Jonathan Druart from comment #17)

...

We could have a "localization" directory with common files (from Koha codebase) to have some localization stuffs set by default. localization/es-ES de-DE, etc. Then an empty localization/custom directory that we would look up to find .sql files.

You mean something like: [...]

Yes! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #22 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Switching to NSO to get feedback on the POC. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #104971|0 |1 is obsolete| | --- Comment #23 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 105032 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105032&action=edit Bug 24973: Load custom localization file It adds a new localization directory that could host a custom SQL file for each language, xx-YY/custom.sql This will be the last file to be loaded at install time, it can be used to set any difference from default values. It can't be deselected at install time. Moved corresponding files for de-DE, it-IT, nb-NO and completed es-ES To test: 1) Apply the patch 2) Install es-ES/de-DE/it-IT/nb-NO translation (any lang) ( cd misc/translation; ./translate install es-ES ) 3) Do a clean install using es-ES 4) After all files have been loaded check: * A new section labeled "Localization data added" with one file, custom.sql * Inspect the value of FrameworksLoaded syspref, last entry must be custom.sql 5) Remove/rename the file or localization dir and repeat 3/4, install must proceed normally -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #25 from Magnus Enger <magnus@libriotech.no> --- (In reply to Bernardo Gonzalez Kriegel from comment #24)

nb-NO uses NORMARC, this case needs more thinking

NORMARC is actually dead (Bug 18984), no new sites should be created with NORMARC as the MARC flavour. New sites should just use the default English MARC 21 frameworks. (Existing sites will continue to use NORMARC a bit longer, but hopefully we will convert to MARC 21 some time before the end of the year.) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #26 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- (In reply to Bernardo Gonzalez Kriegel from comment #24)

(In reply to Jonathan Druart from comment #22)

...

Switching to NSO to get feedback on the POC.

Uploaded revised patch, now include files for de-DE, it-IT and nb-NO, taken from their language dirs.

With something like this de-DE is covered and main dir could be removed. it-IT needs UNIMARC, more work to do nb-NO uses NORMARC, this case needs more thinking

But any future language can be added there to have the few customizations deemed essential by its maintainer.

If this approach is accepted we need to change the Bug description.

This looks great to me and seems to achieve what's desired.. but I'm not the target audience so I'm not comfortable doing a QA step.. I suppose doing the SO would politely push it into Katrins pile. One note though.. I don't see the additional languages you suggest you added in the above patch? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #105032|0 |1 is obsolete| | --- Comment #28 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 108130 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108130&action=edit Bug 24973: Load custom localization file It adds a new localization directory that could host a custom SQL file for each language, xx-YY/custom.sql This will be the last file to be loaded at install time, it can be used to set any difference from default values. It can't be deselected at install time. Moved corresponding files for de-DE, it-IT, nb-NO and completed es-ES To test: 1) Apply the patch 2) Install es-ES/de-DE/it-IT/nb-NO translation (any lang) ( cd misc/translation; ./translate install es-ES ) 3) Do a clean install using es-ES 4) After all files have been loaded check: * A new section labeled "Localization data added" with one file, custom.sql * Inspect the value of FrameworksLoaded syspref, last entry must be custom.sql 5) Remove/rename the file or localization dir and repeat 3/4, install must proceed normally -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #30 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to Bernardo Gonzalez Kriegel from comment #29)

(In reply to Martin Renvoize from comment #27)

...

Bernardo.. do you still have the de-DE, it-IT parts of the patch somewhere?

If so, I can signoff and we can move into QA

Hi Martin, uploaded patch with all changes this time. Thanks!

It's the same patch, sorry. But if you look at the details, you will see that some of the files are simply renamed rename installer/data/mysql/{de-DE/mandatory/system_preferences.sql => localization/de-DE/custom.sql} (98%) rename installer/data/mysql/{de-DE/mandatory/system_preferences.txt => localization/de-DE/custom.txt} (100%) rename installer/data/mysql/{it-IT/necessari/system_preferences.sql => localization/it-IT/custom.sql} (99%) rename installer/data/mysql/{nb-NO/1-Obligatorisk/system_preferences.sql => localization/nb-NO/custom.sql} (100%) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #108130|0 |1 is obsolete| | --- Comment #32 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 108166 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108166&action=edit Bug 24973: Load custom localization file It adds a new localization directory that could host a custom SQL file for each language, xx-YY/custom.sql This will be the last file to be loaded at install time, it can be used to set any difference from default values. It can't be deselected at install time. Moved corresponding files for de-DE, it-IT, nb-NO and completed es-ES To test: 1) Apply the patch 2) Install es-ES/de-DE/it-IT/nb-NO translation (any lang) ( cd misc/translation; ./translate install es-ES ) 3) Do a clean install using es-ES 4) After all files have been loaded check: * A new section labeled "Localization data added" with one file, custom.sql * Inspect the value of FrameworksLoaded syspref, last entry must be custom.sql 5) Remove/rename the file or localization dir and repeat 3/4, install must proceed normally Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #35 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Here we are! Simple and efficient, great work :) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #36 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #37 from Lucas Gass <lucas@bywatersolutions.com> --- enhancement will not be backported to 20.05.x -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 --- Comment #39 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Awesome work everyone, thanks 🙂 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |27619 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27619 [Bug 27619] Remove de-DE installer data -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |27621 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27621 [Bug 27621] Remove it-IT installer data -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |27623 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27623 [Bug 27623] Remove pl-PL installer data -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |27625 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27625 [Bug 27625] Remove uk-UA installer data -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24973 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|release-notes-needed | -- You are receiving this mail because: You are watching all bug changes.