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

Jump to bottom

Remove 'nb_uniq_visitors' from all API calls when period != day as the data is not correct #640

Closed
mattab opened this issue Apr 3, 2009 · 3 comments
Closed

Remove 'nb_uniq_visitors' from all API calls when period != day as the data is not correct #640

mattab opened this issue Apr 3, 2009 · 3 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone
RobotRock

Comments

@mattab
Copy link
Member

mattab commented Apr 3, 2009

currently if you request search engines or keywords for a week, the ‘nb_uniq_visitors’ in the response is not the number of unique visitors for this keyword or search engine. It is the sum of the nb_uniq_visitors for each day in the week.

It will be renamed to ‘sum_daily_nb_uniq_visitors’ in all API calls.
It also affects some fields in Actions.get*

This ticket also mean that we should not show ‘Unique Visitors’ columns in any table when the period is not ‘day’.
Note that it is customizable on a plugin basis. Another plugin could run the query on the full set of logs to extract the exact number of unique visitors for each keyword/search engine/website/resolution/etc. but that would be very resource intensive.

We still have plans to process the Number of Unique Visitors accurately for any period, see #5428
@mattab
Copy link
Member Author

mattab commented Apr 3, 2009

Actions.get\* are affected as follows:

```
‘nb_uniq_visitors’ => ‘sum_daily_nb_uniq_visitors’,
‘entry_nb_uniq_visitors’ => ‘sum_daily_entry_nb_uniq_visitors’,
‘exit_nb_uniq_visitors’ => ‘sum_daily_exit_nb_uniq_visitors’,
```

@mattab
Copy link
Member Author

mattab commented Apr 3, 2009

(In 1046) – changing ways of applying filters to a datatable, now $table→filter(‘Piwik_DataTable_Filter_Sort’, array (‘nb_visits’, ‘desc’));
- refs #640 now accurately reporting nb_uniq_visitors in all reports
- adding tests

@mattab
Copy link
Member Author

mattab commented Apr 7, 2009

(In 1050) – finishes fixes #640 now unique visitors column won’t show for periods in reports and API responses

@mattab mattab added this to the RobotRock milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Milestone
RobotRock
Development

No branches or pull requests
1 participant
@mattab