From c7aa7901b79d561fda5c15acdf27bb4da36dd1cb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Thu, 18 Mar 2010 18:17:16 +0100
Subject: [PATCH] Remove ids from on_change* calls

---
 purchase.py | 35 ++++++++++++++++-------------------
 1 file changed, 16 insertions(+), 19 deletions(-)

diff --git a/purchase.py b/purchase.py
index 697e4a9599..95941fe946 100644
--- a/purchase.py
+++ b/purchase.py
@@ -205,7 +205,7 @@ class Purchase(ModelWorkflow, ModelSQL, ModelView):
     def default_shipment_state(self, cursor, user, context=None):
         return 'none'
 
-    def on_change_party(self, cursor, user, ids, vals, context=None):
+    def on_change_party(self, cursor, user, vals, context=None):
         party_obj = self.pool.get('party.party')
         address_obj = self.pool.get('party.address')
         payment_term_obj = self.pool.get('account.invoice.payment_term')
@@ -232,7 +232,7 @@ class Purchase(ModelWorkflow, ModelSQL, ModelView):
                     res['payment_term'], context=context).rec_name
         return res
 
-    def on_change_with_currency_digits(self, cursor, user, ids, vals,
+    def on_change_with_currency_digits(self, cursor, user, vals,
             context=None):
         currency_obj = self.pool.get('currency.currency')
         if vals.get('currency'):
@@ -257,8 +257,7 @@ class Purchase(ModelWorkflow, ModelSQL, ModelView):
             res[purchase.id] = purchase.currency.digits
         return res
 
-    def on_change_with_party_lang(self, cursor, user, ids, vals,
-            context=None):
+    def on_change_with_party_lang(self, cursor, user, vals, context=None):
         party_obj = self.pool.get('party.party')
         if vals.get('party'):
             party = party_obj.browse(cursor, user, vals['party'],
@@ -281,7 +280,7 @@ class Purchase(ModelWorkflow, ModelSQL, ModelView):
                 res['language'] = purchase.party.lang.code
         return res
 
-    def on_change_lines(self, cursor, user, ids, vals, context=None):
+    def on_change_lines(self, cursor, user, vals, context=None):
         currency_obj = self.pool.get('currency.currency')
         tax_obj = self.pool.get('account.tax')
         invoice_obj = self.pool.get('account.invoice')
@@ -999,8 +998,7 @@ class PurchaseLine(ModelSQL, ModelView):
             res[line.id] = val
         return res
 
-    def on_change_with_unit_digits(self, cursor, user, ids, vals,
-            context=None):
+    def on_change_with_unit_digits(self, cursor, user, vals, context=None):
         uom_obj = self.pool.get('product.uom')
         if vals.get('unit'):
             uom = uom_obj.browse(cursor, user, vals['unit'],
@@ -1031,7 +1029,7 @@ class PurchaseLine(ModelSQL, ModelView):
         res = {}
         return res
 
-    def on_change_product(self, cursor, user, ids, vals, context=None):
+    def on_change_product(self, cursor, user, vals, context=None):
         party_obj = self.pool.get('party.party')
         product_obj = self.pool.get('product.product')
         uom_obj = self.pool.get('product.uom')
@@ -1098,11 +1096,11 @@ class PurchaseLine(ModelSQL, ModelView):
         vals = vals.copy()
         vals['unit_price'] = res['unit_price']
         vals['type'] = 'line'
-        res['amount'] = self.on_change_with_amount(cursor, user, ids,
-                vals, context=context)
+        res['amount'] = self.on_change_with_amount(cursor, user, vals,
+                context=context)
         return res
 
-    def on_change_quantity(self, cursor, user, ids, vals, context=None):
+    def on_change_quantity(self, cursor, user, vals, context=None):
         product_obj = self.pool.get('product.product')
 
         if context is None:
@@ -1126,10 +1124,10 @@ class PurchaseLine(ModelSQL, ModelView):
                 context=ctx2)[vals['product']]
         return res
 
-    def on_change_unit(self, cursor, user, ids, vals, context=None):
-        return self.on_change_quantity(cursor, user, ids, vals, context=context)
+    def on_change_unit(self, cursor, user, vals, context=None):
+        return self.on_change_quantity(cursor, user, vals, context=context)
 
-    def on_change_with_amount(self, cursor, user, ids, vals, context=None):
+    def on_change_with_amount(self, cursor, user, vals, context=None):
         currency_obj = self.pool.get('currency.currency')
         if vals.get('type') == 'line':
             if isinstance(vals.get('_parent_purchase.currency'), (int, long)):
@@ -1418,8 +1416,7 @@ class Template(ModelSQL, ModelView):
             return True
         return False
 
-    def on_change_with_purchase_uom(self, cursor, user, ids, vals,
-            context=None):
+    def on_change_with_purchase_uom(self, cursor, user, vals, context=None):
         uom_obj = self.pool.get('product.uom')
         res = False
 
@@ -1460,9 +1457,9 @@ Template()
 class Product(ModelSQL, ModelView):
     _name = 'product.product'
 
-    def on_change_with_purchase_uom(self, cursor, user, ids, vals, context=None):
+    def on_change_with_purchase_uom(self, cursor, user, vals, context=None):
         template_obj = self.pool.get('product.template')
-        return template_obj.on_change_with_purchase_uom(cursor, user, ids, vals,
+        return template_obj.on_change_with_purchase_uom(cursor, user, vals,
                 context=context)
 
     def get_purchase_price(self, cursor, user, ids, quantity=0, context=None):
@@ -1800,7 +1797,7 @@ class Move(ModelSQL, ModelView):
         return self.on_change_with_purchase_visible(cursor, user, [], vals,
                 context=context)
 
-    def on_change_with_purchase_visible(self, cursor, user, ids, vals,
+    def on_change_with_purchase_visible(self, cursor, user, vals,
             context=None):
         location_obj = self.pool.get('stock.location')
         if vals.get('from_location'):