Bug 12999 - Untranslatable string on enhanced_content.pref
Summary: Untranslatable string on enhanced_content.pref
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: I18N/L10N (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bernardo Gonzalez Kriegel
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-26 00:33 UTC by Bernardo Gonzalez Kriegel
Modified: 2016-10-13 20:40 UTC (History)
4 users (show)

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


Attachments
Bug 12999: Untranslatable string on enhanced_content.pref (2.01 KB, patch)
2014-09-26 00:43 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
Bug 12999: Untranslatable string on enhanced_content.pref (1.99 KB, patch)
2014-09-26 00:45 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 12999: Untranslatable string on enhanced_content.pref (2.10 KB, patch)
2014-10-10 14:15 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 12999: Untranslatable string on enhanced_content.pref (2.10 KB, patch)
2014-10-11 19:09 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 11299 small fix because TransformHtmlToMarc parameters it's changed (842 bytes, patch)
2016-10-13 20:37 UTC, Bouzid Fergani
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Bernardo Gonzalez Kriegel 2014-09-26 00:33:31 UTC
There is a untranslatable "NOTE" on enhanced_content.pref
Comment 1 Bernardo Gonzalez Kriegel 2014-09-26 00:43:15 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2014-09-26 00:45:39 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2014-10-10 14:15:12 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-10-11 19:09:27 UTC
Created attachment 32259 [details] [review]
Bug 12999: Untranslatable string on enhanced_content.pref

This patch modifies LangInstaller.pm to catch strings
on syspref files that have no associated syspref

To test:
1) Update strings for your preffered language
2) Check that the string "you can only choose one source"
is not present on xx-YY-pref.po file

3) Apply the patch
4) Update strings again
5) Check the strig now appears [ and another one :) ]

Doing a diff before and after will show
 # Enhanced Content > All
 msgid "enhanced_content.pref## <strong>NOTE:</strong> you can only
 choose one source of cover images from below, otherwise Koha will show
 the images from all sources selected."
 msgstr ""

 # Local Use
 msgid "local_use.pref## Nothing defined yet."
 msgstr ""

6) Translate the string
7) Install your language, check Administration > System preferences >
   Enhanced content > All
   (on your languge) and check the string is now translated

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Jonathan Druart 2014-10-11 19:10:30 UTC
I am not sure I can explain this patch, but I confirm the only change in po files are:

diff fr-FR-pref.po old/fr-FR-pref.po
2295a2296,2299
> # Enhanced Content > All
> msgid "enhanced_content.pref## <strong>NOTE:</strong> you can only choose one source of cover images from below, otherwise Koha will show the images from all sources selected."
> msgstr ""
> 
3014a3019,3022
> 
> # Local Use
> msgid "local_use.pref## Nothing defined yet."
> msgstr ""
Comment 6 Tomás Cohen Arazi 2014-10-14 14:11:49 UTC
Patch pushed to master.

Thanks Bernardo!
Comment 7 Bouzid Fergani 2016-10-13 20:37:10 UTC
Created attachment 56441 [details] [review]
Bug 11299 small fix because TransformHtmlToMarc parameters it's changed
Comment 8 Bouzid Fergani 2016-10-13 20:40:19 UTC
sorry, it's error