X-Git-Url: https://jfr.im/git/solanum.git/blobdiff_plain/633531a4aac5b70e13a428361517bcf80c41f5b2..460e7932744e7d48795875270a1a610a6a8dfa31:/modules/m_topic.c

diff --git a/modules/m_topic.c b/modules/m_topic.c
index f6a79fc4..214e4a94 100644
--- a/modules/m_topic.c
+++ b/modules/m_topic.c
@@ -20,8 +20,6 @@
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
  *  USA
- *
- *  $Id: m_topic.c 254 2005-09-21 23:35:12Z nenolod $
  */
 
 #include "stdinc.h"
@@ -43,24 +41,27 @@
 #include "logger.h"
 #include "inline/stringops.h"
 
-static int m_topic(struct Client *, struct Client *, int, const char **);
-static int ms_topic(struct Client *, struct Client *, int, const char **);
+static const char topic_desc[] =
+	"Provides the TOPIC command to set, remove, and inspect channel topics";
+
+static void m_topic(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void ms_topic(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 struct Message topic_msgtab = {
-	"TOPIC", 0, 0, 0, MFLG_SLOW,
+	"TOPIC", 0, 0, 0, 0,
 	{mg_unreg, {m_topic, 2}, {m_topic, 2}, {ms_topic, 5}, mg_ignore, {m_topic, 2}}
 };
 
 mapi_clist_av1 topic_clist[] = { &topic_msgtab, NULL };
-DECLARE_MODULE_AV1(topic, NULL, NULL, topic_clist, NULL, NULL, "$Revision: 254 $");
+DECLARE_MODULE_AV2(topic, NULL, NULL, topic_clist, NULL, NULL, NULL, NULL, topic_desc);
 
 /*
  * m_topic
  *      parv[1] = channel name
  *	parv[2] = new topic, if setting topic
  */
-static int
-m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
+static void
+m_topic(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
 	struct Channel *chptr = NULL;
 	struct membership *msptr;
@@ -82,7 +83,7 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 		{
 			sendto_one(source_p, form_str(ERR_NEEDMOREPARAMS),
 					me.name, source_p->name, "TOPIC");
-			return 0;
+			return;
 		}
 	}
 
@@ -95,7 +96,7 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 	{
 		sendto_one_numeric(source_p, ERR_NOSUCHCHANNEL,
 				form_str(ERR_NOSUCHCHANNEL), name);
-		return 0;
+		return;
 	}
 
 	/* setting topic */
@@ -107,27 +108,27 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 		{
 			sendto_one_numeric(source_p, ERR_NOTONCHANNEL,
 					form_str(ERR_NOTONCHANNEL), name);
-			return 0;
+			return;
 		}
 
 		if(MyClient(source_p) && !is_chanop_voiced(msptr) &&
-				!IsOper(source_p) &&
+				!IsOperGeneral(source_p) &&
 				!add_channel_target(source_p, chptr))
 		{
 			sendto_one(source_p, form_str(ERR_TARGCHANGE),
 				   me.name, source_p->name, chptr->chname);
-			return 0;
+			return;
 		}
 
 		if(((chptr->mode.mode & MODE_TOPICLIMIT) == 0 ||
-					get_channel_access(source_p, msptr, MODE_ADD, NULL) >= CHFL_CHANOP) &&
+					get_channel_access(source_p, chptr, msptr, MODE_ADD, NULL) >= CHFL_CHANOP) &&
 				(!MyClient(source_p) ||
 				 can_send(chptr, source_p, msptr)))
 		{
 			char topic[TOPICLEN + 1];
 			char topic_info[USERHOST_REPLYLEN];
 			rb_strlcpy(topic, parv[2], sizeof(topic));
-			rb_sprintf(topic_info, "%s!%s@%s",
+			sprintf(topic_info, "%s!%s@%s",
 					source_p->name, source_p->username, source_p->host);
 
 			if (ConfigChannel.strip_topic_colors)
@@ -139,7 +140,7 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 					":%s TOPIC %s :%s",
 					use_id(source_p), chptr->chname,
 					chptr->topic == NULL ? "" : chptr->topic);
-			sendto_channel_local(ALL_MEMBERS,
+			sendto_channel_local(source_p, ALL_MEMBERS,
 					chptr, ":%s!%s@%s TOPIC %s :%s",
 					source_p->name, source_p->username,
 					source_p->host, chptr->chname,
@@ -159,7 +160,7 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 		{
 			sendto_one_numeric(source_p, ERR_NOTONCHANNEL,
 					form_str(ERR_NOTONCHANNEL), name);
-			return 0;
+			return;
 		}
 		if(chptr->topic == NULL)
 			sendto_one(source_p, form_str(RPL_NOTOPIC),
@@ -172,11 +173,9 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
 			sendto_one(source_p, form_str(RPL_TOPICWHOTIME),
 					me.name, source_p->name, chptr->chname,
 					chptr->topic_info,
-					(unsigned long)chptr->topic_time);
+					(long long)chptr->topic_time);
 		}
 	}
-
-	return 0;
 }
 
 /*
@@ -188,19 +187,17 @@ m_topic(struct Client *client_p, struct Client *source_p, int parc, const char *
  *
  * Let servers always set a topic
  */
-static int
-ms_topic(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
+static void
+ms_topic(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
 	struct Channel *chptr = NULL;
 
 	if((chptr = find_channel(parv[1])) == NULL)
-		return 0;
+		return;
 
 	set_channel_topic(chptr, parv[4], parv[2], atoi(parv[3]));
 
-	sendto_channel_local(ALL_MEMBERS, chptr, ":%s TOPIC %s :%s",
+	sendto_channel_local(source_p, ALL_MEMBERS, chptr, ":%s TOPIC %s :%s",
 			     source_p->name, parv[1],
 			     chptr->topic == NULL ? "" : chptr->topic);
-
-	return 0;
 }