[VB3 RC3] Who viewed this thread?

I know there is already one version of this by Gary W but I already made (and documented) my own so I thought I would post it.

Advantages of my hack:

My hack adds only 1 query to showthread, and it only queries the user table, so it is less server intensive. That is important to any busy board or anyone that wants to save some resources. It integrates with the "thread views" system to update the people that have seen the thread.

Disadvantages:

Well.. with mine even if you browse the forum as "invisible" you would end up showing in the "who read" list.


Info for hackers:
You may modify, improve, upgrade, redistribute this hack, include it
in another hack or yours or translate it provided you do it free of
charge and you distribute it in www.vbulletin.org at least, there is no
need to pm me asking for permission
Some portions of the code are (c) Jelsoft Enterprises Ltd.

[Kentaurus]

Originally Posted by Boofo

The sorting feature doesn't seem to be working for me. I have me (Boofo) in the list, the next one strats with an M and the next one strats with an A. They are not going alphabetically.

The sort will only work in new threads or threads where the list needs to be updated, if the who viewed is already stored for a thread then it won't be sorted. Call it an insertion sort It only sorts when something is added.

I'll check for that username markup, shouldn't be that hard to modify, but keep in mind that a query is needed for that. It is not an update query so it shouldn't hurt a lot the performance but still for people that are trying to mantain queries to a minimum (like me) an extra query might not sound practical.

[Kentaurus]

Here is the first one. For people that already installed the hack and want to add how many users have seen the thread:

In the showthread.php file, this:

$+++++ad = @implode(", ",$+++++adarray);

Replace it with:

$+++++ad = @implode(", ",$+++++adarray);
$total+++++ad = @count($+++++adarray);

In the SHOWTHREAD template, this:

Users that have seen this thread:

Replace it with:

$total+++++ad users have seen this thread:

Guests are a little more difficult because it means storing ip addreses as well instead of userid's. I'l check if there is an easy way to do it.

Those +++ are the word w h o r e that even if its a substring it is still being censored (/me makes a note on not using variable names that might be censored).

TXT file updated

[Boofo]

Is there any way to run a query to sort the users who have already seen a thread so it can strat off alphabetically from now on?

Can you just use the code in vBulletin for who is browsing this thread now to get the html markup that it is using already?

[squawell]

i got this error when i run scheduled task in my admincp

Warning: asort() expects parameter 1 to be array, boolean given in C:\Apache Group\Apache2\htdocs\forum\includes\cron\threadviews.php on line 41

the line 41 is

asort($+++++adarray);

any idea?thankz

edit:delete the line 41 than it works

[Kentaurus]

Originally Posted by squawell

i got this error when i run scheduled task in my admincp

Warning: asort() expects parameter 1 to be array, boolean given in C:\Apache Group\Apache2\htdocs\forum\includes\cron\threadviews.php on line 41

the line 41 is

asort($+++++adarray);

any idea?thankz

edit:delete the line 41 than it works

Don't worry about it, it's a warning (it doesn't stop the code from executing), that's because some threads might not have any data to sort. If you really don't want to see the warning change it to this:

if (is_array($+++++adarray)) {
asort($+++++adarray);
}

[Boofo]

Originally Posted by Kentaurus

Here is the first one. For people that already installed the hack and want to add how many users have seen the thread:

In the showthread.php file, this:

$+++++ad = @implode(", ",$+++++adarray);

Replace it with:

$+++++ad = @implode(", ",$+++++adarray);
$total+++++ad = @count($+++++adarray);

In the SHOWTHREAD template, this:

Users that have seen this thread:

Replace it with:

$total+++++ad users have seen this thread:

Guests are a little more difficult because it means storing ip addreses as well instead of userid's. I'l check if there is an easy way to do it.

Those +++ are the word w h o r e that even if its a substring it is still being censored (/me makes a note on not using variable names that might be censored).

TXT file updated

On guests, maybe just do a guest count?

[Kentaurus]

Here is another update

As requested by Boofo, to make the users appear with the html markup as they appear in the online users and almost anywhere, at the cost of 1 query.

I'm now attaching a txt since I'm getting tired of that censor.....

For threads before I added the sorting piece, just create a new user and view the unsorted thread, it will be sorted.

[Boofo]

That's Boofo, not Boofoo.

And thank you, sir.

[Boofo]

That worked great! Thank you, again.

You forgot to add the space after the comma in this line. The names were all running together. But I fixed it on my site.

PHP Code:

 $+++++ad = @implode(",", $+++++adnames); 


[Boofo]

Now I'm getting nothing but DB errors with this. I emailed you about it.