From 1de1cdb000d76c2632449fe4b8799270cd6c35a2 Mon Sep 17 00:00:00 2001 From: Thomas Steur Date: Mon, 19 May 2014 08:25:10 +0200 Subject: refs #1915 "columns" is added as a special query parameter in ViewDataTable 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 | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'core/ViewDataTable') diff --git a/core/ViewDataTable/Manager.php b/core/ViewDataTable/Manager.php index 858b208742..82ffff118d 100644 --- a/core/ViewDataTable/Manager.php +++ b/core/ViewDataTable/Manager.php @@ -302,6 +302,13 @@ class Manager $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; } -- cgit v1.2.3