From 3d4abe3815193cc954b719d6e4cafafd5c30c980 Mon Sep 17 00:00:00 2001 From: w00t Date: Thu, 10 May 2007 18:38:30 +0000 Subject: 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 --- src/cmd_whois.cpp | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/src/cmd_whois.cpp b/src/cmd_whois.cpp index 5945a1002..85c4c904a 100644 --- 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)) -- cgit v1.2.3