Changed period in row evolution breaks graph/metric association (#14423)

* Changed period in row evolution breaks graph/metric association

fix #14208

worked for me, also with multi row evolution. Only known issue I found is if requesting eg first yearly row evolution which may have the `users` metric hidden, then switching to `day` period, and then the `users` metric is maybe supposed to become available but didn't. That should be quite edge case though.

* add class whether row is hidden

* Fixing couple regressions visible in ui tests.

* fix build + update comment

* tweak

* test tweak

1 files changed, 6 insertions, 1 deletions

diff --git a/plugins/CoreVisualizations/Visualizations/Graph.php b/plugins/CoreVisualizations/Visualizations/Graph.php
index 1dbdfc824e..28ffab7768 100644
--- a/plugins/CoreVisualizations/Visualizations/Graph.php
+++ b/plugins/CoreVisualizations/Visualizations/Graph.php
@@ -201,7 +201,12 @@ abstract class Graph extends Visualization
         $columnsToDisplay = $this->removeLabelFromArray($columnsToDisplay);
 
         // Strip out any columns_to_display that are not in the dataset
-        $allColumns = $this->getDataTable()->getColumns();
+        $allColumns = [];
+        if ($this->report) {
+            $allColumns = $this->report->getAllMetrics();
+        }
+        $allColumns = array_merge($allColumns, $this->getDataTable()->getColumns());
+        $allColumns = array_unique($allColumns);
 
         // If the datatable has no data, use the default columns (there must be data for evolution graphs or else nothing displays)
         if (empty($allColumns)) {