Merge lp:~camptocamp/openerp-connector-ecommerce/7.0-related_backorder_ids_copy-rde into lp:~openerp-connector-core-editors/openerp-connector-ecommerce/7.0

[Romain Deheele - Camptocamp]

Hello,

This MP is linked with https://bugs.launchpad.net/openerp-connector-ecommerce/+bug/1303655
It proposes to do not link related_backorder_ids from an original picking to a new picking copy.

Regards,

Romain

[Leonardo Pistone (lepistone)]

lgtm

review: Approve (code review)

[lp:~camptocamp/openerp-connector-ecommerce/7.0-related_backorder_ids_copy-rde]

351. By Guewen Baconnier @ Camptocamp on 2014-04-24

do not build a new dict when default is an empty dict, not necessary to build a context dict (not used)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Thanks!

review: Approve

=== modified file 'connector_ecommerce/stock.py'
--- connector_ecommerce/stock.py	2013-03-19 09:42:19 +0000
+++ connector_ecommerce/stock.py	2014-04-24 08:18:47 +0000
@@ -24,6 +24,7 @@
 from openerp.addons.connector.session import ConnectorSession
 from .event import on_picking_out_done, on_tracking_number_added
 
+
 class stock_picking(orm.Model):
     _inherit = 'stock.picking'
 
@@ -39,8 +40,8 @@
         session = ConnectorSession(cr, uid, context=context)
         # Look if it exists a backorder, in that case call for partial
         picking_records = self.read(cr, uid, ids,
-                                 ['id', 'related_backorder_ids', 'type'],
-                                 context=context)
+                                    ['id', 'related_backorder_ids', 'type'],
+                                    context=context)
         for picking_vals in picking_records:
             if picking_vals['type'] != 'out':
                 continue
@@ -52,6 +53,15 @@
                                      picking_vals['id'], picking_method)
         return res
 
+    def copy(self, cr, uid, id, default=None, context=None):
+        if default is None:
+            default = {}
+        else:
+            default = default.copy()
+        default['related_backorder_ids'] = False
+        return super(stock_picking, self).copy(cr, uid,
+                                               id, default, context=context)
+
 
 class stock_picking_out(orm.Model):
     _inherit = 'stock.picking.out'