Summary: | i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation | ||
---|---|---|---|
Product: | Koha | Reporter: | paxed <pasi.kallinen> |
Component: | I18N/L10N | Assignee: | Bernardo Gonzalez Kriegel <bgkriegel> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | bgkriegel, f.demians, gmcharlt |
Version: | unspecified | ||
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: | ||
Circulation function: | |||
Attachments: |
Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation
Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation [PASSED QA] Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation |
Description
paxed
2014-01-08 11:51:38 UTC
Created attachment 26801 [details] [review] Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation i18n toolchain fails to pick text defined inside [% ... %] This patch fixes all the cases I could find To test: 1. Update your preferred language (cd misc/translator; perl translate update fi-FI) 2. Count headers for example egrep "^msgid.*Koha online" misc/translator/po/fi-FI-* | wc -l 3. Apply the patch 4. Repeat 1 and 2, now there are more lines Or translate to som lang and visit every touched page and look the title Created attachment 27549 [details] [review] Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation i18n toolchain fails to pick text defined inside [% ... %] This patch fixes all the cases I could find To test: 1. Update your preferred language (cd misc/translator; perl translate update fi-FI) 2. Count headers for example egrep "^msgid.*Koha online" misc/translator/po/fi-FI-* | wc -l 3. Apply the patch 4. Repeat 1 and 2, now there are more lines Or translate to some lang and visit every touched page and look the title Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Created attachment 27663 [details] [review] [PASSED QA] Bug 11497 - i18n toolchain fails to pick up TT code in the format 'SomeVariable or "Some Text"' for translation i18n toolchain fails to pick text defined inside [% ... %] This patch fixes all the cases I could find To test: 1. Update your preferred language (cd misc/translator; perl translate update fi-FI) 2. Count headers for example egrep "^msgid.*Koha online" misc/translator/po/fi-FI-* | wc -l 3. Apply the patch 4. Repeat 1 and 2, now there are more lines Or translate to some lang and visit every touched page and look the title Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Works as described. Template changes only. Maybe we should add this to the coding guidelines? Pushed to master. Thanks, Bernardo! (In reply to Katrin Fischer from comment #4) > Maybe we should add this to the coding guidelines? Although I pushed the patch, the construct that it replaces seems useful on the face of it, and it's certainly less verbose. I'd rather see the translation code be taught how to deal with it. |