if (ServerInstance->Parser->LoopCall(user, this, parameters, pcnt, 0))
return CMD_SUCCESS;
- dest = ServerInstance->FindNick(parameters[0]);
+
+ /*
+ * If 2 paramters are specified (/whois nick nick), ignore the first one like spanningtree
+ * does, and use the second one, otherwise, use the only paramter. -- djGrrr
+ */
+ if (pcnt > 1)
+ {
+ dest = ServerInstance->FindNick(parameters[1]);
+ }
+ else
+ {
+ dest = ServerInstance->FindNick(parameters[0]);
+ }
if (dest)
{
/*
- * Determine whether to show idletime. We show it if hidewhois is turned off.
- * spanningtree deals with 2 param whois, and remote users, so this is all we need to do.
- *
- * The assumption here (not a huge one) is that cmd_whois is only ever invoked
- * remotely! Keep that in mind! -- w00t
+ * Okay. Umpteenth attempt at doing this, so let's re-comment...
+ * For local users (/w localuser), we show idletime if hidewhois is disabled
+ * For local users (/w localuser localuser), we always show idletime, hence pcnt > 1 check.
+ * For remote users (/w remoteuser), we do NOT show idletime
+ * For remote users (/w remoteuser remoteuser), spanningtree will handle calling do_whois, so we can ignore this case.
+ * Thanks to djGrrr for not being impatient while I have a crap day coding. :p -- w00t
*/
- if (!*ServerInstance->Config->HideWhoisServer)
+ if (IS_LOCAL(dest) && (!*ServerInstance->Config->HideWhoisServer || pcnt > 1))
{
- /* this really is safe, we're only called for local users .. */
idle = abs((dest->idle_lastmsg)-ServerInstance->Time());
signon = dest->signon;
}