Bug Fixed New feature "storage mode" displaying 2 times, when enter to "threads/1/whoReadThread"

[toppy]

Hello.
@Fredxd, Thanks for new feature "storage mode". Thank you, that you are hearing your clients. You're cool!

I have found a little bug in new feature.
When option is:
"Who has read this thread - Mode" - All dates
User, that have rights to see common list, is displating 2 times, when entering to "threads/1/whoReadThread"

So, if I'm admin and I enter to thread
localhost/threads/1/
and than open popup:
localhost/threads/1/whoReadThread
I see, that I read this thread 2 times.

It's because entering to page localhost/threads/1/whoReadThread

If user don't have rights to see "whoReadThread" list, everything is okay and it displaying 1 time.

It's a little bug, unimportant, but it exists.

 

[Fred]

Ok thanks for report
I will investigate on it

 

[toppy]

Also suggestion on this topic. I don't want to "spam" and open new threads each time

If option is:
"Who has read this thread - Mode" - All dates
=> Don't write to log user's pagination in thread.

Now, if member open localhost/threads/1/ and than localhost/threads/1/page-2, addon mark this as 2 different log entries. In fact, it's one.

 

[Fred]

Also suggestion on this topic. I don't want to "spam" and open new threads each time

If option is:
"Who has read this thread - Mode" - All dates
=> Don't write to log user's pagination in thread.

Now, if member open localhost/threads/1/ and than localhost/threads/1/page-2, addon mark this as 2 different log entries. In fact, it's one.

I will see what i can do about this, but i am not sure it's possible.

 

[toppy]

Update. That bug from 1st post exists only if thread has more than 1 page.

 

[Fred]

Update. That bug from 1st post exists only if thread has more than 1 page.

Ok i note this
Maybe a solution can exist: Add in database entries every x minutes for this mode.
What do you think ?

 

[toppy]

Ok i note this
Maybe a solution can exist: Add in database entries every x minutes for this mode.
What do you think ?

Yes, it could be. Simply to check timespan and user id - I mean to add entry of one user in DB, if previous of this user was added more than 500 ms (for example) ago. And if less this x ms ago, not to write. But this don't solve pagination problem, for this this interval should be more than 500 ms

 

[Fred]

I think 500ms is very very short, but i can add an options in order to configure number in minutes.

 

[toppy]

I think 500ms is very very short, but i can add an options in order to configure number in minutes.

Thank you! I think, such option should be good. And with it to check last user ID, is it similar or not.

 

[Fred]

Hi
Added in 1.4.0
If you have any issue please create a new thread, and if you like this addon please let a rating on xenforo.com

 

[toppy]

Hi
Added in 1.4.0
If you have any issue please create a new thread, and if you like this addon please let a rating on xenforo.com

Thank you. Of course, I'll do it