summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorbrain <brain@e03df62e-2008-0410-955e-edbf42e46eb7>2007-03-09 11:49:41 +0000
committerbrain <brain@e03df62e-2008-0410-955e-edbf42e46eb7>2007-03-09 11:49:41 +0000
commit166dfe4163f0941148f5927c28dfbdb8bdb49c5e (patch)
tree464bba36a8c277f26b655f6eb0c2068826a78f79 /src
parent4903848416ecc4c50f520b4a0f0a02dfc8bfad89 (diff)
/who tidyup - needs QA'ing
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@6641 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src')
-rw-r--r--src/cmd_who.cpp121
1 files changed, 38 insertions, 83 deletions
diff --git a/src/cmd_who.cpp b/src/cmd_who.cpp
index e6d6287ff..c4c37e528 100644
--- a/src/cmd_who.cpp
+++ b/src/cmd_who.cpp
@@ -6,7 +6,7 @@
* See: http://www.inspircd.org/wiki/index.php/Credits
*
* This program is free but copyrighted software; see
- * the file COPYING for details.
+ * the file COPYING for details.
*
* ---------------------------------------------------
*/
@@ -91,6 +91,35 @@ bool cmd_who::CanView(chanrec* chan, userrec* user)
return false;
}
+void cmd_who::SendWhoLine(userrec* user, const std::string &initial, chanrec* ch, userrec* u, std::vector<std::string> &whoresults)
+{
+ std::string lcn = getlastchanname(u);
+ chanrec* chlast = ServerInstance->FindChan(lcn);
+
+ std::string wholine = initial + (ch ? ch->name : lcn) + " " + u->ident + " " + (opt_showrealhost ? u->host : u->dhost) + " " +
+ ((*ServerInstance->Config->HideWhoisServer && !*user->oper) ? ServerInstance->Config->HideWhoisServer : u->server) +
+ " " + u->nick + " ";
+
+ /* away? */
+ if (u->awaymsg)
+ {
+ wholine.append("G");
+ }
+ else
+ {
+ wholine.append("H");
+ }
+
+ /* oper? */
+ if (u->oper)
+ {
+ wholine.append("*");
+ }
+
+ wholine = wholine + (ch ? ch->GetPrefixChar(u) : (chlast ? chlast->GetPrefixChar(u) : "")) + " :0 " + u->fullname;
+ whoresults.push_back(wholine);
+}
+
CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user)
{
/*
@@ -101,11 +130,11 @@ CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user)
*/
/* WHO options */
- bool opt_viewopersonly = false;
- bool opt_showrealhost = false;
- bool opt_unlimit = false;
- bool opt_realname = false;
- bool opt_mode = false;
+ opt_viewopersonly = false;
+ opt_showrealhost = false;
+ opt_unlimit = false;
+ opt_realname = false;
+ opt_mode = false;
chanrec *ch = NULL;
std::vector<std::string> whoresults;
@@ -165,31 +194,7 @@ CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user)
if (opt_viewopersonly && !*(i->second)->oper)
continue;
- /* XXX - code duplication; this could be more efficient -- w00t */
- std::string wholine = initial;
-
- wholine = wholine + ch->name + " " + i->second->ident + " " + (opt_showrealhost ? i->second->host : i->second->dhost) + " " +
- ((*ServerInstance->Config->HideWhoisServer && !*user->oper) ? ServerInstance->Config->HideWhoisServer : i->second->server) +
- " " + i->second->nick + " ";
-
- /* away? */
- if (*(i->second)->awaymsg)
- {
- wholine.append("G");
- }
- else
- {
- wholine.append("H");
- }
-
- /* oper? */
- if (*(i->second)->oper)
- {
- wholine.append("*");
- }
-
- wholine = wholine + ch->GetPrefixChar(i->second) + " :0 " + i->second->fullname;
- whoresults.push_back(wholine);
+ SendWhoLine(user, initial, ch, i->second, whoresults);
}
}
else
@@ -205,32 +210,7 @@ CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user)
if (whomatch(oper, matchtext, opt_realname, opt_showrealhost, opt_mode))
{
- std::string wholine = initial;
-
- wholine = wholine + getlastchanname(oper) + " " + oper->ident + " " + (opt_showrealhost ? oper->host : oper->dhost) + " " +
- ((*ServerInstance->Config->HideWhoisServer && !*user->oper) ? ServerInstance->Config->HideWhoisServer : oper->server) +
- " " + oper->nick + " ";
-
- ch = ServerInstance->FindChan(getlastchanname(oper));
-
- /* away? */
- if (*oper->awaymsg)
- {
- wholine.append("G");
- }
- else
- {
- wholine.append("H");
- }
-
- /* oper? */
- if (*oper->oper)
- {
- wholine.append("*");
- }
-
- wholine = wholine + (ch ? ch->GetPrefixChar(oper) : "") + " :0 " + oper->fullname;
- whoresults.push_back(wholine);
+ SendWhoLine(user, initial, NULL, oper, whoresults);
}
}
}
@@ -240,32 +220,7 @@ CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user)
{
if (whomatch(i->second, matchtext, opt_realname, opt_showrealhost, opt_mode))
{
- std::string wholine = initial;
-
- wholine = wholine + getlastchanname(i->second) + " " + i->second->ident + " " + (opt_showrealhost ? i->second->host : i->second->dhost) + " " +
- ((*ServerInstance->Config->HideWhoisServer && !*user->oper) ? ServerInstance->Config->HideWhoisServer : i->second->server) +
- " " + i->second->nick + " ";
-
- ch = ServerInstance->FindChan(getlastchanname(i->second));
-
- /* away? */
- if (*(i->second)->awaymsg)
- {
- wholine.append("G");
- }
- else
- {
- wholine.append("H");
- }
-
- /* oper? */
- if (*(i->second)->oper)
- {
- wholine.append("*");
- }
-
- wholine = wholine + (ch ? ch->GetPrefixChar(i->second) : "") + " :0 " + i->second->fullname;
- whoresults.push_back(wholine);
+ SendWhoLine(user, initial, NULL, i->second, whoresults);
}
}
}