From 1b19cb71e3ddb1c73fec90d482124cc689ce9360 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sun, 26 Jan 2014 14:29:27 +0100
Subject: [PATCH] Use fields.depends decorator

issue3487
review2301002
---
 work.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/work.py b/work.py
index d53a68b926..fe3bc163d3 100644
--- a/work.py
+++ b/work.py
@@ -23,8 +23,7 @@ class Work:
         states={
             'invisible': ~Eval('timesheet_available'),
             },
-        depends=['timesheet_available'],
-        on_change=['product', 'party', 'hours', 'company'])
+        depends=['timesheet_available'])
     list_price = fields.Numeric('List Price',
         digits=(16, Eval('currency_digits', 2)), depends=['currency_digits'])
     revenue = fields.Function(fields.Numeric('Revenue',
@@ -36,8 +35,8 @@ class Work:
                 },
             digits=(16, Eval('currency_digits', 2)),
             depends=['currency_digits', 'timesheet_available']), 'get_cost')
-    currency_digits = fields.Function(fields.Integer('Currency Digits',
-            on_change_with=['company']), 'on_change_with_currency_digits')
+    currency_digits = fields.Function(fields.Integer('Currency Digits'),
+        'on_change_with_currency_digits')
 
     @classmethod
     def get_cost(cls, works, name):
@@ -101,6 +100,7 @@ class Work:
 
         return cls.sum_tree(works, getter)
 
+    @fields.depends('company')
     def on_change_with_currency_digits(self, name=None):
         if self.company:
             return self.company.currency.digits
@@ -115,6 +115,7 @@ class Work:
             return company.currency.digits
         return 2
 
+    @fields.depends('product', 'party', 'hours', 'company')
     def on_change_product(self):
         pool = Pool()
         User = pool.get('res.user')