Merge lp:~serpent-consulting-services/openobject-addons/7.0-sale_margin-1090931 into lp:openobject-addons

[Serpent Consulting Services]

[Olivier Dony (Odoo) (odo-openerp)]

Indeed, functions that are passed as arguments to stored triggers are plain functions and cannot use the inheritance system. It is possibly to do it by having a simple lambda function that calls self._the_real_method, when inheriting it is really necessary. In that case only the trivial lambda needs to be repeated, and the original method can be overridden.

Let's keep it simple here, your patch does the job :-)

Thanks!

review: Approve

=== modified file 'sale_margin/sale_margin.py'
--- sale_margin/sale_margin.py	2012-11-29 22:26:45 +0000
+++ sale_margin/sale_margin.py	2012-12-16 18:41:23 +0000
@@ -70,7 +70,10 @@
         return result
 
     def _get_order(self, cr, uid, ids, context=None):
-        return super(sale_order, self)._get_order(cr, uid, ids, context=context)
+        result = {}
+        for line in self.pool.get('sale.order.line').browse(cr, uid, ids, context=context):
+            result[line.order_id.id] = True
+        return result.keys()
 
     _columns = {
         'margin': fields.function(_product_margin, string='Margin', help="It gives profitability by calculating the difference between the Unit Price and the cost price.", store={
@@ -81,4 +84,4 @@
 
 sale_order()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file