Changeset df701918

Timestamp:

2015-12-08T22:31:05Z (9 years ago)

Author:

dequis <dx@…>

Parents:

459dec8

Message:

Add imcb_buddy_nick_change(), like nick_hint but stronger

nick_hint only works when creating new users, it's a no-op after the
user is online. This new function takes care of nick changes after that.

It also helps clean up couple of hacks in irc_im.c \o/

Files:

• irc_im.c (modified) (7 diffs)
• protocols/bee.h (modified) (1 diff)
• protocols/nogaim.c (modified) (2 diffs)
• protocols/nogaim.h (modified) (1 diff)

irc_im.c

@@ -342 +342 @@
 }
 
-static gboolean bee_irc_user_nick_update(irc_user_t *iu);
+static gboolean bee_irc_user_nick_update(irc_user_t *iu, gboolean offline_only);
 
 static gboolean bee_irc_user_fullname(bee_t *bee, bee_user_t *bu)
@@ -370 +370 @@
         }
 
-        bee_irc_user_nick_update(iu);
+        bee_irc_user_nick_update(iu, TRUE);
 
         return TRUE;
@@ -377 +377 @@
 static gboolean bee_irc_user_nick_hint(bee_t *bee, bee_user_t *bu, const char *hint)
 {
-        bee_irc_user_nick_update((irc_user_t *) bu->ui_data);
+        bee_irc_user_nick_update((irc_user_t *) bu->ui_data, TRUE);
+
+        return TRUE;
+}
+
+static gboolean bee_irc_user_nick_change(bee_t *bee, bee_user_t *bu, const char *nick)
+{
+        bee_irc_user_nick_update((irc_user_t *) bu->ui_data, FALSE);
 
         return TRUE;
@@ -388 +395 @@
         bee_user_flags_t online;
 
-        /* Take the user offline temporarily so we can change the nick (if necessary). */
-        if ((online = bu->flags & BEE_USER_ONLINE)) {
-                bu->flags &= ~BEE_USER_ONLINE;
-        }
-
         bee_irc_channel_update(irc, NULL, iu);
-        bee_irc_user_nick_update(iu);
-
-        if (online) {
-                bu->flags |= online;
-                bee_irc_channel_update(irc, NULL, iu);
-        }
-
-        return TRUE;
-}
-
-static gboolean bee_irc_user_nick_update(irc_user_t *iu)
+        bee_irc_user_nick_update(iu, FALSE);
+
+        return TRUE;
+}
+
+static gboolean bee_irc_user_nick_update(irc_user_t *iu, gboolean offline_only)
 {
         bee_user_t *bu = iu->bu;
         char *newnick;
 
-        if (bu->flags & BEE_USER_ONLINE) {
+        if (offline_only && bu->flags & BEE_USER_ONLINE) {
                 /* Ignore if the user is visible already. */
                 return TRUE;
@@ -432 +429 @@
 {
         bee_user_t *bu = iu->bu;
-        bee_user_flags_t online;
 
         if (bu == FALSE) {
@@ -438 +434 @@
         }
 
-        /* In this case, pretend the user is offline. */
-        if ((online = bu->flags & BEE_USER_ONLINE)) {
-                bu->flags &= ~BEE_USER_ONLINE;
-        }
-
         nick_del(bu);
-        bee_irc_user_nick_update(iu);
-
-        bu->flags |= online;
+        bee_irc_user_nick_update(iu, FALSE);
+
 }
 
@@ -1146 +1136 @@
 
         bee_irc_log,
+        bee_irc_user_nick_change,
 };

protocols/bee.h

@@ -131 +131 @@
 
         void (*log)(bee_t *bee, const char *tag, const char *msg);
+        gboolean (*user_nick_change)(bee_t *bee, bee_user_t *bu, const char *hint);
 } bee_ui_funcs_t;
 

protocols/nogaim.c

@@ -479 +479 @@
 }
 
-/* Mainly meant for ICQ (and now also for Jabber conferences) to allow IM
-   modules to suggest a nickname for a handle. */
-void imcb_buddy_nick_hint(struct im_connection *ic, const char *handle, const char *nick)
+/* Implements either imcb_buddy_nick_hint() or imcb_buddy_nick_change() depending on the value of 'change' */
+static void buddy_nick_hint_or_change(struct im_connection *ic, const char *handle, const char *nick, gboolean change)
 {
         bee_t *bee = ic->bee;
@@ -493 +492 @@
         bu->nick = g_strdup(nick);
 
-        if (bee->ui->user_nick_hint) {
+        if (change && bee->ui->user_nick_change) {
+                bee->ui->user_nick_change(bee, bu, nick);
+        } else if (!change && bee->ui->user_nick_hint) {
                 bee->ui->user_nick_hint(bee, bu, nick);
         }
 }
 
+/* Soft variant, for newly created users. Does nothing if it's already online */
+void imcb_buddy_nick_hint(struct im_connection *ic, const char *handle, const char *nick)
+{
+        buddy_nick_hint_or_change(ic, handle, nick, FALSE);
+}
+
+/* Hard variant, always changes the nick */
+void imcb_buddy_nick_change(struct im_connection *ic, const char *handle, const char *nick)
+{
+        buddy_nick_hint_or_change(ic, handle, nick, TRUE);
+}
 
 struct imcb_ask_cb_data {

protocols/nogaim.h

@@ -325 +325 @@
 G_MODULE_EXPORT void imcb_rename_buddy(struct im_connection *ic, const char *handle, const char *realname);
 G_MODULE_EXPORT void imcb_buddy_nick_hint(struct im_connection *ic, const char *handle, const char *nick);
+G_MODULE_EXPORT void imcb_buddy_nick_change(struct im_connection *ic, const char *handle, const char *nick);
 G_MODULE_EXPORT void imcb_buddy_action_response(bee_user_t *bu, const char *action, char * const args[], void *data);