From 15bd1a609dc97c9b71bac254441b05dca9f81032 Mon Sep 17 00:00:00 2001 From: Jari Matilainen Date: Mon, 30 Jun 2014 21:04:56 +0200 Subject: [PATCH 1/2] Update doc/special_vars to include examples of /alias al script exec usage and escaping --- docs/special_vars.txt | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/docs/special_vars.txt b/docs/special_vars.txt index f19b2bb2..36517f78 100644 --- a/docs/special_vars.txt +++ b/docs/special_vars.txt @@ -114,3 +114,19 @@ surrounding text will not affect the expression's return value. /eval echo foo${N}foo /* ${N} returns current nickname */ fooYourNickfoo /* returned by above command */ +When writing an alias containing a /script exec, special consideration has to be +taken to $vars and statement delimiters, ie. ; +/alias tries to evaluate all $vars as expandos, which would mean that what you +pass on to /script exec isn't necessarily what you intended. +Compare: + + 1. /alias al1 script exec my $var = "Hello"; print $var; + 2. /alias al2 script exec my $$var = "Hello"\; print $$var; + 3. /alias al3 script exec my \$var = "Hello"\; print \$var; (Same as nr 2) + +In example nr 1 $var would be expanded to an empty string and ; would end +the /script exec command, leaving print $var as a separate command to be run by +irssi. In example 2 $$ is evaluated to a literal $ leaving a literal $var to be +passed on to /script exec. The same goes for \; which is turned into a +literal ; and thus is passed on to /script exec as a statement delimiter. +This would mean print $$var is part of the /script exec being evaluated. From 819ae3027d134e8b48ecd6aa72136aba2958039b Mon Sep 17 00:00:00 2001 From: Jari Matilainen Date: Wed, 2 Jul 2014 17:11:19 +0200 Subject: [PATCH 2/2] Add a setting to configure rejoin on reconnect behaviour --- src/irc/core/irc-servers.c | 45 +++++++++++++++++++++++++++++--------- 1 file changed, 35 insertions(+), 10 deletions(-) diff --git a/src/irc/core/irc-servers.c b/src/irc/core/irc-servers.c index e5f86c20..603ac5e6 100644 --- a/src/irc/core/irc-servers.c +++ b/src/irc/core/irc-servers.c @@ -32,6 +32,7 @@ #include "irc-queries.h" #include "irc-servers-setup.h" #include "irc-servers.h" +#include "channels-setup.h" #include "channel-rejoin.h" #include "servers-idle.h" #include "servers-reconnect.h" @@ -55,6 +56,7 @@ void irc_servers_reconnect_init(void); void irc_servers_reconnect_deinit(void); static int cmd_tag; +static char *rejoin_channels_mode; static int isnickflag_func(SERVER_REC *server, char flag) { @@ -482,6 +484,11 @@ char *irc_server_get_channels(IRC_SERVER_REC *server) GSList *tmp; GString *chans, *keys; char *ret; + rejoin_channels_mode = settings_get_str("rejoin_channels_on_reconnect"); + + if (rejoin_channels_mode == NULL || (g_ascii_strcasecmp(rejoin_channels_mode, "on") != 0 && g_ascii_strcasecmp(rejoin_channels_mode, "on") != 0 && g_ascii_strcasecmp(rejoin_channels_mode, "on") != 0)) + rejoin_channels_mode = g_strdup("on"); + int use_keys; g_return_val_if_fail(server != NULL, FALSE); @@ -490,25 +497,40 @@ char *irc_server_get_channels(IRC_SERVER_REC *server) keys = g_string_new(NULL); use_keys = FALSE; + /* do we want to rejoin channels in the first place? */ + if(g_ascii_strcasecmp(rejoin_channels_mode, "off") == 0) { + g_string_free(chans, TRUE); + g_string_free(keys, TRUE); + return g_strdup(""); + } + /* get currently joined channels */ for (tmp = server->channels; tmp != NULL; tmp = tmp->next) { CHANNEL_REC *channel = tmp->data; - - g_string_append_printf(chans, "%s,", channel->name); - g_string_append_printf(keys, "%s,", channel->key == NULL ? "x" : - channel->key); - if (channel->key != NULL) - use_keys = TRUE; + CHANNEL_SETUP_REC *setup = channel_setup_find(channel->name, + channel->server->connrec->chatnet); + if ((setup != NULL && setup->autojoin && g_ascii_strcasecmp(rejoin_channels_mode, "auto") == 0) || g_ascii_strcasecmp(rejoin_channels_mode, "on") == 0) { + g_string_append_printf(chans, "%s,", channel->name); + g_string_append_printf(keys, "%s,", channel->key == NULL ? "x" : + channel->key); + if (channel->key != NULL) + use_keys = TRUE; + } } /* get also the channels that are in rejoin list */ for (tmp = server->rejoin_channels; tmp != NULL; tmp = tmp->next) { REJOIN_REC *rec = tmp->data; + CHANNEL_SETUP_REC *setup = channel_setup_find(rec->channel, + server->tag); - g_string_append_printf(chans, "%s,", rec->channel); - g_string_append_printf(keys, "%s,", rec->key == NULL ? "x" : - rec->key); - if (rec->key != NULL) use_keys = TRUE; + if ((setup != NULL && setup->autojoin && g_ascii_strcasecmp(rejoin_channels_mode, "auto") == 0) || g_ascii_strcasecmp(rejoin_channels_mode, "on") == 0) { + g_string_append_printf(chans, "%s,", rec->channel); + g_string_append_printf(keys, "%s,", rec->key == NULL ? "x" : + rec->key); + + if (rec->key != NULL) use_keys = TRUE; + } } if (chans->len > 0) { @@ -865,6 +887,7 @@ void irc_server_init_isupport(IRC_SERVER_REC *server) void irc_servers_init(void) { + settings_add_str("servers", "rejoin_channels_on_reconnect", "on"); settings_add_str("misc", "usermode", DEFAULT_USER_MODE); settings_add_time("flood", "cmd_queue_speed", DEFAULT_CMD_QUEUE_SPEED); settings_add_int("flood", "cmds_max_at_once", DEFAULT_CMDS_MAX_AT_ONCE); @@ -897,6 +920,8 @@ void irc_servers_deinit(void) { if (cmd_tag != -1) g_source_remove(cmd_tag); + if (rejoin_channels_mode != NULL) + g_free(rejoin_channels_mode); signal_remove("server connected", (SIGNAL_FUNC) sig_connected); signal_remove("server disconnected", (SIGNAL_FUNC) sig_disconnected);