Merge lp:~openerp-dev/openobject-addons/7.0-595239-opw-skh into lp:openobject-addons/7.0

[Somesh Khare]

Hello Sir,

[FIX]delivery: delivery - partial delivery will copy tracking number in the backorder picking.

Steps to reproduce this issue,
1. connect to runbot
2. go to Warehouse -> delivery orders
3. select OUT/001
4. force availability of the picking
5. edit the picking, set a carrier tracking number of Z1234
6. Make a partial delivery, with only 1 EXT-HDD

Expected result:

* the OUT/001 picking should have the tracking number Z1234, and be in state Delivered
* the OUT/xxxx picking, backorder of OUT/001 should be in state "ready to deliver" and have a blank tracking number

Actual result:

* the backorder of OUT/001 has the Z1234 tracking number

This branch fixes this issue.Kindly review the branch and share your views.

Thanks,
Somesh Khare

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

missing test.

I added one and submitted for MP in https://code.launchpad.net/~camptocamp/openobject-addons/7.0-595239-opw-skh-afe/+merge/201002

review: Disapprove

Unmerged revisions

9332. By Somesh Khare(OpenERP) <email address hidden> on 2013-07-30

[FIX]delivery: delivery - partial delivery will copy tracking number in the backorder picking (Case: ref 595239)

=== modified file 'delivery/stock.py'
--- delivery/stock.py	2012-12-18 08:59:14 +0000
+++ delivery/stock.py	2013-10-14 12:28:43 +0000
@@ -51,6 +51,12 @@
             result[line.picking_id.id] = True
         return result.keys()
 
+    def write(self, cr, uid, ids, vals, context=None):
+        if context is None: context = {}
+        if vals.get('backorder_id'):
+            vals.update(carrier_tracking_ref = '')
+        return super(stock_picking, self).write(cr, uid, ids, vals, context=context)
+
     _columns = {
         'carrier_id':fields.many2one("delivery.carrier","Carrier"),
         'volume': fields.float('Volume'),