From 5fc7311f1a6bea35701d4aa7c2a68ee1f3ad1a2e Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Mon, 11 Apr 2011 14:54:50 +0200
Subject: [PATCH] [ADD] row-wise colors

bzr revid: xmo@openerp.com-20110411125450-3h0ay8fskd1s6nz5
---
 addons/base/controllers/main.py     | 34 +++++++++++++++++++++++++----
 addons/base/static/src/xml/base.xml | 13 +++++++----
 addons/base/test/test_view.py       | 33 ++++++++++++++++++++++++++++
 3 files changed, 72 insertions(+), 8 deletions(-)

diff --git a/addons/base/controllers/main.py b/addons/base/controllers/main.py
index 36e9b0feaee..4d75b063a35 100644
--- a/addons/base/controllers/main.py
+++ b/addons/base/controllers/main.py
@@ -566,10 +566,36 @@ class ListView(View):
         rows = DataSet().do_search_read(request, model,
                                         offset=offset, limit=limit,
                                         domain=domain)
-        # rows pipe
-        #   process_attrs
-        #   process_colors
-        return rows
+        eval_context = request.session.evaluation_context(
+            request.context)
+        return [
+            {'data': row,
+             'attrs': self.process_attrs(view, row, eval_context),
+             'color': self.process_colors(view, row, eval_context)}
+            for row in rows
+        ]
+
+    def process_attrs(self, view, row, context):
+        pass
+
+    def process_colors(self, view, row, context):
+        colors = view['arch']['attrs'].get('colors')
+
+        if not colors:
+            return None
+
+        color = [
+            pair.split(':')[0]
+            for pair in colors.split(';')
+            if eval(pair.split(':')[1], dict(context, **row))
+        ]
+
+        if not color:
+            return None
+        elif len(color) == 1:
+            return color[0]
+        return 'maroon'
+
 
 class SearchView(View):
     _cp_path = "/base/searchview"
diff --git a/addons/base/static/src/xml/base.xml b/addons/base/static/src/xml/base.xml
index caf463cf4e0..8a71b8ebc7a 100644
--- a/addons/base/static/src/xml/base.xml
+++ b/addons/base/static/src/xml/base.xml
@@ -197,9 +197,14 @@
     </thead>
 </t>
 <t t-name="ListView.rows" t-foreach="rows" t-as="row">
-    <t t-call="ListView.row"/>
+    <t t-call="ListView.row">
+        <t t-set="style" t-value="null"/>
+        <t-if test="row.color">
+            <t t-set="style" t-value="'color: ' + row.color"/>
+        </t-if>
+    </t>
 </t>
-<tr t-name="ListView.row">
+<tr t-name="ListView.row" t-att-style="style">
     <th t-if="options.selectable" class="oe-record-selector">
         <input type="checkbox"/>
     </th>
@@ -208,8 +213,8 @@
         <td t-if="column.invisible !== '1'" t-att-title="column.help">
             <t t-set="is_button" t-value="column.tag === 'button'"/>
             <!-- TODO: get correct widget from form -->
-            <t t-if="!is_button and row[column.id]">
-                <t t-esc="row[column.id]"/>
+            <t t-if="!is_button and row['data'][column.id]">
+                <t t-esc="row['data'][column.id]"/>
             </t>
             <button type="button" t-att-title="column.help"
                     t-if="is_button">
diff --git a/addons/base/test/test_view.py b/addons/base/test/test_view.py
index 141bbd5e0db..0070ea11cf4 100644
--- a/addons/base/test/test_view.py
+++ b/addons/base/test/test_view.py
@@ -205,3 +205,36 @@ class ListViewTest(unittest2.TestCase):
 
         self.assertEqual(view['arch']['attrs']['editable'],
                          'bottom')
+
+    def test_color_nocolor(self):
+        self.assertEqual(
+            self.view.process_colors(
+                {'arch': {'attrs': {}, 'children': []}}, {}, {}),
+            None)
+    def test_color_literal(self):
+        self.assertEqual(
+            self.view.process_colors(
+                {'arch': {'attrs': {'colors': 'black:1'}}, 'children': []},
+                {}, {}),
+            'black')
+    def test_color_miss(self):
+        self.assertEqual(
+            self.view.process_colors(
+                {'arch': {'attrs': {'colors': "grey:state in ('cancelled','done');blue:state in ('pending')"}},
+                 'children': []
+                }, {'state': 'open'}, {}),
+            None)
+    def test_color_compute(self):
+        self.assertEqual(
+            self.view.process_colors(
+                {'arch': {'attrs': {'colors': "grey:state in ('cancelled','done');blue:state in ('pending')"}},
+                 'children': []
+                }, {'state': 'done'}, {}),
+            'grey')
+    def test_color_multiple(self):
+        self.assertEqual(
+            self.view.process_colors(
+                {'arch': {'attrs': {'colors': "grey:state in ('cancelled','done');blue:state in ('done')"}},
+                 'children': []
+                }, {'state': 'done'}, {}),
+            'maroon')