Rifakat (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.0-opw-579221-rha into 
lp:openobject-addons/6.0.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)
Related bugs:
  Bug #1050957 in OpenERP Addons: "[6.0] stock: stock.partial.picking wizard 
has a function with '__' prefix which prevents inheritance"
  https://bugs.launchpad.net/openobject-addons/+bug/1050957

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-579221-rha/+merge/125160

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
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-579221-rha/+merge/125160
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-579221-rha.
=== modified file 'stock/wizard/stock_partial_picking.py'
--- stock/wizard/stock_partial_picking.py	2012-03-14 13:56:28 +0000
+++ stock/wizard/stock_partial_picking.py	2012-09-19 10:44:32 +0000
@@ -68,7 +68,7 @@
             for m in pick.move_lines:
                 if m.state in ('done', 'cancel'):
                     continue
-                result.append(self.__create_partial_picking_memory(m, pick_type))
+                result.append(self._create_partial_picking_memory(m, pick_type))
 
         if 'product_moves_in' in fields:
             res.update({'product_moves_in': result})
@@ -117,7 +117,7 @@
         result['fields'] = _moves_fields
         return result
 
-    def __create_partial_picking_memory(self, move, pick_type):
+    def _create_partial_picking_memory(self, move, pick_type):
         move_memory = {
             'product_id' : move.product_id.id,
             'quantity' : move.product_qty,

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to