diff --git a/core/API/DataTableGenericFilter.php b/core/API/DataTableGenericFilter.php
index a87f3305b0244a6df18705d0d6ec8c887c3642f6..7a88cb417502fd0bc7887d0125bf609aa3553a9a 100644
--- a/core/API/DataTableGenericFilter.php
+++ b/core/API/DataTableGenericFilter.php
@@ -12,6 +12,7 @@ use Exception;
use Piwik\Common;
use Piwik\DataTable\Filter\AddColumnsProcessedMetricsGoal;
use Piwik\DataTable;
+use Piwik\Plugin\ProcessedMetric;
use Piwik\Plugin\Report;
class DataTableGenericFilter
@@ -169,7 +170,7 @@ class DataTableGenericFilter
return $filterApplied;
}
- public function areProcessedMetricsNeededFor(Report $report)
+ public function areProcessedMetricsNeededFor($metrics)
{
$columnQueryParameters = array(
'filter_column',
@@ -181,7 +182,7 @@ class DataTableGenericFilter
foreach ($columnQueryParameters as $queryParamName) {
$queryParamValue = Common::getRequestVar($queryParamName, false, $type = null, $this->request);
if (!empty($queryParamValue)
- && $report->hasProcessedMetric($queryParamValue)
+ && $this->containsProcessedMetric($metrics, $queryParamValue)
) {
return true;
}
@@ -189,4 +190,21 @@ class DataTableGenericFilter
return false;
}
+
+ /**
+ * @param ProcessedMetric[] $metrics
+ * @param string $name
+ * @return bool
+ */
+ private function containsProcessedMetric($metrics, $name)
+ {
+ foreach ($metrics as $metric) {
+ if ($metric instanceof ProcessedMetric
+ && $metric->getName() == $name
+ ) {
+ return true;
+ }
+ }
+ return false;
+ }
}
\ No newline at end of file
diff --git a/core/API/DataTablePostProcessor.php b/core/API/DataTablePostProcessor.php
index 6b80082f69679166d362b7ca0774f7f3fb979733..e6bab6aacac520b36a388e88d72fb1bf2551ddd7 100644
--- a/core/API/DataTablePostProcessor.php
+++ b/core/API/DataTablePostProcessor.php
@@ -28,6 +28,38 @@ class DataTablePostProcessor
const PROCESSED_METRICS_FORMATTED_FLAG = 'processed_metrics_formatted';
const PROCESSED_METRICS_COMPUTED_FLAG = 'processed_metrics_computed';
+ /**
+ * @var null|Report
+ */
+ private $report;
+
+ /**
+ * @var string[]
+ */
+ private $request;
+
+ /**
+ * @var string
+ */
+ private $apiModule;
+
+ /**
+ * @var string
+ */
+ private $apiMethod;
+
+ /**
+ * Constructor.
+ */
+ public function __construct($apiModule, $apiMethod, $request)
+ {
+ $this->apiModule = $apiModule;
+ $this->apiMethod = $apiMethod;
+ $this->request = $request;
+
+ $this->report = Report::factory($apiModule, $apiMethod);
+ }
+
/**
* Apply post-processing logic to a DataTable of a report for an API request.
*
@@ -38,41 +70,39 @@ class DataTablePostProcessor
* @param bool $applyFormatting Whether to format processed metrics or not.
* @return DataTableInterface A new data table.
*/
- public function process(DataTableInterface $dataTable, $report, $request, $applyFormatting = true)
+ public function process(DataTableInterface $dataTable, $applyFormatting = true)
{
- $label = self::getLabelFromRequest($request);
+ $label = self::getLabelFromRequest($this->request);
- $dataTable = $this->applyPivotByFilter($dataTable, $report, $request);
- $dataTable = $this->applyFlattener($dataTable, $report, $request);
- $dataTable = $this->applyTotalsCalculator($dataTable, $report, $request);
- $dataTable = $this->applyGenericFilters($label, $dataTable, $report, $request);
+ $dataTable = $this->applyPivotByFilter($dataTable);
+ $dataTable = $this->applyFlattener($dataTable);
+ $dataTable = $this->applyTotalsCalculator($dataTable);
+ $dataTable = $this->applyGenericFilters($label, $dataTable);
- $dataTable->filter(array($this, 'computeProcessedMetrics'), array($report));
+ $dataTable->filter(array($this, 'computeProcessedMetrics'));
// we automatically safe decode all datatable labels (against xss)
$dataTable->queueFilter('SafeDecodeLabel');
- $dataTable = $this->applyQueuedFilters($dataTable, $request);
- $dataTable = $this->applyRequestedColumnDeletion($dataTable, $request);
- $dataTable = $this->applyLabelFilter($label, $dataTable, $report, $request);
- $dataTable = $this->applyProcessedMetricsFormatting($dataTable, $report, $applyFormatting);
+ $dataTable = $this->applyQueuedFilters($dataTable);
+ $dataTable = $this->applyRequestedColumnDeletion($dataTable);
+ $dataTable = $this->applyLabelFilter($label, $dataTable);
+ $dataTable = $this->applyProcessedMetricsFormatting($dataTable, $applyFormatting);
return $dataTable;
}
/**
* @param DataTableInterface $dataTable
- * @param Report|null $report
- * @param $request
* @return DataTableInterface
*/
- private function applyPivotByFilter(DataTableInterface $dataTable, $report, $request)
+ private function applyPivotByFilter(DataTableInterface $dataTable)
{
- $pivotBy = Common::getRequestVar('pivotBy', false, 'string', $request);
+ $pivotBy = Common::getRequestVar('pivotBy', false, 'string', $this->request);
if (!empty($pivotBy)) {
- $reportId = $report->getModule() . '.' . $report->getAction();
- $pivotByColumn = Common::getRequestVar('pivotByColumn', false, 'string', $request);
- $pivotByColumnLimit = Common::getRequestVar('pivotByColumnLimit', false, 'int', $request);
+ $reportId = $this->apiModule . '.' . $this->apiMethod;
+ $pivotByColumn = Common::getRequestVar('pivotByColumn', false, 'string', $this->request);
+ $pivotByColumnLimit = Common::getRequestVar('pivotByColumnLimit', false, 'int', $this->request);
$dataTable->filter('PivotByDimension', array($reportId, $pivotBy, $pivotByColumn, $pivotByColumnLimit,
PivotByDimension::isSegmentFetchingEnabledInConfig()));
@@ -82,15 +112,13 @@ class DataTablePostProcessor
/**
* @param DataTableInterface $dataTable
- * @param Report|null $report
- * @param $request
* @return DataTable|DataTableInterface|DataTable\Map
*/
- private function applyFlattener($dataTable, $report, $request)
+ private function applyFlattener($dataTable)
{
- if (Common::getRequestVar('flat', '0', 'string', $request) == '1') {
- $flattener = new Flattener($report->getModule(), $report->getAction(), $request);
- if (Common::getRequestVar('include_aggregate_rows', '0', 'string', $request) == '1') {
+ if (Common::getRequestVar('flat', '0', 'string', $this->request) == '1') {
+ $flattener = new Flattener($this->apiModule, $this->apiMethod, $this->request);
+ if (Common::getRequestVar('include_aggregate_rows', '0', 'string', $this->request) == '1') {
$flattener->includeAggregateRows();
}
$dataTable = $flattener->flatten($dataTable);
@@ -100,14 +128,12 @@ class DataTablePostProcessor
/**
* @param DataTableInterface $dataTable
- * @param Report|null $report
- * @param $request
* @return DataTableInterface
*/
- private function applyTotalsCalculator($dataTable, $report, $request)
+ private function applyTotalsCalculator($dataTable)
{
- if (1 == Common::getRequestVar('totals', '1', 'integer', $request)) {
- $reportTotalsCalculator = new ReportTotalsCalculator($report->getModule(), $report->getAction(), $request);
+ if (1 == Common::getRequestVar('totals', '1', 'integer', $this->request)) {
+ $reportTotalsCalculator = new ReportTotalsCalculator($this->apiModule, $this->apiMethod, $this->request);
$dataTable = $reportTotalsCalculator->calculate($dataTable);
}
return $dataTable;
@@ -116,23 +142,21 @@ class DataTablePostProcessor
/**
* @param string $label
* @param DataTableInterface $dataTable
- * @param Report|null $report
- * @param $request
* @return DataTableInterface
*/
- private function applyGenericFilters($label, $dataTable, $report, $request)
+ private function applyGenericFilters($label, $dataTable)
{
// if the flag disable_generic_filters is defined we skip the generic filters
- if (0 == Common::getRequestVar('disable_generic_filters', '0', 'string', $request)) {
- $self = $this;
-
- $genericFilter = new DataTableGenericFilter($request, $report);
+ if (0 == Common::getRequestVar('disable_generic_filters', '0', 'string', $this->request)) {
+ $genericFilter = new DataTableGenericFilter($this->request);
- if ($genericFilter->areProcessedMetricsNeededFor($report)) {
- $dataTable->filter(function (DataTable $table) use ($self, $report) {
- $self->computeProcessedMetrics($table, $report);
- });
- }
+ $self = $this;
+ $report = $this->report;
+ $dataTable->filter(function (DataTable $table) use ($genericFilter, $report, $self) {
+ if ($genericFilter->areProcessedMetricsNeededFor($this->getProcessedMetricsFor($table, $this->report))) {
+ $self->computeProcessedMetrics($table);
+ }
+ });
if (!empty($label)) {
$genericFilter->disableFilters(array('Limit', 'Truncate'));
@@ -146,13 +170,12 @@ class DataTablePostProcessor
/**
* @param DataTableInterface $dataTable
- * @param $request
* @return DataTableInterface
*/
- private function applyQueuedFilters($dataTable, $request)
+ private function applyQueuedFilters($dataTable)
{
// if the flag disable_queued_filters is defined we skip the filters that were queued
- if (Common::getRequestVar('disable_queued_filters', 0, 'int', $request) == 0) {
+ if (Common::getRequestVar('disable_queued_filters', 0, 'int', $this->request) == 0) {
$dataTable->applyQueuedFilters();
}
return $dataTable;
@@ -160,19 +183,18 @@ class DataTablePostProcessor
/**
* @param DataTableInterface $dataTable
- * @param $request
* @return DataTableInterface
*/
- private function applyRequestedColumnDeletion($dataTable, $request)
+ private function applyRequestedColumnDeletion($dataTable)
{
// use the ColumnDelete filter if hideColumns/showColumns is provided (must be done
// after queued filters are run so processed metrics can be removed, too)
- $hideColumns = Common::getRequestVar('hideColumns', '', 'string', $request);
- $showColumns = Common::getRequestVar('showColumns', '', 'string', $request);
+ $hideColumns = Common::getRequestVar('hideColumns', '', 'string', $this->request);
+ $showColumns = Common::getRequestVar('showColumns', '', 'string', $this->request);
if (empty($showColumns)) {
// if 'columns' is used, we remove all temporary metrics by showing only the columns specified in
// 'columns'
- $showColumns = Common::getRequestVar('columns', '', 'string', $request);
+ $showColumns = Common::getRequestVar('columns', '', 'string', $this->request);
}
if (!empty($hideColumns)
@@ -187,16 +209,15 @@ class DataTablePostProcessor
/**
* @param string $label
* @param DataTableInterface $dataTable
- * @param Report $report
* @return DataTableInterface
*/
- private function applyLabelFilter($label, $dataTable, $report, $request)
+ private function applyLabelFilter($label, $dataTable)
{
// apply label filter: only return rows matching the label parameter (more than one if more than one label)
if (!empty($label)) {
- $addLabelIndex = Common::getRequestVar('labelFilterAddLabelIndex', 0, 'int', $request) == 1;
+ $addLabelIndex = Common::getRequestVar('labelFilterAddLabelIndex', 0, 'int', $this->request) == 1;
- $filter = new LabelFilter($report->getModule(), $report->getAction(), $request);
+ $filter = new LabelFilter($this->apiModule, $this->apiMethod, $this->request);
$dataTable = $filter->filter($label, $dataTable, $addLabelIndex);
}
return $dataTable;
@@ -204,17 +225,15 @@ class DataTablePostProcessor
/**
* @param DataTableInterface $dataTable
- * @param Report $report
* @return DataTableInterface
*/
- private function applyProcessedMetricsFormatting($dataTable, $report, $applyFormatting)
+ private function applyProcessedMetricsFormatting($dataTable, $applyFormatting)
{
if ($applyFormatting) {
- $dataTable->filter(array($this, 'formatProcessedMetrics'), array($report));
+ $dataTable->filter(array($this, 'formatProcessedMetrics'));
} else {
- $dataTable->queueFilter(array($this, 'formatProcessedMetrics'), array($report)); // TODO: queuing does not always work.
+ $dataTable->queueFilter(array($this, 'formatProcessedMetrics')); // TODO: queuing does not always work.
}
-
return $dataTable;
}
@@ -248,7 +267,7 @@ class DataTablePostProcessor
return $label;
}
- private function computeProcessedMetrics(DataTable $dataTable, $report)
+ public function computeProcessedMetrics(DataTable $dataTable)
{
if ($dataTable->getMetadata(self::PROCESSED_METRICS_COMPUTED_FLAG)) {
return;
@@ -256,13 +275,13 @@ class DataTablePostProcessor
$dataTable->setMetadata(self::PROCESSED_METRICS_COMPUTED_FLAG, true);
- $processedMetrics = $this->getProcessedMetricsFor($dataTable, $report);
+ $processedMetrics = $this->getProcessedMetricsFor($dataTable, $this->report);
if (empty($processedMetrics)) {
return;
}
foreach ($processedMetrics as $name => $processedMetric) {
- if (!$processedMetric->beforeCompute($this, $dataTable)) {
+ if (!$processedMetric->beforeCompute($this->report, $dataTable)) {
continue;
}
@@ -272,7 +291,7 @@ class DataTablePostProcessor
$subtable = $row->getSubtable();
if (!empty($subtable)) {
- $this->computeProcessedMetrics($subtable, $report);
+ $this->computeProcessedMetrics($subtable, $this->report);
}
}
}
@@ -282,7 +301,7 @@ class DataTablePostProcessor
/**
* public for use as callback.
*/
- public function formatProcessedMetrics(DataTable $dataTable, $report)
+ public function formatProcessedMetrics(DataTable $dataTable)
{
if ($dataTable->getMetadata(self::PROCESSED_METRICS_FORMATTED_FLAG)) {
return;
@@ -290,7 +309,7 @@ class DataTablePostProcessor
$dataTable->setMetadata(self::PROCESSED_METRICS_FORMATTED_FLAG, true);
- $processedMetrics = $this->getProcessedMetricsFor($dataTable, $report);
+ $processedMetrics = $this->getProcessedMetricsFor($dataTable, $this->report);
if (empty($processedMetrics)) {
return;
}
@@ -304,7 +323,7 @@ class DataTablePostProcessor
$subtable = $row->getSubtable();
if (!empty($subtable)) {
- $this->formatProcessedMetrics($subtable, $report);
+ $this->formatProcessedMetrics($subtable, $this->report);
}
}
}
@@ -320,7 +339,7 @@ class DataTablePostProcessor
$processedMetrics = $dataTable->getMetadata(DataTable::EXTRA_PROCESSED_METRICS_METADATA_NAME) ?: array();
if (!empty($report)) {
- $processedMetrics = array_merge($processedMetrics, $report->processedMetrics ?: array());
+ $processedMetrics = array_merge($processedMetrics, $report->getProcessedMetricsById());
}
$result = array();
diff --git a/core/API/ResponseBuilder.php b/core/API/ResponseBuilder.php
index dbf79b1ec8a306894edf4a1e6e0a654e54b24a5d..97c27693db153b8ead4930a50dac9d7ead95e488 100644
--- a/core/API/ResponseBuilder.php
+++ b/core/API/ResponseBuilder.php
@@ -29,8 +29,6 @@ class ResponseBuilder
private $apiModule = false;
private $apiMethod = false;
- private $postProcessor;
-
/**
* @param string $outputFormat
* @param array $request
@@ -40,7 +38,6 @@ class ResponseBuilder
$this->outputFormat = $outputFormat;
$this->request = $request;
$this->apiRenderer = ApiRenderer::factory($outputFormat, $request);
- $this->postProcessor = new DataTablePostProcessor();
}
public function disableSendHeader()
@@ -167,10 +164,10 @@ class ResponseBuilder
private function handleDataTable(DataTableInterface $datatable)
{
- $report = Report::factory($this->apiModule, $this->apiMethod);
- $applyFormatting = !($this->apiRenderer instanceof Original);
+ $applyFormatting = Common::getRequestVar('force_format_processed_metrics', !($this->apiRenderer instanceof Original), null, $this->request) == 1;
- $datatable = $this->postProcessor->process($datatable, $report, $this->request, $applyFormatting);
+ $postProcessor = new DataTablePostProcessor($this->apiModule, $this->apiMethod, $this->request);
+ $datatable = $postProcessor->process($datatable, $applyFormatting);
return $this->apiRenderer->renderDataTable($datatable);
}
diff --git a/core/Plugin/ProcessedMetric.php b/core/Plugin/ProcessedMetric.php
index b2d563f633e352973bdc1a6d16f42a1d17ef0963..1d1e1eaae4f88121a5c69ee4f911cd233241e90b 100644
--- a/core/Plugin/ProcessedMetric.php
+++ b/core/Plugin/ProcessedMetric.php
@@ -50,7 +50,7 @@ abstract class ProcessedMetric extends Metric
* @return bool Return `true` to compute the metric for the table, `false` to skip computing
* this metric.
*/
- public function beforeCompute(Report $report, DataTable $table)
+ public function beforeCompute($report, DataTable $table)
{
return true;
}
diff --git a/core/Plugin/Report.php b/core/Plugin/Report.php
index 11b50fc3efd2305b2768cce9bcfe9afc99cf8cfa..e83832bb39ad775a6c46246bd0eeddd1e308fd63 100644
--- a/core/Plugin/Report.php
+++ b/core/Plugin/Report.php
@@ -705,28 +705,6 @@ class Report
return Request::processRequest($module . '.' . $action, $paramOverride);
}
- /**
- * Returns true if this report has a processed metric with the `$name` name.
- *
- * Will only search through {@link Piwik\Plugin\ProcessedMetric} instances, so string entries
- * in {@link $processedMetrics} will be ignored.
- *
- * @param string $name
- * @return bool
- */
- public function hasProcessedMetric($name)
- {
- $processedMetrics = $this->processedMetrics ?: array(); // TODO: shouldn't allow processedMetrics to be non-array
- foreach ($processedMetrics as $metric) {
- if ($metric instanceof ProcessedMetric
- && $metric->getName() == $name
- ) {
- return true;
- }
- }
- return false;
- }
-
/**
* Get an instance of a specific report belonging to the given module and having the given action.
* @param string $module
@@ -831,12 +809,16 @@ class Report
}
/**
+ * Returns an array mapping the ProcessedMetrics served by this report by their string names.
+ *
* @return ProcessedMetric[]
*/
- private function getProcessedMetricsById()
+ public function getProcessedMetricsById()
{
+ $processedMetrics = $this->processedMetrics ?: array(); // TODO: shouldn't allow processedMetrics to be non-array
+
$result = array();
- foreach ($this->processedMetrics as $processedMetric) {
+ foreach ($processedMetrics as $processedMetric) {
if ($processedMetric instanceof ProcessedMetric) { // instanceof check for backwards compatibility
$result[$processedMetric->getName()] = $processedMetric;
}
diff --git a/plugins/API/ProcessedReport.php b/plugins/API/ProcessedReport.php
index 2b72e07849820082f34941c372f3bc9487d827f9..61c34c3378ef2ba39bddb56cc874b8a42e36c7e9 100644
--- a/plugins/API/ProcessedReport.php
+++ b/plugins/API/ProcessedReport.php
@@ -390,7 +390,8 @@ class ProcessedReport
'format' => 'original',
'serialize' => '0',
'language' => $language,
- 'idSubtable' => $idSubtable
+ 'idSubtable' => $idSubtable,
+ 'force_format_processed_metrics' => 1,
));
if (!empty($segment)) $parameters['segment'] = $segment;
@@ -411,8 +412,6 @@ class ProcessedReport
throw new Exception("API returned an error: " . $e->getMessage() . " at " . basename($e->getFile()) . ":" . $e->getLine() . "\n");
}
- $dataTable->filter(array(Report::factory($apiModule, $apiAction), 'formatProcessedMetrics'));
-
list($newReport, $columns, $rowsMetadata, $totals) = $this->handleTableReport($idSite, $dataTable, $reportMetadata, $showRawMetrics);
foreach ($columns as &$name) {
diff --git a/plugins/Actions/Metrics/AveragePageGenerationTime.php b/plugins/Actions/Metrics/AveragePageGenerationTime.php
index f3081af35fc980b65486a5ef3ba57082edc6a092..3b0399bb04dfea0724a23ce317f64b935891f177 100644
--- a/plugins/Actions/Metrics/AveragePageGenerationTime.php
+++ b/plugins/Actions/Metrics/AveragePageGenerationTime.php
@@ -43,7 +43,7 @@ class AveragePageGenerationTime extends ProcessedMetric
return Piwik::getQuotientSafe($sumGenerationTime, $hitsWithTimeGeneration, $precision = 3);
}
- public function beforeCompute(Report $report, DataTable $table)
+ public function beforeCompute($report, DataTable $table)
{
$columnName = Metrics::INDEX_PAGE_SUM_TIME_GENERATION;
diff --git a/plugins/MultiSites/API.php b/plugins/MultiSites/API.php
index 1357271e55f13887165a5d48cdf90cc3283b5b21..1851fe4397f3db266c7709d88405e3609307a665 100755
--- a/plugins/MultiSites/API.php
+++ b/plugins/MultiSites/API.php
@@ -240,7 +240,6 @@ class API extends \Piwik\Plugin\API
// use past data to calculate evolution percentages
$this->calculateEvolutionPercentages($dataTable, $pastData, $apiMetrics);
- Common::destroy($pastData);
}
// remove eCommerce related metrics on non eCommerce Piwik sites
diff --git a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageTitle_day.xml b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageTitle_day.xml
index 629847ae61a78cdcae847453f46bdccdce130a17..5f687c3a3fac76164bc01a2ab2b26d8778415ef3 100644
--- a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageTitle_day.xml
+++ b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageTitle_day.xml
@@ -6,6 +6,7 @@
<nb_uniq_visitors>1</nb_uniq_visitors>
<nb_hits>1</nb_hits>
<sum_time_spent>0</sum_time_spent>
+ <sum_time_generation>0.134</sum_time_generation>
<nb_hits_with_time_generation>1</nb_hits_with_time_generation>
<min_time_generation>0.134</min_time_generation>
<max_time_generation>0.134</max_time_generation>
diff --git a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageUrl_day.xml b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageUrl_day.xml
index 0880f587504f6a4c2531175472545a28f57bfb84..dda8223114bc0395434e89ba0bfa91e2d58e3a3c 100644
--- a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageUrl_day.xml
+++ b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__Actions.getPageUrl_day.xml
@@ -6,6 +6,7 @@
<nb_uniq_visitors>1</nb_uniq_visitors>
<nb_hits>1</nb_hits>
<sum_time_spent>0</sum_time_spent>
+ <sum_time_generation>0.023</sum_time_generation>
<nb_hits_with_time_generation>1</nb_hits_with_time_generation>
<min_time_generation>0.023</min_time_generation>
<max_time_generation>0.023</max_time_generation>
diff --git a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__subtable__API.getProcessedReport_week.xml b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__subtable__API.getProcessedReport_week.xml
index 72793292356494d3bdc6b42406146218ea07a51e..1c2d3f6a74de896d9a986909e502955d45123c61 100644
--- a/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__subtable__API.getProcessedReport_week.xml
+++ b/tests/PHPUnit/System/expected/test_OneVisitorTwoVisits__subtable__API.getProcessedReport_week.xml
@@ -12,19 +12,21 @@
<metrics>
<nb_hits>Pageviews</nb_hits>
<nb_visits>Unique Pageviews</nb_visits>
- <bounce_rate>Bounce Rate</bounce_rate>
- <avg_time_on_page>Avg. time on page</avg_time_on_page>
- <exit_rate>Exit rate</exit_rate>
- <avg_time_generation>Avg. generation time</avg_time_generation>
</metrics>
<metricsDocumentation>
<nb_hits>The number of times this page was visited.</nb_hits>
<nb_visits>The number of visits that included this page. If a page was viewed multiple times during one visit, it is only counted once.</nb_visits>
- <bounce_rate>The percentage of visits that started on this page and left the website straight away.</bounce_rate>
<avg_time_on_page>The average amount of time visitors spent on this page (only the page, not the entire website).</avg_time_on_page>
+ <bounce_rate>The percentage of visits that started on this page and left the website straight away.</bounce_rate>
<exit_rate>The percentage of visits that left the website after viewing this page.</exit_rate>
<avg_time_generation>The average time it took to generate the page. This metric includes the time it took the server to generate the web page, plus the time it took for the visitor to download the response from the server. A lower 'Avg. generation time' means a faster website for your visitors!</avg_time_generation>
</metricsDocumentation>
+ <processedMetrics>
+ <avg_time_on_page>Avg. time on page</avg_time_on_page>
+ <bounce_rate>Bounce Rate</bounce_rate>
+ <exit_rate>Exit rate</exit_rate>
+ <avg_time_generation>Avg. generation time</avg_time_generation>
+ </processedMetrics>
<actionToLoadSubTables>getPageUrls</actionToLoadSubTables>
<imageGraphUrl>index.php?module=API&method=ImageGraph.get&idSite=1&apiModule=Actions&apiAction=getPageUrls&period=week&date=2010-03-06&idSubtable=</imageGraphUrl>
<imageGraphEvolutionUrl>index.php?module=API&method=ImageGraph.get&idSite=1&apiModule=Actions&apiAction=getPageUrls&period=week&date=2009-08-10,2010-03-07&idSubtable=</imageGraphEvolutionUrl>
@@ -34,8 +36,8 @@
<label>Page URL</label>
<nb_hits>Pageviews</nb_hits>
<nb_visits>Unique Pageviews</nb_visits>
- <bounce_rate>Bounce Rate</bounce_rate>
<avg_time_on_page>Avg. time on page</avg_time_on_page>
+ <bounce_rate>Bounce Rate</bounce_rate>
<exit_rate>Exit rate</exit_rate>
<avg_time_generation>Avg. generation time</avg_time_generation>
</columns>
@@ -44,8 +46,8 @@
<label>/purchase.htm</label>
<nb_visits>1</nb_visits>
<nb_hits>1</nb_hits>
- <avg_time_on_page>00:00:00</avg_time_on_page>
<bounce_rate>100%</bounce_rate>
+ <avg_time_on_page>00:00:00</avg_time_on_page>
<exit_rate>100%</exit_rate>
<avg_time_generation>0.13s</avg_time_generation>
</row>