Much thanks to djGrrr for being helpful. Give this a go.

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@6964 e03df62e-2008-0410-955e-edbf42e46eb7

diff --git a/src/cmd_whois.cpp b/src/cmd_whois.cpp
index 5945a1002e4d495acc9e79bed6f4ef61f895a356..85c4c904ab0b95f1d083f9e72577582904ddadfd 100644 (file)
--- a/src/cmd_whois.cpp
+++ b/src/cmd_whois.cpp
@@ -107,13 +107,14 @@ CmdResult cmd_whois::Handle (const char** parameters, int pcnt, userrec *user)
        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 .. */
                        if (IS_LOCAL(dest))