[patch] Implement i18n support

[Steffen Hoffmann]

We should start a 0.12 branch and work on this.

[Steffen Hoffmann]

I'll provide patches based on current 0.11 branch as promised per email.

I work with Mercurial here cloning full t-h.o SVN repo, so these patches are meant to become changesets and should apply directly to current screenshotsplugin branch.

[Radek Bartoň]

OK, please attach them then here.

[Steffen Hoffmann]

despite of name mostly creating some init() modules, just one occurance of self.config.get needed, formatting (proposal) for better readability

[Steffen Hoffmann]

a proposal on more intuitive permission inheritance - not a requirement at all

[Steffen Hoffmann]

make titels regarding case and wording with(out) colon more like native Trac - later i18n work relies on that exact strings

[Steffen Hoffmann]

another optional proposal: more readable SQL statements, the second hardest part in this patch stack, but not required for i18n at all

[Steffen Hoffmann]

just branching of 0.12 off from 0.11 here after the previous preparation

[Steffen Hoffmann]

core code and markup to bring in and setup i18n/l10n for Trac plugins, required markup for translatable strings (=msgid's), at least as far as they are not auto-detected (in Genshi HTML templates), including minor related code changes, i.e. to avoid msgid duplication - packed to circumvent SPAM t-h.o's filter

[Steffen Hoffmann]

workaround needed for still unknown reasons, but wouldn't get some button label text's translated otherwise - might lead to a Genshi ticket later on

[Steffen Hoffmann]

inital message catalog template extraction, German translations

[Steffen Hoffmann]

Ok, just finished my work for working i18n/l10n now.

I've attached in patch order and already commented the patches above. All are automatically created by Mercurial Queues extension, but Trac has problems to show some of them, nevertheless they are there (see original format then).

Where I brought in disputable stuff, I've tried to flag and comment accordingly. No problem to rework for most of it, if you insist. However there are some restrictions that are imposed by the way Trac i18n support for plugins is meant to word. You may want to take a look at the reference documentation for ​adding i18n/l10n to Trac plugins.

Beware, the Genshi/Babel part is still not totally settled - we have a critical case here as well, that I needed some extra code to circumvent limitations most probably related to automatic Genshi template translations by genshi.filters.i18n.Translator for now. I've just shifted string definition and translation from the Genshi template into python code space know to provide more reliable i18n support and only import back into template afterwards.

Let me know, how you'd like to proceed. I.e., if you see need for extensive discussion on particular patches, please tell me, so I could open separate tickets for them as needed. For my own part I already happily use the patched version in production and will fix/improve, as needed for my application. At the very minimum it's already known to be safe for non-ascii strings in screenshot names, description, etc.

I'd still like to add message catalogs to our Trac plugin l10n project via ​Transifex. We'll certainly see contributions for other languages soon without much additional effort on plugin maintainers side, as the Trac plugin l10n project is managed by myself.

[Radek Bartoň]

OK, thank you. I'll review the patches tomorrow.

[Radek Bartoň]

I'm sequentially applying and reviewing the patches now.

Regarding the inheriting-perms.patch: I don't see the reason of this patch. An explanation for the former version is that when you wan't to edit or delete a screenshot you have to see it but you don't have to have rights for adding it (in case of edit) or editing it (in case of delete). SCREENSHOTS_ADMIN is just shortcut for permission to do anything.

[Radek Bartoň]

Ehh, wan't is really nice typo :-).

[Radek Bartoň]

Pre 0.12 branch split reviewed patch.

[Radek Bartoň]

OK, I've attached the pre-0.12-branch.patch. You can commit it as soon as you'll have permission to do it. Then I'll create a 0.12 branch and review the i18n patches.

[Radek Bartoň]

Now you have the permission to commit the patch.

[Steffen Hoffmann]

Thanks for the pre-branch review as well as for SVN commit. Nice take on the SQL statements making them even more uniform and readable. There is definitely good stuff in this one. Nice cooperation so far. Patch committed in changeset [8663]. Will comment regarding permissions concept later.

[Radek Bartoň]

l18n.patch​ is file with reviewed patches for l18n. I didn't apply the [btt-label-translation.patch] because we should find proper way how to translate such literals. I'm asking now at #trac IRC channel about it. You can commit it now, please.

I also started translation of ScreenshotPlugin to Czech and registered at Transifex (user Blackhex). Could you please add ScreenshotsPlugin (and maybe DiscussionPlugin) components to Trac Plugin L10N project and add me as member of Czech team?

[Radek Bartoň]

I replaced the l18n.patch​ with a new version that fixes the problem. It was caused by the fact that implementation of _() function in template was default one. You have to import customized implementation from the plugin. Now all translations work.

[Steffen Hoffmann]

   <xi:include href="layout.html"/>
   <xi:include href="macros.html"/>
+  <?python
+    from tracscreenshots.core import _
+  ?>
   <head>
     <title>Screenshots</title>

I love this one. Simple, straightforward - beautiful. I was really at a loss, had asked at #trac myself before, but I couldn't get a grip on this laying that near as it is obvious now. Thanks so much for lightning a bunch of torches for me. So we can happily forget about the additional fix-up patch, as I hoped for some day.

[Steffen Hoffmann]

I'm missing the new setup.cfg file. You'll certainly fail to test and get the i18n stuff working without that configuration (extractor/translator calls, i18n catalog domain, paths). As I see nothing seriously disputable in there, I'll simply re-add the file before commit, right?

[Radek Bartoň]

Ah, sorry, I forgot to add it to version control before doing svn diff. Please do so but change Babel version requirement to 0.9.4 because it is the current version for Ubuntu stable.

[Steffen Hoffmann]

Replying to Blackhex:

l18n.patch​ is file with reviewed patches for l18n. I didn't apply the [btt-label-translation.patch] because we should find proper way how to translate such literals. I'm asking now at #trac IRC channel about it. You can commit it now, please.

Just doing a second check and local test as you see from my comment before.

I also started translation of ScreenshotPlugin to Czech and registered at Transifex (user Blackhex). Could you please add ScreenshotsPlugin (and maybe DiscussionPlugin) components to Trac Plugin L10N project and add me as member of Czech team?

Sure, it'll be a pleasure for me to add plugins. But sorry, you can't be simply a member of the Czech translator team, I needed a coordinator to bring it into life - done right now. :-)

