Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yearly evolution doesn't show unique visitos #1464

Closed
anonymous-matomo-user opened this issue Jul 6, 2010 · 6 comments
Closed

Yearly evolution doesn't show unique visitos #1464

anonymous-matomo-user opened this issue Jul 6, 2010 · 6 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@anonymous-matomo-user
Copy link

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.

@robocoder
Copy link
Contributor

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.

@sgiehl
Copy link
Member

sgiehl commented Jul 6, 2010

Seems to be that this is wanted.

https://github.com/piwik/piwik/blob/master/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.

@sgiehl
Copy link
Member

sgiehl commented Jul 6, 2010

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

@sgiehl
Copy link
Member

sgiehl commented Jul 6, 2010

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?

@mattab
Copy link
Member

mattab commented Jul 6, 2010

I think it looks good like this :)

@sgiehl
Copy link
Member

sgiehl commented Jul 7, 2010

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

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

4 participants