Merge lp:~openerp-dev/openobject-addons/7.0-opw-586661-ado into lp:openobject-addons/7.0

Proposed by Amit Dodiya (OpenERP)
Status: Merged
Merged at revision: 9210
Proposed branch: lp:~openerp-dev/openobject-addons/7.0-opw-586661-ado
Merge into: lp:openobject-addons/7.0
Diff against target: 53 lines (+28/-4)
2 files modified
account/account_invoice.py (+4/-4)
account/i18n/account.pot (+24/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-586661-ado
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Needs Fixing
Xavier ALT Pending
Review via email: mp+153061@code.launchpad.net

Description of the change

Hello,

[FIX] make the breadcrump string translatable

Steps:
1). Install any language other than english
2). Now switch to that installed language and goto supplier invoice and create new one and save it
You will find the breadcrump string is not translated.
I have also attached here a link of video which show the issue : http://www.youtube.com/watch?v=-NuBlSdrWKQ

Regards,
Amit

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hi Amit,

The missing terms in .POT's will be filled upon when a complete re-export of the translation terms are done.So there is no need to add them to .pot. Please uncommit the changes realted to .pot's.

Thanks,
Naresh Soni
OpenERP Enterprise Service

review: Needs Fixing
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

I will fix it while merging:

- no change to the POTs, will be done when exporting as Naresh explained
- avoid creating translatable terms with trailing spaces - it creates extra terms to translate ("Invoice" is already translated, so why ask translators to translate "Invoice "?!) and there's always a chance translators will not copy the whitespace characters. If the position of the space could change between languages, then the whole sentence needs to be made translatable in a way that translators can understand, for example "Invoice %(invoice_number)s" .

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account/account_invoice.py'
--- account/account_invoice.py 2013-03-06 15:01:42 +0000
+++ account/account_invoice.py 2013-03-13 06:42:22 +0000
@@ -1121,10 +1121,10 @@
1121 if not ids:1121 if not ids:
1122 return []1122 return []
1123 types = {1123 types = {
1124 'out_invoice': 'Invoice ',1124 'out_invoice': _('Invoice '),
1125 'in_invoice': 'Sup. Invoice ',1125 'in_invoice': _('Sup. Invoice '),
1126 'out_refund': 'Refund ',1126 'out_refund': _('Refund '),
1127 'in_refund': 'Supplier Refund ',1127 'in_refund': _('Supplier Refund '),
1128 }1128 }
1129 return [(r['id'], (r['number']) or types[r['type']] + (r['name'] or '')) for r in self.read(cr, uid, ids, ['type', 'number', 'name'], context, load='_classic_write')]1129 return [(r['id'], (r['number']) or types[r['type']] + (r['name'] or '')) for r in self.read(cr, uid, ids, ['type', 'number', 'name'], context, load='_classic_write')]
11301130
11311131
=== modified file 'account/i18n/account.pot'
--- account/i18n/account.pot 2013-03-07 12:19:34 +0000
+++ account/i18n/account.pot 2013-03-13 06:42:22 +0000
@@ -53,6 +53,30 @@
53msgstr ""53msgstr ""
5454
55#. module: account55#. module: account
56#: code:addons/account/account_invoice.py:1124
57#, python-format
58msgid "Invoice "
59msgstr ""
60
61#. module: account
62#: code:addons/account/account_invoice.py:1125
63#, python-format
64msgid "Sup. Invoice "
65msgstr ""
66
67#. module: account
68#: code:addons/account/account_invoice.py:1126
69#, python-format
70msgid "Refund "
71msgstr ""
72
73#. module: account
74#: code:addons/account/account_invoice.py:1127
75#, python-format
76msgid "Supplier Refund "
77msgstr ""
78
79#. module: account
56#: code:addons/account/account_bank_statement.py:36880#: code:addons/account/account_bank_statement.py:368
57#, python-format81#, python-format
58msgid "Journal item \"%s\" is not valid."82msgid "Journal item \"%s\" is not valid."