openerp-community-reviewer team mailing list archive

[Lionel Sausin - Numérigraphe <ls@xxxxxxxxxxxxxxxx>]

Lionel Sausin - Numérigraphe has proposed merging lp:~numerigraphe-team/ocb-server/7.0-translation-order-933496-numerigraphe into lp:ocb-server.

Requested reviews:
  OpenERP Community Backports Team (ocb)

For more details, see:
https://code.launchpad.net/~numerigraphe-team/ocb-server/7.0-translation-order-933496-numerigraphe/+merge/209910

This branch lets the server export translations in a consistent order.
This is useful for comparing/merging translations, for example when you have custom modules outside Launchpad's translation mechanism, or when you customize translations using the client: when you export translations from the client, the patch is clean.
-- 
https://code.launchpad.net/~numerigraphe-team/ocb-server/7.0-translation-order-933496-numerigraphe/+merge/209910
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~numerigraphe-team/ocb-server/7.0-translation-order-933496-numerigraphe into lp:ocb-server.

=== modified file 'openerp/tools/translate.py'
--- openerp/tools/translate.py	2014-02-06 10:51:41 +0000
+++ openerp/tools/translate.py	2014-03-07 12:46:00 +0000
@@ -458,7 +458,7 @@
                 row.setdefault('tnrs', []).append((type, name, res_id))
                 row.setdefault('comments', set()).update(comments)
 
-            for src, row in grouped_rows.items():
+            for src, row in sorted(grouped_rows.items()):
                 if not lang:
                     # translation template, so no translation value
                     row['translation'] = ''