From 97a199aae7c65b9bcd7d82b5bf28fc446bfaf3cd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Tue, 26 Mar 2013 21:02:37 +0100
Subject: [PATCH] Add Function field type on product as some fields could
 depend on

review743003
---
 product.py | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/product.py b/product.py
index 4406b0da7f..8973b7259b 100644
--- a/product.py
+++ b/product.py
@@ -13,6 +13,11 @@ STATES = {
     'readonly': ~Eval('active', True),
     }
 DEPENDS = ['active']
+TYPES = [
+    ('goods', 'Goods'),
+    ('assets', 'Assets'),
+    ('service', 'Service'),
+    ]
 
 
 class Template(ModelSQL, ModelView):
@@ -20,11 +25,8 @@ class Template(ModelSQL, ModelView):
     __name__ = "product.template"
     name = fields.Char('Name', size=None, required=True, translate=True,
         select=True, states=STATES, depends=DEPENDS)
-    type = fields.Selection([
-            ('goods', 'Goods'),
-            ('assets', 'Assets'),
-            ('service', 'Service')
-            ], 'Type', required=True, states=STATES, depends=DEPENDS)
+    type = fields.Selection(TYPES, 'Type', required=True, states=STATES,
+        depends=DEPENDS)
     consumable = fields.Boolean('Consumable',
         states={
             'readonly': ~Eval('active', True),
@@ -123,6 +125,9 @@ class Product(ModelSQL, ModelView):
     active = fields.Boolean('Active', select=True)
     default_uom = fields.Function(fields.Many2One('product.uom',
             'Default UOM'), 'get_default_uom', searcher='search_default_uom')
+    type = fields.Function(fields.Selection(TYPES, 'Type',
+            on_change_with=['template']),
+        'on_change_with_type', searcher='search_type')
     list_price_uom = fields.Function(fields.Numeric('List Price',
         digits=(16, 4)), 'get_price_uom')
     cost_price_uom = fields.Function(fields.Numeric('Cost Price',
@@ -161,6 +166,14 @@ class Product(ModelSQL, ModelView):
     def search_default_uom(cls, name, clause):
         return [('template.default_uom',) + tuple(clause[1:])]
 
+    def on_change_with_type(self, name=None):
+        if self.template:
+            return self.template.type
+
+    @classmethod
+    def search_type(cls, name, clause):
+        return [('template.type',) + tuple(clause[1:])]
+
     @staticmethod
     def get_price_uom(products, name):
         Uom = Pool().get('product.uom')