refs #1915 "columns" is added as a special query parameter in ViewDat…

…aTable class which results in having to do this. Problem: Switching to treemap visualization in eg PageUrls will save a columns "nb_hits" param, loading the normal view and sorting by column will result in only displaying nb_hits column instead of all columns. This seems to be the only case. If there are more we need to let the visualization decide which parameters to reset in case of a visualization change

core/ViewDataTable/Manager.php

@@ -302,6 +302,13 @@ public static function saveViewDataTableParameters($login, $controllerAction, $p
         $params = self::getViewDataTableParameters($login, $controllerAction);
 
         foreach ($parametersToOverride as $key => $value) {
+            if ($key === 'viewDataTable'
+                && !empty($params['columns'])
+                && !empty($params[$key])
+                && $params[$key] !== $value) {
+                unset($params['columns']);
+            }
+
             $params[$key] = $value;
         }