Skip to content

Commit

Permalink
Refs matomo-org#4077, use proper table ID when returning treemap data…
Browse files Browse the repository at this point in the history
… by AJAX.
  • Loading branch information
Benaka Moorthi committed Sep 20, 2013
1 parent f722945 commit 035204a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
5 changes: 4 additions & 1 deletion plugins/TreemapVisualization/TreemapDataGenerator.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
*/
namespace Piwik\Plugins\TreemapVisualization;

use Piwik\Common;
use Piwik\DataTable;
use Piwik\DataTable\Map;
use Piwik\DataTable\Filter\CalculateEvolutionFilter;
Expand Down Expand Up @@ -157,8 +158,10 @@ public function generate($dataTable)
$dataTable->filter('Truncate', array($this->truncateAfter));
}

$tableId = Common::getRequestVar('idSubtable', '');

$root = $this->makeNode('treemap-root', $this->rootName);
$this->addDataTableToNode($root, $dataTable, $pastData, $tableId = '', $this->firstRowOffset);
$this->addDataTableToNode($root, $dataTable, $pastData, $tableId, $this->firstRowOffset);
return $root;
}

Expand Down
3 changes: 0 additions & 3 deletions plugins/TreemapVisualization/javascripts/treemapViz.js
Original file line number Diff line number Diff line change
Expand Up @@ -434,9 +434,6 @@
apiMethod: this.param.module + '.' + this.param.action, // TODO: will this work for all subtables?
format: 'json',
column: this.param.columns,
truncateAfter: this.props.max_graph_elements - 1,
filter_limit: -1,
expanded: 1,
show_evolution_values: this.props.show_evolution_values || 0
});

Expand Down

0 comments on commit 035204a

Please sign in to comment.