From 313186e04569011e85162cb53af37fad2364e7ee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Wed, 20 Jan 2010 11:02:28 +0100
Subject: [PATCH] Use PySON

---
 work.py  | 41 +++++++++++++++++------------------------
 work.xml |  6 +++---
 2 files changed, 20 insertions(+), 27 deletions(-)

diff --git a/work.py b/work.py
index 86ac657997..d7c31a680b 100644
--- a/work.py
+++ b/work.py
@@ -3,6 +3,7 @@
 "Project"
 from trytond.model import ModelView, ModelSQL, fields
 from trytond.model.modelstorage import OPERATORS
+from trytond.pyson import Not, Bool, Eval, Get, Equal
 import copy
 
 
@@ -13,8 +14,8 @@ class TimesheetWork(ModelSQL, ModelView):
             'Timesheet Lines',
             depends=['timesheet_available', 'active'],
             states={
-                'invisible': 'not bool(timesheet_available)',
-                'readonly': '''not active'''
+                'invisible': Not(Bool(Eval('timesheet_available'))),
+                'readonly': Not(Bool(Eval('not active'))),
             })
     sequence = fields.Integer('Sequence')
 
@@ -23,16 +24,8 @@ class TimesheetWork(ModelSQL, ModelView):
         self._order.insert(0, ('sequence', 'ASC'))
 
         self.parent = copy.copy(self.parent)
-        if not self.parent.context:
-            self.parent.context = "{'type': locals().get('type')}"
-        elif 'type' not in self.parent.context:
-            sep = ''
-            if self.parent.context[-2] != ',':
-                sep = ','
-            self.parent.context = self.parent.context[:-1] + \
-                    sep + "'type': locals().get('type')" + \
-                    self.parent.context[-1:]
-
+        self.parent.context = copy.copy(self.parent.context)
+        self.parent.context['type'] = Eval('type')
         self._reset_columns()
 
 TimesheetWork()
@@ -52,20 +45,20 @@ class Work(ModelSQL, ModelView):
             ],
             'Type', required=True, select=1,
             states={
-                'invisible': 'context.get("type", False)',
+                'invisible': Get(Eval('context', {}), 'type', False),
             })
     party = fields.Many2One('party.party', 'Party',
             states={
-                'invisible': "type!= 'project'"
+                'invisible': Not(Equal(Eval('type'), 'project')),
             }, depends=['type'])
     party_address = fields.Many2One('party.address', 'Contact Address',
-            domain=["('party', '=', party)"],
+            domain=[('party', '=', Eval('party'))],
             states={
-                'invisible': "type != 'project'"
+                'invisible': Not(Equal(Eval('type'), 'project')),
             }, depends=['party', 'type'])
     effort = fields.Float("Effort",
             states={
-                'invisible': "type != 'task'"
+                'invisible': Not(Equal(Eval('type'), 'task')),
             }, depends=['type'], help="Estimated Effort for this work")
     total_effort = fields.Function('get_total_effort', type='float',
             string='Total Effort',
@@ -80,8 +73,8 @@ class Work(ModelSQL, ModelView):
             ('done', 'Done'),
             ], 'State',
             states={
-                'invisible': "type != 'task'",
-                'required': "type == 'task'",
+                'invisible': Not(Equal(Eval('type'), 'task')),
+                'required': Equal(Eval('type'), 'task'),
             }, select=1, depends=['type'])
 
     def default_type(self, cursor, user, context=None):
@@ -102,11 +95,11 @@ class Work(ModelSQL, ModelView):
         if 'company' in self._inherit_fields:
             company_field = copy.copy(
                 self._inherit_fields['company'])
-
-            company_field[2].states={
-                'invisible': 'context.get("type", False)',
-                'readonly': '''not active'''
-            }
+            company_field[2].states = copy.copy(company_field[2].states)
+            company_field[2].states.update({
+                'invisible': Bool(Get(Eval('context', {}), 'type')),
+                'readonly': Not(Bool(Eval('active'))),
+            })
 
             self._inherit_fields['company'] = company_field
 
diff --git a/work.xml b/work.xml
index a2697ca4b4..3640678a5b 100644
--- a/work.xml
+++ b/work.xml
@@ -92,7 +92,7 @@ this repository contains the full copyright notices and license terms. -->
                             <field name="comment" colspan="4"/>
                         </page>
                         <page string="Children" id="children"
-                              states="{'invisible': '''type != 'project' '''}">
+                              states="{'invisible': Not(Equal(Eval('type'), 'project'))}">
                             <field name="children" colspan="4">
                                 <tree string="Children">
                                   <field name="name"/>
@@ -104,7 +104,7 @@ this repository contains the full copyright notices and license terms. -->
                             </field>
                         </page>
                         <page string="Timesheet" id="timesheet_lines"
-                              states="{'invisible': '''not bool(timesheet_available) '''}">
+                              states="{'invisible': Not(Bool(Eval('timesheet_available')))}">
                             <field name="timesheet_lines" colspan="4"/>
                         </page>
                     </notebook>
@@ -196,7 +196,7 @@ this repository contains the full copyright notices and license terms. -->
             <field name="res_model">project.work</field>
             <field name="view_type">form</field>
             <field name="context">{'type': 'task'}</field>
-            <field name="domain">[('parent', '=', active_id), ('type', '=', 'task')]</field>
+            <field name="domain">[('parent', '=', Eval('active_id')), ('type', '=', 'task')]</field>
         </record>
         <record model="ir.action.keyword"
                 id="act_open_project_work_keyword1">