Added support for bahamut @+#channel notices. Removed notice_public_ops

format, notice_public is just fine if the channel just contains @ or @+
at the beginning.


git-svn-id: http://svn.irssi.org/repos/irssi/trunk@1538 dbcabf3a-b0e7-0310-adc4-f8d773084564

src/fe-common/irc/fe-irc-commands.c

@@ -40,6 +40,19 @@
 #include "printtext.h"
 #include "keyboard.h"
 
+static char *skip_target(char *target)
+{
+	if (*target == '@') {
+		/* @#channel, @+#channel - Hybrid6 / Bahamut features */
+		if (target[1] == '+' && ischannel(target[2]))
+			target += 2;
+		else if (ischannel(target[1]))
+			target++;
+	}
+
+	return target;
+}
+
 /* SYNTAX: ME <message> */
 static void cmd_me(const char *data, IRC_SERVER_REC *server, WI_ITEM_REC *item)
 {
@@ -75,6 +88,7 @@ static void cmd_action(const char *data, IRC_SERVER_REC *server)
 
 	signal_emit("message irc own_action", 3, server, text, target);
 
+	target = skip_target(target);
 	irc_send_cmdv(server, "PRIVMSG %s :\001ACTION %s\001", target, text);
 	cmd_params_free(free_arg);
 }
@@ -94,11 +108,7 @@ static void cmd_notice(const char *data, IRC_SERVER_REC *server)
 	if (*target == '\0' || *msg == '\0')
 		cmd_param_error(CMDERR_NOT_ENOUGH_PARAMS);
 
-	if (*target == '@' && ischannel(target[1])) {
+	target = skip_target(target);
-		/* Hybrid 6 feature, send notice to all ops in channel */
-		target++;
-	}
-
 	signal_emit("message irc own_notice", 3, server, msg, target);
 	cmd_params_free(free_arg);
 }
@@ -124,10 +134,7 @@ static void cmd_ctcp(const char *data, IRC_SERVER_REC *server)
 		return;
 	}
 
-	if (*target == '@' && ischannel(target[1])) {
+	target = skip_target(target);
-                /* Hybrid 6 feature, send ctcp to all ops in channel */
-		target++;
-	}
 
 	g_strup(ctcpcmd);
 	signal_emit("message irc own_ctcp", 4,
@@ -151,11 +158,7 @@ static void cmd_nctcp(const char *data, IRC_SERVER_REC *server)
 	if (*target == '\0' || *text == '\0')
 		cmd_param_error(CMDERR_NOT_ENOUGH_PARAMS);
 
-	if (*target == '@' && ischannel(target[1])) {
+	target = skip_target(target);
-                /* Hybrid 6 feature, send notice to all ops in channel */
-		target++;
-	}
-
 	signal_emit("message irc own_notice", 3, server, text, target);
 	cmd_params_free(free_arg);
 }

src/fe-common/irc/fe-irc-messages.c

@@ -37,22 +37,38 @@
 #include "fe-queries.h"
 #include "window-items.h"
 
