[po4a-Bugs][314748] Text.pm: add an option to force no wrap lines in translated text
by po4a-bugs@alioth.debian.org
po4a-Bugs item #314748 was changed at 12.07.2014 01:14 by Alexander Golubev
You can respond by visiting:
https://alioth.debian.org/tracker/?func=detail&atid=410622&aid=314748&gro...
Status: Open
Priority: 3
Submitted By: Alexander Golubev (fat-zer-guest)
Assigned to: Nobody (None)
Summary: Text.pm: add an option to force no wrap lines in translated text
Category: None
Group: None
Resolution: None
Initial Comment:
subj;
such option may be useful when e.g. translating a docbook witch includes shell scripts directly to it self and the translator wants to get the same structure on output.
----------------------------------------------------------------------
Comment By: Alexander Golubev (fat-zer-guest)
Date: 12.07.2014 01:14
Message:
Here is the promised patch.
I haven't done any unittest's yet because the Text module has no such yet.
----------------------------------------------------------------------
Comment By: Alexander Golubev (fat-zer-guest)
Date: 11.07.2014 16:46
Message:
> Also, I have a doubt about the fact that --auto accepts a comma separated list of tags that should be wrapped. It seems to me that your patch accepts a boolean option where wrap is completely disabled or not, am I wrong? If so, tests cases would be welcome. If I'm right please update the doc.
The patch accept 1 option --wrap with one of 3 parameters:
auto : wrap lines, the current behavior
force_no : no line wrapping: the lines in the translation will be broken on the same place as they were in the source.
no : even I don't clearly understand that's that option does =) it supposed to broke lines in special formats: asciidoc, control, debianchangelog, etc only if there are some special conditions. Although, I'm not really into those formats so I don't really know how that supposed to be done...
So IMO it would be better to stay only with 1 boolean option "--no-wrap" or "--force-no-wrap" to behave like the "--wrap=forse_no" in the current patch.
I'll prepare the new patch in a day...
----------------------------------------------------------------------
Comment By: Martin Quinson (mquinson)
Date: 09.07.2014 00:44
Message:
I would feel better if you could provide a little test case in the t/ directory to ensure that it works as expected (and that it still does in the future).
Also, I have a doubt about the fact that --auto accepts a comma separated list of tags that should be wrapped. It seems to me that your patch accepts a boolean option where wrap is completely disabled or not, am I wrong? If so, tests cases would be welcome. If I'm right please update the doc.
In any case, many thanks for the report and for the patch!
----------------------------------------------------------------------
Comment By: Alexander Golubev (fat-zer-guest)
Date: 07.07.2014 14:20
Message:
Here is a proposed patch... But somebody should review it...
----------------------------------------------------------------------
You can respond by visiting:
https://alioth.debian.org/tracker/?func=detail&atid=410622&aid=314748&gro...