Make sure evolution parameters are initialized in Config so Controllers can override them. (#14386)

2 files changed, 7 insertions, 3 deletions

diff --git a/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution.php b/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution.php
index 143f841adb..ae8e260bc1 100644
--- a/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution.php
+++ b/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution.php
@@ -85,9 +85,6 @@ class Evolution extends JqplotGraph
         $originalDate = Common::getRequestVar('date', 'last' . $defaultLastN, 'string');
 
         if ('range' != $period) { // show evolution limit if the period is not a range
-            $this->config->show_limit_control = true;
-            $this->config->show_periods = true;
-
             // set the evolution_{$period}_last_n query param
             if (Range::parseDateRange($originalDate)) {
                 // if a multiple period
diff --git a/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution/Config.php b/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution/Config.php
index d1482959a4..8b3745a390 100644
--- a/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution/Config.php
+++ b/plugins/CoreVisualizations/Visualizations/JqplotGraph/Evolution/Config.php
@@ -9,6 +9,7 @@
 
 namespace Piwik\Plugins\CoreVisualizations\Visualizations\JqplotGraph\Evolution;
 
+use Piwik\Common;
 use Piwik\Plugins\CoreVisualizations\Visualizations\JqplotGraph\Config as JqplotGraphConfig;
 
 /**
@@ -36,6 +37,12 @@ class Config extends JqplotGraphConfig
 
         $this->addPropertiesThatShouldBeAvailableClientSide(array('show_line_graph'));
         $this->addPropertiesThatCanBeOverwrittenByQueryParams(array('show_line_graph'));
+
+        $period = Common::getRequestVar('period');
+        if ($period !== 'range') {
+            $this->show_limit_control = true;
+            $this->show_periods = true;
+        }
     }
 
 }