Kunal Chavda (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-bug-906783-kch into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
  Bug #906783 in OpenERP Web: "Pie chart should be display"
  https://bugs.launchpad.net/openerp-web/+bug/906783

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-906783-kch/+merge/86361

Hello,
I have coded for showing pie chart. 

Thanks,
Kunal Chavda
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-906783-kch/+merge/86361
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-906783-kch.
=== modified file 'addons/web_graph/static/src/js/graph.js'
--- addons/web_graph/static/src/js/graph.js	2011-12-16 13:00:00 +0000
+++ addons/web_graph/static/src/js/graph.js	2011-12-20 10:09:50 +0000
@@ -173,7 +173,7 @@
         graph_data = _(graph_data).sortBy(function (point) {
             return point[self.abscissa] + '[[--]]' + point[self.group_field];
         });
-        if (this.chart == 'bar' || 'line' || 'area') {
+        if (_.include(['bar','line','area'],this.chart)) {
             return this.schedule_bar_line_area(graph_data);
         } else if (this.chart == "pie") {
             return this.schedule_pie(graph_data);

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to