Changeset 16ea00d

Timestamp:

2018-02-11T21:27:17Z (7 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

8167346

Parents:

1dcbd3e

Message:

root_commands: Improve 'chat add' error/success messages

This handles the most common error case with an obvious error message
(channel already exists, join it, etc), and the rarer ones with vague
but different wording. For those cases:

• "Could not add chatroom." This wording is not present in this version anymore, likely means the channel already exists.

• "Error creating channel for chatroom." irc_channel_new() failed but the channel doesn't exist already. This could be that some name validation failed, but this function already generates a suitable name.

• "Error adding chatroom." irc_channel_new() succeeded, but one of the set_setstr() failed. There are account checks earlier, and the room setting is not normally validated at this point.

Can't really imagine situations where these vague errors would still
show up, but at least now you only get those for unknown cases.

File:

• root_commands.c (modified) (1 diff)

root_commands.c

@@ -1342 +1342 @@
                     set_setstr(&ic->set, "account", cmd[2]) &&
                     set_setstr(&ic->set, "room", room)) {
-                        irc_rootmsg(irc, "Chatroom successfully added.");
+                        irc_rootmsg(irc, "Chatroom successfully added, join with \002/join %s\002", channel);
                 } else {
                         if (ic) {
                                 irc_channel_free(ic);
-                        }
-
-                        irc_rootmsg(irc, "Could not add chatroom.");
+
+                                irc_rootmsg(irc, "Error adding chatroom.");
+                        } else if (irc_channel_by_name(irc, channel)) {
+                                irc_rootmsg(irc, "A channel named `%s' already exists. "
+                                            "Join with \002/join %s\002 or see \002help channel\002 to modify it",
+                                            channel, channel);
+                        } else {
+                                irc_rootmsg(irc, "Error creating channel for chatroom.");
+                        }
                 }
                 g_free(channel);