Nice, although you should be using vBulletin's database class ;)

It works really well untill you have several hundered thousands users online and you need to scan the user table every time someone access the forumhome.

This should be flagged for poor programming.

I'm trying to help ktp with his code but after writing additional code to it I thought "what the hell am I going to do with this?".
I thought I could limit it so it would only check once a week but don't know how to go about caching the data until the query ran again.

Use a vB cron and have it run once an hour, day, whatever, and have it update another row possibly in the datastore so it can be pulled on demand.

I wish I had that problem. :cross-eyed:

So is this one coded equally as bad? : https://vborg.vbsupport.ru/showthread.php?t=105546

They're the exact same thing. So to answer oyur question, yes, it is equally bad if not worse.

D'oh. http://wildpitchforums.com/images/smilies/banghead.gif

I informed Princeton and told him that the hack on this thread and the hack on another thread are coded exactly the same way. The only difference between these two is that this one fetches 5 rows and the other gets 10.

As people already stated, This hack is horribly coded which means the other one is equally if not worse. I say it's worse because it has to get 10 rows as opposed to 5.
You may also experience conflict if you installed the two hacks or had another other plugin in "forum_start" hook where you use $test as your output variable.
What I mean by that is that you use $test on the "forumhome" template.

I tried to privately deal with this to lessen the chances of hurting someone's creditably but I guess I'm not important enough for someone to read my private message.