[IMP] graph_view now properly honors the 'interval' attribute of field names in the xml description (addon web_graph)

bzr revid: ged@openerp.com-20140124084222-dq1gpg1tg8rinozx

1 files changed, 9 insertions, 5 deletions

diff --git a/addons/web_graph/static/src/js/graph_view.js b/addons/web_graph/static/src/js/graph_view.js
index 39634ff103a..1122089f060 100644
--- a/addons/web_graph/static/src/js/graph_view.js
+++ b/addons/web_graph/static/src/js/graph_view.js
@@ -47,23 +47,27 @@ instance.web_graph.GraphView = instance.web.View.extend({
         };
 
         _.each(arch.children, function (field) {
+            var field_name = field.attrs.name;
+            if (_.has(field.attrs, 'interval')) {
+                field_name = field.attrs.name + ':' + field.attrs.interval;
+            } 
             if (_.has(field.attrs, 'type')) {
                 switch (field.attrs.type) {
                     case 'row':
-                        self.widget_config.row_groupby.push(field.attrs.name);
+                        self.widget_config.row_groupby.push(field_name);
                         break;
                     case 'col':
-                        self.widget_config.col_groupby.push(field.attrs.name);
+                        self.widget_config.col_groupby.push(field_name);
                         break;
                     case 'measure':
-                        self.widget_config.measures.push(field.attrs.name);
+                        self.widget_config.measures.push(field_name);
                         break;
                 }
             } else {  // old style, kept for backward compatibility
                 if ('operator' in field.attrs) {
-                    self.widget_config.measures.push(field.attrs.name);
+                    self.widget_config.measures.push(field_name);
                 } else {
-                    self.widget_config.row_groupby.push(field.attrs.name);
+                    self.widget_config.row_groupby.push(field_name);
                 }
             }
         });