Only I need to have a look at DiscussionPlugin, was not aware, that it already has i18n in. But OTOH it was months ago and well before Trac 0.12 release, when I last scanned all plugin code from t-h.o for traces of i18n support. I'm really glad, that there is now some progress regarding i18n.

[anonymous]

Only I need to have a look at DiscussionPlugin, was not aware, that it already has i18n in. But OTOH it was months ago and well before Trac 0.12 release, when I last scanned all plugin code from t-h.o for traces of i18n support. I'm really glad, that there is now some progress regarding i18n.

Currently it doesn't have l18n support but I can write it.

[Steffen Hoffmann]

Replying to Blackhex:

Ah, sorry, I forgot to add it to version control before doing svn diff. Please do so but change Babel version requirement to 0.9.4 because it is the current version for Ubuntu stable.

Ok, thanks for the quick response. Requirement is set in setup.py; will check that one, but I see no serious problem for that move. I've just copied that requirement form Trac 0.12, since I had no clue on my own, working with Babel > 1.0 all the time.

[Steffen Hoffmann]

Replying to anonymous:

Only I need to have a look at DiscussionPlugin, was not aware, that it already has i18n in. But OTOH it was months ago and well before Trac 0.12 release, when I last scanned all plugin code from t-h.o for traces of i18n support. I'm really glad, that there is now some progress regarding i18n.

Currently it doesn't have l18n support but I can write it.

Patch welcome. ;-) Not kidding, you seem quite up to the task considering the way you helped me with this one. And I'll be happy to assist, if you just need a second eye.

[Radek Bartoň]

Hmm, I quite don't understand how Transifex works. When I'm coordinator I can't submit translations while I can't be both coordinator and member :-).

Than you for letting me in anyway.

[Radek Bartoň]

Ok, I've commited (r8717) Czech translation. We are done with this ticket so I'm closing it. Thank you for your cooperation.

[Steffen Hoffmann]

Well thank you too. I'll setup ScreenshotsPlugin as another component in our Transifex project for Trac plugin translation and add links to ScreenshotsPlugin wiki page.