@hpvd opened this Issue on August 5th 2015

In the visitor profile you can see the visits of a visitor.
These visits are numbered from top to bottom.

Shouldn't it be from the logical pow the other way around,
so that the first visit has number 1 and every new visits gets the following number?

In addition beside the logic problem, the number of the visits in the past changes, so one could not talk about e.g. the visit with number 4 because when there is a new visit these number changes...

What do you think?

Please see attachment
2015-08-05_16h29_52

@tsteur commented on August 6th 2015 Owner

:+1:

@Flux3PO commented on August 28th 2015

I can confirm, I do see this as an issue as well (or at least confusing)

@tsteur commented on October 27th 2015 Owner

It's not as trivial as we could change change it to 4, 3, 2, 1 but it might be wrong if there were eg 50 visits. Then it should be rather "Visit 50, 49, 48, 47, ...".

There might be an issue if there were more than 100 visits since we apparently only fetch 100 visits. In this case we would possible show "Visit 100, visit 99, visit 98" which might be actually a higher visit.

To be confirmed

@hpvd commented on October 28th 2015

yes sure. 4,3,2,1 was only an example. Should take all "known" visits into account.

Regarding only fetching 100 visits, one maybe can think of a two step approach:
1) make it working <100 visits within this ticket (which is already a lot)
2) and open a new ticket for making it working fine for >100 visits

in 1) one may show instead of numbering the visits for every shown visit "99+"

@hpvd commented on October 28th 2015

btw: are you sure there is a 100 visit limit? One could select show to 500 visits please see attachment.
Or is the 100 a limit for visits of the same visitor?
2015-10-28_06h59_28

@Glisse1 commented on October 28th 2015

Yes, the numbering is wrong, or at least confusing as stated. First visit should be the first visit, not the last. Opening a visitor profile in live widget should list me the visitor with 14th visit today as visit #14, not #1

@tsteur commented on November 16th 2015 Owner

btw: are you sure there is a 100 visit limit? One could select show to 500 visits please see attachment.
Or is the 100 a limit for visits of the same visitor?

There is actually a limit of 10 visits in the visitor profile. The limit selector is for the visitor log, not for the visitor profile. This means we'd need to fetch all visits initially to show a correct number which will be most likely slow. I tried to implement it but as soon as one clicks on the "load more" link it is a bit confusing even when showing 10+. If it was 100 we could maybe do it as there might be not so many users having 100+ visits but 10+ visits happens pretty often.

I think the only solution will be to actually fetch the number of visits for a certain user. There is an index on idsite, idvisitor so it should be pretty fast actually.

@mattab commented on July 10th 2017 Owner

A few users have reported this recently so increasing priority

This Issue was closed on September 4th 2017
Powered by GitHub Issue Mirror