Unless you want something like a Settings preference, or a per-user preference, you can just edit the code of the extension. On line 31 it says $s->AddWhere("u", "DateLastActive", "", "DATE_SUB(NOW(), INTERVAL 5 MINUTE)", ">=", NULL, NULL, 0);
Just change the 5 to the amount of minutes it should check for, or change 5 MINUTE into 1 DAY to get a whole day.

What determines the order of the Who's Online list? It's not alphabetized and I'm always on top of my list.

Any chance of someone packing all the enhancements/bugfixes up and making an unofficial release? Or better still, just getting it updated on the addons site :)

Ok to make whos online appear on pages that use postbackaction (ie PageMaker addon) you need to change the big if statement to this:
if ((in_array($Context->SelfUrl, array("categories.php", "index.php", "post.php", "comments.php")) && !isset($_GET['PageID']) || ($Context->SelfUrl == "index.php" && isset($_GET['PostBackAction'])))
&& $Context->Session->UserID > 0 && !$Context->Session->User->Preference("HideWhosOnline") ) {
To make it show count when not signed in, add this code to the end of the file (outside all other methods / ifs):
if ((in_array($Context->SelfUrl, array("categories.php", "index.php", "comments.php")) && !isset($_GET['PageID']) )
&& $Context->Session->UserID == 0 ) {
$ListName = $Context->GetDefinition("Who's Online");
$Panel->AddList($ListName,501);
$online_list = $WhosOnline->GetWhosOnline();
$guest_count = $WhosOnline->GetGuestCount();
$phantom_count = 0;
$user_count = 0;
if ($online_list) {
foreach ($online_list as $name) {
if ($name["Phantom"]) {
$phantom_count++;
}
if (!$name["Phantom"] || !isset($name["Phantom"]) || $Context->Session->User->Permission("PERMISSION_WHOS_PHANTOM")) {
$user_count++;
}
}
if ($user_count > 0) {
$user_string = "$user_count registered user";
if ($user_count > 1)
$user_string .= "s";
$Panel->AddListItem($ListName,'','',$user_string);
}
if ($phantom_count > 0) {
$phantom_string = "$phantom_count phantom user";
if ($phantom_count > 1)
$phantom_string .= "s";
$Panel->AddListItem($ListName,'','',$phantom_string);
}
if ($guest_count > 0) {
$guest_string = "$guest_count guest";
if ($guest_count > 1)
$guest_string .= "s";
$Panel->AddListItem($ListName,'','',$guest_string);
}
} else {
$Panel->AddListItem($ListName,'','',$Context->Dictionary['None online']);
}
}

With an option for the inline names display please ;)

Just pop it on your own webspace and leave a link in here, then a mod can update the official one if they want. If you don't have spac eyou can pop it on, just email me and I'll host it.

Most recent activity I think. Everyone will probably be at the top of their own list.

Hm. I had someone ask me why they are always at the bottom. The list order doesn't seem to change, for example, I'm always on top and, if Silver is online, she's always second, and deebee is always at the bottom. So if I've got 15 people online the order will never change with those 15.

add this on line 29 (in the function whoisonline, after the $s->AddSelect [..]) $s->AddOrderBy("DateLastActive",'i','DESC');
and get back to me if that doesnt work.

I get the following error when viewing site from another computer and no one is online:

Notice: Undefined index: None online in .../forums/extensions/WhosOnline/default.php on line 237

I had this issue, and if I remember well, the dictionary definition is missing. Try adding this to your language file:
$Context->Dictionary["None online"] = "Nobody is online.";

Hi guys,

This plugin is giving us serious overhead issues. With the plugin disabled system load looks like this:

10:23am up 226 days 0:47, 1 user, load average: 0.05, 0.27, 0.23

With it enabled system load looks like this:

10:23am up 226 days 0:07, 1 user, load average: 65.32, 45.22, 23.01

In fact load increases exponentially until the site grinds to a halt and we have to restart the database process.

We obvioulsy want to use the plugin, but can't in its current form. Has anyone else experienced this issue? And does anyone have any suggestions for reducing the plugins database requirement?

We recieve around 2 million impressions a month btw.

Thanks for this. We have a developer looking at amending the plugin on this basis at the moment. We'll publish the source code here once complete.

Hello all,

We've now finished updating this plugin - you can see results (when signed in) here: http://andymurray.com/community/

I'll post the update shortly.

airlock, any chance of getting the update posted somewhere so we can spread the love throughout the Vanilla community? :)

I'm sure i posted here....could this be a timezone issue?

mr do:
the fix you suggested for ordering by most recently active user does not work for me.

$s->AddOrderBy("DateLastActive",'i','DESC');

i put it on line 29 in the function GetWhosOnline() but i got an undefined index on 'i'.

perfect. thank you.

Wouldn't using Ajax add unnecessary strain to the server?

You could do it without AJAX, just by wrapping older names in a class (eg. new, old, older) and then adding a little CSS.

Props to Mr Do, thanks for the updated version.

Any update on this airlock? I'm having issues with this putting a strain on our server; any chance we could have a look at your version?

I'd like to use this add-on (my users request this function), but the latest version from Mr Do gives me 404 and airlock silents.

Any news, please?

two things.

first, there's a fairly serious sql-injection problem with some of the queries in this extension. the contents of the IpHistoryID cookie are not being escaped before being used in a database query.

here's a quick-fix diff:

--- WhosOnline-2007-03-14/WhosOnline/default.php    2007-03-14 16:58:58.000000000 -0700
+++ WhosOnline-2007-03-14-fixed/WhosOnline/default.php  2008-04-09 09:04:34.000000000 -0700
@@ -103,7 +103,7 @@

     if (isset($_COOKIE['IpHistoryID'])) {
       $s->SetMainTable("IpHistory", "i");
-      $s->AddWhere("i", "IpHistoryID", "", $_COOKIE['IpHistoryID'], "=");
+      $s->AddWhere("i", "IpHistoryID", "", FormatStringForDatabaseInput($_COOKIE['IpHistoryID']), "=");
       $this->Context->Database->Delete($s, $this->Name, "UpdateDateLastActive", "An error occurred while deleting guest pr
ofile");
       setcookie('IpHistoryID', '', time() - 3600);
     }
@@ -120,7 +120,7 @@
       $s->Clear();
       $s->SetMainTable("IpHistory", "i");
       $s->AddFieldNameValue("DateLogged", "now()", 0);
-      $s->AddWhere("i", "IpHistoryID", "", $_COOKIE['IpHistoryID'], "=");
+      $s->AddWhere("i", "IpHistoryID", "", FormatStringForDatabaseInput($_COOKIE['IpHistoryID']), "=");
       $result = $this->Context->Database->Update($s, $this->Name, "UpdateGuestLastActive", "An error occurred while loggin
g user data.");
     } else {
       $my_ip = GetRemoteIp(1);

second, if you want to order by date last active (most recently active at the top), add the line:

    $s->AddOrderBy("DateLastActive", "u", "desc");

after the AddWhere statement on line 31.

That's part of the diff output. It means at line 120 take out the next 7 lines and replace them with the new 7.

It solves the possible SQL injection stuff he was talking about.