Merge lp:~florian-dacosta/ocb-addons/6.1_purchase_line_partner_fix into lp:ocb-addons/6.1

[Florian da Costa]

Automatically derived from https://code.launchpad.net/~akretion-team/openobject-addons/6.1_purchase_line_partner_fix for https://code.launchpad.net/~openerp/openobject-addons/6.1.

[Holger Brunn (Therp) (hbrunn)]

Please make _update_partner_from_purchase being able to cope with multiple ids (it only returns the last PO's order lines), something like

res += purchase['order_line']

in #12

review: Needs Fixing (code review)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Same comments than: https://code.launchpad.net/~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix/+merge/185331/comments/449347 :

l.11 The context must be propagated in the read.
l.24 No mutable in keyword argument: replace "c={}" by "c=None".

l.23-24 Indentation is dubious, see http://www.python.org/dev/peps/pep-0008/#indentation

review: Needs Fixing

[Stefan Rijnhart (Opener) (stefan-opener)]

Setting to WIP after 3 months of inactivity.

Unmerged revisions

6788. By Florian da Costa on 2013-09-12

[FIX] Add function to recalculate the related field partner_id of purchase_order_line because it would not update when the purchase order's partner_id is updated

=== modified file 'purchase/purchase.py'
--- purchase/purchase.py	2012-10-26 15:32:12 +0000
+++ purchase/purchase.py	2013-09-12 17:51:15 +0000
@@ -655,6 +655,13 @@
         except Exception, ex:
             return False
 
+    def _update_partner_from_purchase(self, cr, uid, ids, context=None):
+        res = []
+        obj_purchase = self.pool.get('purchase.order')
+        for purchase in obj_purchase.read(cr, uid, ids, ['order_line']):
+            res = purchase['order_line']
+        return res
+
     _columns = {
         'name': fields.char('Description', size=256, required=True),
         'product_qty': fields.float('Quantity', digits_compute=dp.get_precision('Product UoM'), required=True),
@@ -677,7 +684,9 @@
                                        \n* The \'Cancelled\' state is set automatically when user cancel purchase order.'),
         'invoice_lines': fields.many2many('account.invoice.line', 'purchase_order_line_invoice_rel', 'order_line_id', 'invoice_id', 'Invoice Lines', readonly=True),
         'invoiced': fields.boolean('Invoiced', readonly=True),
-        'partner_id': fields.related('order_id','partner_id',string='Partner',readonly=True,type="many2one", relation="res.partner", store=True),
+        'partner_id': fields.related('order_id','partner_id',string='Partner',readonly=True,type="many2one", relation="res.partner",
+        store = {'purchase.order.line': (lambda self, cr, uid, ids, c={}: ids, [], 10),
+        'purchase.order': (_update_partner_from_purchase, ['partner_id'], 10)}),
         'date_order': fields.related('order_id','date_order',string='Order Date',readonly=True,type="date")
 
     }