Merge lp:~openerp-dev/openobject-addons/6.1-opw-573426-skh into lp:openobject-addons/6.1

Proposed by Somesh Khare
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 6731
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-573426-skh
Merge into: lp:openobject-addons/6.1
Diff against target: 18 lines (+3/-3)
1 file modified
product_margin/wizard/product_margin.py (+3/-3)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-573426-skh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Somesh Khare (community) Needs Resubmitting
Review via email: mp+101064@code.launchpad.net

Description of the change

Hello Sir,

[Fix]: Product margin header is not translated due to the context did not passed lang correctly.

This branch fixes the issue.

Kindly review the branch and please share your view on it.

Thanks
Somesh Khare

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

somesh use context.update(key=value) in the code also make necessary changes at other places too in the same file.

review: Needs Fixing
6731. By Somesh Khare

[IMP] product,margin.py : used context.update(key=value) for the rest of the code to update the context as suggested

Revision history for this message
Somesh Khare (somesh.khare) wrote :

Hello Sir,

I have made the changes as suggested by you.

Thanks
Somesh Khare

review: Needs Resubmitting
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-573426-port-psi/+merge/132090 and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh

Unmerged revisions

6731. By Somesh Khare

[IMP] product,margin.py : used context.update(key=value) for the rest of the code to update the context as suggested

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'product_margin/wizard/product_margin.py'
--- product_margin/wizard/product_margin.py 2011-01-14 00:11:01 +0000
+++ product_margin/wizard/product_margin.py 2012-04-06 09:36:19 +0000
@@ -64,11 +64,11 @@
64 #get the current product.margin object to obtain the values from it64 #get the current product.margin object to obtain the values from it
65 product_margin_obj = self.browse(cr, uid, ids, context=context)[0]65 product_margin_obj = self.browse(cr, uid, ids, context=context)[0]
6666
67 context = {'invoice_state' : product_margin_obj.invoice_state}67 context.update(invoice_state = product_margin_obj.invoice_state)
68 if product_margin_obj.from_date:68 if product_margin_obj.from_date:
69 context.update({'date_from': product_margin_obj.from_date})69 context.update(date_from = product_margin_obj.from_date)
70 if product_margin_obj.to_date:70 if product_margin_obj.to_date:
71 context.update({'date_to': product_margin_obj.to_date})71 context.update(date_to = product_margin_obj.to_date)
72 return {72 return {
73 'name': _('Product Margins'),73 'name': _('Product Margins'),
74 'context': context,74 'context': context,