From 9394c5691162f4e8bccbdcdcaa84f32026377d28 Mon Sep 17 00:00:00 2001
From: Guewen Baconnier <guewen.baconnier@camptocamp.com>
Date: Tue, 6 Mar 2018 21:36:38 +0100
Subject: [PATCH] web_tree_dynamic_colored_field: Parse field options with
 pyeval

The options in <field> attributes are parsed as python expressions:
https://github.com/odoo/odoo/blob/d18976d7489f6cd1c904f72557104807233a178d/addons/web/static/src/js/services/data_manager.js#L273

And the options in <button> are parsed as json...
https://github.com/odoo/odoo/blob/d18976d7489f6cd1c904f72557104807233a178d/addons/web/static/src/js/services/data_manager.js#L411

This code only support the <field> element because I'm not sure there
is a use for the <button> element.
---
 .../static/src/js/web_tree_dynamic_colored_field.js         | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/web_tree_dynamic_colored_field/static/src/js/web_tree_dynamic_colored_field.js b/web_tree_dynamic_colored_field/static/src/js/web_tree_dynamic_colored_field.js
index 2a96aba6..baa5616e 100644
--- a/web_tree_dynamic_colored_field/static/src/js/web_tree_dynamic_colored_field.js
+++ b/web_tree_dynamic_colored_field/static/src/js/web_tree_dynamic_colored_field.js
@@ -56,7 +56,11 @@ odoo.define('web_tree_dynamic_colored_field', function (require) {
             }
             // apply <field>'s own `options`
             if (!node.attrs.options) { return; }
-            var nodeOptions = JSON.parse(node.attrs.options);
+            if (node.tag !== 'field') { return; }
+            var nodeOptions = node.attrs.options;
+            if (!_.isObject(nodeOptions)) {
+                nodeOptions = pyeval.py_eval(nodeOptions);
+            }
             this.applyColorizeHelper($td, nodeOptions, node, 'fg_color', 'color', ctx);
             this.applyColorizeHelper($td, nodeOptions, node, 'bg_color', 'background-color', ctx);
         },