Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#1464 closed Bug (fixed)

Yearly evolution doesn't show unique visitos

Reported by: paolobenve Owned by:
Priority: normal Milestone: Piwik 0.6.4
Component: Core Keywords:
Cc: Sensitive: no

Description

If you ask piwik for visitors evolution, daily, weekly and montly screen show unique visitors, while yearly doesn't.

That happens in the resume with graph widget.

Change History (6)

comment:1 Changed 4 years ago by vipsoft (robocoder)

  • Milestone set to 0 - Piwik 0.6.4

Please check your web server log file. If it says php was unable to allocate memory, then you'll have to increase your max memory_limit (php.ini).

Otherwise, post the error message here, so we can investigate further.

comment:2 Changed 4 years ago by SteveG (sgiehl)

Seems to be that this is wanted.

http://dev.piwik.org/trac/browser/trunk/plugins/VisitsSummary/Controller.php:

76	        if($view->period != 'year')
77	        {
78	            $view->urlSparklineNbUniqVisitors     = $this->getUrlSparkline( 'getEvolutionGraph', array('columns' => array('nb_uniq_visitors')));
79	            $view->nbUniqVisitors = $dataRow->getColumn('nb_uniq_visitors');
80	        }

I guess, it would make sense to display it for day only, 'cause the value is inaccurate for other periods.

comment:3 Changed 4 years ago by SteveG (sgiehl)

(In [2439]) refs #1464 remove year restriction

comment:4 Changed 4 years ago by SteveG (sgiehl)

Evolution graph should now be shown in every period.

Should we add a note to the text (if period is not day) explaining that this value is a sum of all daily values?

comment:5 Changed 4 years ago by matt (mattab)

  • Resolution set to fixed
  • Status changed from new to closed

I think it looks good like this :)

comment:6 Changed 4 years ago by SteveG (sgiehl)

(In [2448]) refs #1464 if-statement in template no longer required

Note: See TracTickets for help on using tickets.