modules/m_list.c: Change operspy LIST syntax to match everything else that accepts operspy.

This commit is contained in:
Keith Buck 2011-05-22 20:30:49 -07:00
parent a35e76aded
commit bf0a45920d
2 changed files with 32 additions and 25 deletions

View file

@ -12,4 +12,5 @@ mode !#channel - Gives the full modes of a channel including any keys.
chantrace !#channel - Gives full output despite not being on channel. chantrace !#channel - Gives full output despite not being on channel.
masktrace !nick!user@host :gecos - Lists matching users on all servers. masktrace !nick!user@host :gecos - Lists matching users on all servers.
topic !#channel - Gives full output despite not being on channel. topic !#channel - Gives full output despite not being on channel.
list ![,options...] - Lists all channels, including secret channels. list !#channel - Gives full output despite not being on channel.
list ![options...] - Lists all channels, including secret channels.

View file

@ -63,7 +63,7 @@ static void safelist_client_release(struct Client *);
static void safelist_one_channel(struct Client *source_p, struct Channel *chptr); static void safelist_one_channel(struct Client *source_p, struct Channel *chptr);
static void safelist_iterate_client(struct Client *source_p); static void safelist_iterate_client(struct Client *source_p);
static void safelist_iterate_clients(void *unused); static void safelist_iterate_clients(void *unused);
static void safelist_channel_named(struct Client *source_p, const char *name); static void safelist_channel_named(struct Client *source_p, const char *name, int operspy);
struct Message list_msgtab = { struct Message list_msgtab = {
"LIST", 0, 0, 0, MFLG_SLOW, "LIST", 0, 0, 0, MFLG_SLOW,
@ -157,8 +157,10 @@ static int m_list(struct Client *client_p, struct Client *source_p, int parc, co
static int mo_list(struct Client *client_p, struct Client *source_p, int parc, const char *parv[]) static int mo_list(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
{ {
struct ListClient *params; struct ListClient *params;
char *p, *args; char *p;
char *args = NULL;
int i; int i;
int operspy = 0;
if (source_p->localClient->safelist_data != NULL) if (source_p->localClient->safelist_data != NULL)
{ {
@ -167,10 +169,22 @@ static int mo_list(struct Client *client_p, struct Client *source_p, int parc, c
return 0; return 0;
} }
/* Single channel. */ if (parc > 1)
if (parc > 1 && IsChannelName(parv[1]))
{ {
safelist_channel_named(source_p, parv[1]); args = LOCAL_COPY(parv[1]);
}
if (args && *args == '!' && IsOperSpy(source_p))
{
args++;
report_operspy(source_p, "LIST", args);
operspy = 1;
}
/* Single channel. */
if (args && IsChannelName(args))
{
safelist_channel_named(source_p, args, operspy);
return 0; return 0;
} }
@ -180,14 +194,12 @@ static int mo_list(struct Client *client_p, struct Client *source_p, int parc, c
/* XXX rather arbitrary -- jilles */ /* XXX rather arbitrary -- jilles */
params->users_min = 3; params->users_min = 3;
params->users_max = INT_MAX; params->users_max = INT_MAX;
params->operspy = 0; params->operspy = operspy;
params->created_min = params->topic_min = params->created_min = params->topic_min =
params->created_max = params->topic_max = 0; params->created_max = params->topic_max = 0;
if (parc > 1 && !EmptyString(parv[1])) if (args && !EmptyString(args))
{ {
args = LOCAL_COPY(parv[1]);
/* Cancel out default minimum. */ /* Cancel out default minimum. */
params->users_min = 0; params->users_min = 0;
@ -260,12 +272,6 @@ static int mo_list(struct Client *client_p, struct Client *source_p, int parc, c
} }
} }
} }
/* Only accept operspy as the first option. */
else if (*args == '!' && IsOperSpy(source_p) && i == 0)
{
params->operspy = 1;
report_operspy(source_p, "LIST", p);
}
if (EmptyString(p)) if (EmptyString(p))
break; break;
@ -353,39 +359,39 @@ static void safelist_client_release(struct Client *client_p)
/* /*
* safelist_channel_named() * safelist_channel_named()
* *
* inputs - client pointer, channel name * inputs - client pointer, channel name, operspy
* outputs - none * outputs - none
* side effects - a named channel is listed * side effects - a named channel is listed
*/ */
static void safelist_channel_named(struct Client *source_p, const char *name) static void safelist_channel_named(struct Client *source_p, const char *name, int operspy)
{ {
struct Channel *chptr; struct Channel *chptr;
char *p; char *p;
char *n = LOCAL_COPY(name);
sendto_one(source_p, form_str(RPL_LISTSTART), me.name, source_p->name); sendto_one(source_p, form_str(RPL_LISTSTART), me.name, source_p->name);
if ((p = strchr(n, ','))) if ((p = strchr(name, ',')))
*p = '\0'; *p = '\0';
if (*n == '\0') if (*name == '\0')
{ {
sendto_one_numeric(source_p, ERR_NOSUCHNICK, form_str(ERR_NOSUCHNICK), name); sendto_one_numeric(source_p, ERR_NOSUCHNICK, form_str(ERR_NOSUCHNICK), name);
sendto_one(source_p, form_str(RPL_LISTEND), me.name, source_p->name); sendto_one(source_p, form_str(RPL_LISTEND), me.name, source_p->name);
return; return;
} }
chptr = find_channel(n); chptr = find_channel(name);
if (chptr == NULL) if (chptr == NULL)
{ {
sendto_one_numeric(source_p, ERR_NOSUCHNICK, form_str(ERR_NOSUCHNICK), n); sendto_one_numeric(source_p, ERR_NOSUCHNICK, form_str(ERR_NOSUCHNICK), name);
sendto_one(source_p, form_str(RPL_LISTEND), me.name, source_p->name); sendto_one(source_p, form_str(RPL_LISTEND), me.name, source_p->name);
return; return;
} }
if (!SecretChannel(chptr) || IsMember(source_p, chptr)) if (!SecretChannel(chptr) || IsMember(source_p, chptr) || operspy)
sendto_one(source_p, form_str(RPL_LIST), me.name, source_p->name, "", sendto_one(source_p, form_str(RPL_LIST), me.name, source_p->name,
(operspy && SecretChannel(chptr)) ? "!" : "",
chptr->chname, rb_dlink_list_length(&chptr->members), chptr->chname, rb_dlink_list_length(&chptr->members),
chptr->topic == NULL ? "" : chptr->topic); chptr->topic == NULL ? "" : chptr->topic);