Hehe, sure ;)
(In fact I didn't test it, I just looked at the code):
You are calling do_fetch(), which does work just fine for connectionless caches like APC, XCache, eAccelerator.

Memcache however does need a connection, and do_fetch() does not check if there is one - so you end up with an error (or a segfault in worst case).

So this is a bug in vbulletin, the memcache class should check/create a connection when do_fetch() is used. :)

As it happens, that code only runs if maxloggedin does not exist, which should never be the case on the forumhome, it was a backup in case the code was run on other pages (which used to be an option, but got removed in the 3.8 version). So even if you run memcache, this should actually always work.

In fact, I think I can take out the datastore fetch, as the reason for it being there no longer exists.

Sorry, I dont really understand what you are asking.

Nope, as do_fetch() is a "private" method that really only gets called from within the class - you are abusing it by calling it from the outside ;)

Then it should be set as private so it can't be used from outside the class.

Abusing it, lol, a bit extreme :)

I havent looked at that for a couple of years, since I first played with it, but yes, I presume fetch() should be used.

Build 13 - added the collapse option back.

If it was PHP 5 code - sure.
But it is PHP 4 code still.

Build 16 uploaded.

Various changes based on testing and learning of vb4.

Thanks Paul this is one of the many mods I use :)