From f3cf2f853a89c288cbd53a7bbbc4c83ea78aea9e Mon Sep 17 00:00:00 2001 From: Mokaddem Date: Mon, 1 Aug 2016 11:08:53 +0200 Subject: [PATCH] Fixed a bug in the hover of providers graph. When we click on a part in providers graphs, we hover an item, then we click on a random part not belonging to providers graphs, and we re-hover one part of the providers graphs, there was an undefined variable (The random click overrided it). --- var/www/static/js/moduleTrending.js | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/var/www/static/js/moduleTrending.js b/var/www/static/js/moduleTrending.js index 68ea7e21..ff32cc1b 100644 --- a/var/www/static/js/moduleTrending.js +++ b/var/www/static/js/moduleTrending.js @@ -220,12 +220,14 @@ function plot_top_graph(module_name, init){ plot = $.plot($(chartID), [barData], barOptions); /* rememeber the data for the two graph */ - if (plot_data_old.length>1){ // avoid adding plot_data for previous clicked pie part - plot_data_old = []; - plot_old = []; + if ((module_name == "size") || (module_name == "num")) { // Add only for the provider graph + if (plot_data_old.length>1){ // avoid adding plot_data for previous clicked pie part + plot_data_old = []; + plot_old = []; + } + plot_data_old.push(plot.getData()); + plot_old.push(plot); } - plot_data_old.push(plot.getData()); - plot_old.push(plot); if (can_bind){ // avoid binding two listener for provider graphs $("#flot-bar-chart-"+module_name).unbind( "plothover.customHandler" );