Merge lp:~openerp-dev/openobject-addons/6.0-opw-579221-rha into lp:openobject-addons/6.0

Proposed by Rifakat Husen (OpenERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 5344
Proposed branch: lp:~openerp-dev/openobject-addons/6.0-opw-579221-rha
Merge into: lp:openobject-addons/6.0
Diff against target: 21 lines (+2/-2)
1 file modified
stock/wizard/stock_partial_picking.py (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-579221-rha
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+125160@code.launchpad.net

Description of the change

Dear sir,

I have changed the method name of class stock.partial.picking which has prefix as
__ (2xunderscores). It prevents inheritance for that method.

def __create_partial_picking_memory ==> def _create_partial_picking_memory

Thanks for the review,
Rifakat Haradwala

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

Unmerged revisions

5344. By Niels Huylebroeck

[FIX] stock: stock.partial.picking class has method prefix as __ so that it prevents inheritance for that

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'stock/wizard/stock_partial_picking.py'
2--- stock/wizard/stock_partial_picking.py 2012-03-14 13:56:28 +0000
3+++ stock/wizard/stock_partial_picking.py 2012-09-19 10:44:32 +0000
4@@ -68,7 +68,7 @@
5 for m in pick.move_lines:
6 if m.state in ('done', 'cancel'):
7 continue
8- result.append(self.__create_partial_picking_memory(m, pick_type))
9+ result.append(self._create_partial_picking_memory(m, pick_type))
10
11 if 'product_moves_in' in fields:
12 res.update({'product_moves_in': result})
13@@ -117,7 +117,7 @@
14 result['fields'] = _moves_fields
15 return result
16
17- def __create_partial_picking_memory(self, move, pick_type):
18+ def _create_partial_picking_memory(self, move, pick_type):
19 move_memory = {
20 'product_id' : move.product_id.id,
21 'quantity' : move.product_qty,