+static const char *skip_target(const char *target)
+{
+	if (target != NULL && *target == '@') {
+		/* @#channel, @+#channel - Hybrid6 / Bahamut features */
+		if (target[1] == '+' && ischannel(target[2]))
+			target += 2;
+		else if (ischannel(target[1]))
+			target++;
+	}
+
+	return target;
+}
+
 static void sig_message_own_public(SERVER_REC *server, const char *msg,
 				   const char *target, const char *origtarget)
 {
+	const char *oldtarget;
 	char *nickmode;
 
-	if (IS_IRC_SERVER(server) && target != NULL &&
+	oldtarget = target;
-	    *target == '@' && ischannel(target[1])) {
+	target = skip_target(target);
-		/* Hybrid 6 feature, send msg to all ops in channel */
+	if (IS_IRC_SERVER(server) && target != oldtarget) {
-		nickmode = channel_get_nickmode(channel_find(server, target+1),
+		/* Hybrid 6 / Bahamut feature, send msg to all
+		   ops / ops+voices in channel */
+		nickmode = channel_get_nickmode(channel_find(server, target),
 						server->nick);
 
-		printformat_module("fe-common/core", server, target+1,
+		printformat_module("fe-common/core", server, target,
 				   MSGLEVEL_PUBLIC | MSGLEVEL_NOHILIGHT |
 				   MSGLEVEL_NO_ACT,
 				   TXT_OWN_MSG_CHANNEL,
-				   server->nick, target, msg, nickmode);
+				   server->nick, oldtarget, msg, nickmode);
                 signal_stop();
 	}
 }
@@ -172,7 +188,7 @@ static void sig_message_irc_notice(SERVER_REC *server, const char *msg,
 				   const char *nick, const char *address,
 				   const char *target)
 {
-	int op_notice;
+	const char *oldtarget;
 	
 	if (address == NULL) {
 		/* notice from server */
@@ -184,8 +200,8 @@ static void sig_message_irc_notice(SERVER_REC *server, const char *msg,
                 return;
 	}
 
-	op_notice = *target == '@' && ischannel(target[1]);
+	oldtarget = target;
-	if (op_notice) target++;
+	target = skip_target(target);
 
 	if (ignore_check(server, nick, address,
 			 ischannel(*target) ? target : NULL,
@@ -195,8 +211,7 @@ static void sig_message_irc_notice(SERVER_REC *server, const char *msg,
 	if (ischannel(*target)) {
 		/* notice in some channel */
 		printformat(server, target, MSGLEVEL_NOTICES,
-			    op_notice ? IRCTXT_NOTICE_PUBLIC_OPS :
+			    IRCTXT_NOTICE_PUBLIC, nick, oldtarget, msg);
-			    IRCTXT_NOTICE_PUBLIC, nick, target, msg);
 	} else {
 		/* private notice */
 		privmsg_get_query(SERVER(server), nick, FALSE,

src/fe-common/irc/module-formats.c

@@ -127,7 +127,6 @@ FORMAT_REC fecommon_irc_formats[] = {
 
 	{ "notice_server", "{servernotice $0}$1", 2, { 0, 0 } },
 	{ "notice_public", "{notice $0{pubnotice_channel $1}}$2", 3, { 0, 0, 0 } },
-	{ "notice_public_ops", "{notice $0{pubnotice_channel @$1}}$2", 3, { 0, 0, 0 } },
 	{ "notice_private", "{notice $0{pvtnotice_host $1}}$2", 3, { 0, 0, 0 } },
 	{ "action_private", "{pvtaction $0}$2", 3, { 0, 0, 0 } },
 	{ "action_private_query", "{pvtaction_query $0}$2", 3, { 0, 0, 0 } },

src/fe-common/irc/module-formats.h

@@ -100,7 +100,6 @@ enum {
 
 	IRCTXT_NOTICE_SERVER,
 	IRCTXT_NOTICE_PUBLIC,
-	IRCTXT_NOTICE_PUBLIC_OPS,
 	IRCTXT_NOTICE_PRIVATE,
 	IRCTXT_ACTION_PRIVATE,
 	IRCTXT_ACTION_PRIVATE_QUERY,

src/irc/core/irc-servers.c

@@ -71,7 +71,13 @@ static int isnickflag_func(char flag)
 
 static int ischannel_func(const char *data)
 {
-	return ischannel_target(data);
+	if (*data == '@') {
+		/* @#channel, @+#channel */
+		data++;
+		if (*data == '+' && ischannel(data[1]))
+			return 1;
+	}
+	return ischannel(*data);
 }
 
 static void send_message(SERVER_REC *server, const char *target,

src/irc/core/irc.h

@@ -25,10 +25,6 @@
 	(a) == '!' || /* secure */ \
 	(a) == '+') /* modeless */
 
-#define ischannel_target(a) \
-	(ischannel((a)[0]) || \
-	((a)[0] == '@' && ischannel((a)[1]))) /* hybrid6 @#channel */
-
 #define IS_IRC_ITEM(rec) (IS_IRC_CHANNEL(rec) || IS_IRC_QUERY(rec))
 
 extern char *current_server_event; /* current server event being processed */