Merge lp:~camptocamp/purchase-wkfl/7.0-fix-missing-trigger-nbi into lp:~purchase-core-editors/purchase-wkfl/7.0

[Nicolas Bessi - Camptocamp]

[FIX] merge trouble, missing trigger on purchase order line

[Romain Deheele - Camptocamp (romaindeheele)]

LGTM,

Regards,

Romain

review: Approve (code review)

[Leonardo Pistone (lepistone)]

LGTM.

Out of curiosity (I approve in any case) why do you actually need the trigger on the purchase line itself? in theory, purchase_id should not change.

review: Approve (code review)

[Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount)]

Leonardo,

In ngo-addons we often have code that create create PO first and in a second step the PO line, in this case trigger is not call.

[Leonardo Pistone (lepistone)]

thanks!

=== modified file 'framework_agreement/model/purchase.py'
--- framework_agreement/model/purchase.py	2014-02-05 08:47:53 +0000
+++ framework_agreement/model/purchase.py	2014-02-18 13:48:02 +0000
@@ -36,13 +36,19 @@
             res.update([x.id for x in row.order_line])
         return res
 
+    def _get_po_line_store(self, cr, uid, ids, context=None):
+        return ids
+
+
     _store_tuple = (_get_po_store, ['framework_agreement_id'], 20)
+    _line_store_tuple = (_get_po_line_store, [], 20)
 
     _columns = {'framework_agreement_id': fields.related('order_id',
                                                          'framework_agreement_id',
                                                          type='many2one',
                                                          readonly=True,
-                                                         store={'purchase.order': _store_tuple},
+                                                         store={'purchase.order': _store_tuple,
+                                                                'purchase.order.line': _line_store_tuple},
                                                          relation='framework.agreement',
                                                          string='Agreement')}