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

The visit counter displays incorrect values after more than 100 visits #12034

Closed
sesom42 opened this issue Sep 12, 2017 · 3 comments
Closed

The visit counter displays incorrect values after more than 100 visits #12034

sesom42 opened this issue Sep 12, 2017 · 3 comments
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
3.1.1

Comments

@sesom42
Copy link

sesom42 commented Sep 12, 2017

Visitors with more than 100 visits will only be shown with 100 visits. Scrolling further down you can see that the visit counter becomes negative.

screenshot-piwik-2017-09-12-11-13-35

Piwik 3.1.0
@sgiehl
Copy link
Member

sgiehl commented Sep 13, 2017

Actually it shouldn't be possible to view more than 100 visits in the profile. Gonna fix that...

@sgiehl sgiehl added the Regression Indicates a feature used to work in a certain way but it no longer does even though it should. label Sep 13, 2017
@sgiehl sgiehl added this to the 3.2.0 milestone Sep 13, 2017
@sgiehl sgiehl closed this as completed in 9f861cc Sep 13, 2017
@sgiehl sgiehl mentioned this issue Sep 13, 2017
Closed
@sesom42
Copy link
Author

sesom42 commented Sep 13, 2017

Thank you. The Quickfix only fixes that no more than 101 entries are displayed. Nevertheless, the counters of visits are wrong. When opening the details, the top visit is always marked with # 100 although it can be much higher. At the end of the list, # 1 is always displayed twice, as shown in the above screenshot.

@sgiehl
Copy link
Member

sgiehl commented Sep 13, 2017

you shouldn't be able to load more visits after Visit 1 is displayed. so for a visitor with more than 100 visits it should display from 100 to 1.
All the details in the profile only summarizes the last 100 visits, even if there were more. Opened #12045 to make that configurable

@mattab mattab modified the milestones: 3.2.0, 3.1.1 Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
3.1.1
Development

No branches or pull requests
3 participants
@mattab @sgiehl @sesom42