Fix sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations.
Created attachment 52156 [details] [review] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch
*** Bug 16669 has been marked as a duplicate of this bug. ***
Created attachment 52946 [details] Screen capture testing patch 16687
Patch partially corrects the issue but there are still some problems (see screen capture).
(In reply to jmbroust from comment #4) > Patch partially corrects the issue but there are still some problems (see > screen capture). Hi jmbroust, Thanks for testing. I can not test right now due to a weak internet connection. It seems that the problem is related to the French translation and not to my code changes. Please check french translation on http://translate.koha-community.org/ How does it look like in English? Can you please test the English version and provide a screenhot?
I re-tested. The issues in the screenshots are related to wrong or missing French translations in http://translate.koha-community.org/fr/16.05/fr-FR-pref.po In English the changes look fine. Switching back to 'Needs Signoff'
Some of the choices no longer make grammatical sense following the change of the description because they were designed to fit within a sentence: "Show debugging information in the browser when an internal error occurs:" - none - some - lots of I suggest: "How much debugging information to show in the browser when an internal error occurs:" - None - Some - All ----------------------- For DefaultToLoggedInLibraryCircRules, DefaultToLoggedInLibraryNoticesSlips, and DefaultToLoggedInLibraryOverdueTriggers I suggest the choices be: no: "All libraries" yes: "The logged-in library"
Created attachment 53335 [details] [review] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv
Created attachment 53370 [details] [review] [SGINED-OFF] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv Signed-off-by: Owen Leonard <oleonard@myacpl.org>
This seems like a step back to how the old system preference editor worked - having the choice separate from the sentence and at the end of each. The translation of the system preference file was designed to keep the strings in sequence, so you can 'see' the sentence strings together - but it doesn't allow to change the sequence of sentence parts. I am not sure if this will work better in all possible scenarios and it will require translators to retranslate the whole file - and even more files if we decide to use this pattern for the other preferences as well. I'd like to get another translator's view on this.
(In reply to Katrin Fischer from comment #10) > This seems like a step back to how the old system preference editor worked - > having the choice separate from the sentence and at the end of each. The > translation of the system preference file was designed to keep the strings > in sequence, so you can 'see' the sentence strings together - but it doesn't > allow to change the sequence of sentence parts. > > I am not sure if this will work better in all possible scenarios and it will > require translators to retranslate the whole file - and even more files if > we decide to use this pattern for the other preferences as well. > > I'd like to get another translator's view on this. Note: The problem with the sequence of the sentence parts is described in Bug 16669 - Options in pref.po should be expressed in %s to allow translatability:$ From bug description: misc/translator/po/*-pref.po lacks translatability because the position of options are fixed. For example, admin.pref#casAuthentication consists of three entries. # CAS for login authentication. # Don't use # Use It is used to construct the following sentence with a drop-down list. "[Don't use/Use] CAS for login authentication." The Japanese translation of this would look like the following. "CASをログイン認証に使用[しない/する]" However, it is impossible to use this translation because the position of the drop-down list is fixed. This could be solved by expressing options as %s and allow the translator to modify the position. The following is an example. # %s CAS for login authentication.
Created attachment 55788 [details] [review] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed off, but still would like more translators to take a look before it goes through QA
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref Looking at the diff, I don't understand why "- Note that this value has no effect if the UsageStats system preference is set to "Don't share"" have been removed.
Created attachment 55887 [details] [review] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Rebased to apply on master and re-inserted test (see comment #14) -2016-09-28/mv
Created attachment 56525 [details] [review] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Rebased to apply on master and re-inserted test (see comment #14) -2016-09-28/mv Signed-off-by: Michael Kuhn <mik@adminkuhn.ch>
Still feels a bit like turning back time to me - in a less readable fashion. We used to have the pull down/options in a separate column and then the description next to it. But 3 sign-offs...
Created attachment 56569 [details] [review] [PASSED QA] Bug 16687 - Translatability: Fix issues in Administration preferences This patch fixes sentence splitting and other translation issues in file koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref to avoid odd translations. To test: - Without patch, open Home > Administration > System preferences and go to tab 'Administration" - Apply patch - Open the same page in a new window and compare the text: - Verify that no sentences are splitted - Verify that the text changes do not introduce misunderstandings and have the same meaning as without patch Amended for comment #7 2016-07-12/mv Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Rebased to apply on master and re-inserted test (see comment #14) -2016-09-28/mv Signed-off-by: Michael Kuhn <mik@adminkuhn.ch> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
I don't think this is the way to go. Have a look at es-ES translations, they fixed this issue correctly.
Created attachment 56615 [details] [review] [COUNTER-PATCH] Bug 16687: Translatability: Fix issues in Administration preferences This is how I'd fix that issue, please give it a try.
(In reply to Jonathan Druart from comment #20) > Created attachment 56615 [details] [review] [review] > [COUNTER-PATCH] Bug 16687: Translatability: Fix issues in Administration > preferences > > This is how I'd fix that issue, please give it a try. Hi Jonathan, Thanks for the counter patch. Can you please elaborate a little bit what your counter patch does? How does it ease the problem with splitted sentences? Marc
It removes the 'file.pref#prefname#' parts from the msgstr.
(In reply to Jonathan Druart from comment #22) > It removes the 'file.pref#prefname#' parts from the msgstr. How does it ease the problem with splitted sentences?
It won't, it just removes the strings I mentionned.
(In reply to Jonathan Druart from comment #24) > It won't, it just removes the strings I mentionned. But what I'm doing with the first patch is fixing sentence splitting. It is not related to wrong french translations , see comment #5 I think if you want to fix the problem mentionned by jmbroust (screenshot and comments #4 and #5), your patch should go to a separate bug.
ok, back to Passed QA.
Pushed to master for 16.11, thanks Marc!
Pushed in 16.05. Will be in 16.05.05.
This will force translators to retranslate all the prefs - why push it to a maintenance release?
More so with releasing at the same day - this means they will just be NOT translated at all. What happened to string freezes?
OK. You're correct. I can mark this bug as 'enhancement', and revert it from 16.05. Do you agree with this?
Done.