Bug 1036150 - publican deals badly with "\n" in po files for strings corresponding to non-verbatim tags
Summary: publican deals badly with "\n" in po files for strings corresponding to non-v...
Alias: None
Product: Publican
Classification: Community
Component: publican
Version: 3.2
Hardware: Unspecified
OS: Unspecified
Target Milestone: 3.2
Assignee: Jeff Fearn 🐞
QA Contact: tools-bugs
Depends On:
TreeView+ depends on / blocked
Reported: 2013-11-29 15:01 UTC by Raphaël Hertzog
Modified: 2013-12-19 02:46 UTC (History)
2 users (show)

Fixed In Version: 4.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-12-19 02:46:33 UTC

Attachments (Terms of Use)
Patch fixing newline handling in translated strings (665 bytes, patch)
2013-11-29 15:01 UTC, Raphaël Hertzog
no flags Details | Diff

Description Raphaël Hertzog 2013-11-29 15:01:12 UTC
Created attachment 830694 [details]
Patch fixing newline handling in translated strings

I have a translated string like that:

#. Tag: para
#, no-c-format
msgid "With [...] Thank you!"
msgstr ""
"Nous avons reçu le soutien de 650 personnes pendant la campagne de\n"
"financement initiale, et de plusieurs centaines d'autres pendant la\n"
"campagne de libération.  C'est grâce à vous que ce projet a pu voir le\n"
"jour.  Merci à tous !"

Obviously the translators (or its software) word-wrapped the translation string and added newlines where there were none in the original string. Still this should be perfectly OK because he knows that we're writing XML and that newlines should be treated just like spaces.

Unfortunately, that's not the case, the "\n" end up in the generated documents.

I believe that the attached patch fixes this problem with no bad side-effects.

Comment 2 Raphaël Hertzog 2013-11-29 15:05:28 UTC
To be clearer, the attached patch solves the problem for me and I have not yet seen any new problems since I use that patch.

Comment 3 HSS Product Manager 2013-12-17 02:47:35 UTC
HSS-QE has reviewed and declined this request. QE for this bug will be handled by IED.

Comment 4 Jeff Fearn 🐞 2013-12-17 03:01:19 UTC
Applied patch. Updated PUG it-IT translation and built HTML output.

To ssh://git.fedorahosted.org/git/publican.git
   252d976..6cc0236  HEAD -> devel

Note You need to log in before you can comment on or make changes to this bug.