Changes in protocols/nogaim.c [977a9d5:9076a1c]

File:

• protocols/nogaim.c (modified) (13 diffs)

protocols/nogaim.c

@@ -90 +90 @@
 
 GList *protocols = NULL;
-GList *disabled_protocols = NULL;
 
 void register_protocol(struct prpl *p)
@@ -104 +103 @@
 
         if (refused) {
-                disabled_protocols = g_list_append(disabled_protocols, p);
+                log_message(LOGLVL_WARNING, "Protocol %s disabled\n", p->name);
         } else {
                 protocols = g_list_append(protocols, p);
@@ -110 +109 @@
 }
 
-static int proto_name_cmp(const void *proto_, const void *name)
-{
-        const struct prpl *proto = proto_;
-        return g_strcasecmp(proto->name, name);
-}
-
 struct prpl *find_protocol(const char *name)
 {
-        GList *gl = g_list_find_custom(protocols, name, proto_name_cmp);
-        return gl ? gl->data: NULL;
-}
-
-gboolean is_protocol_disabled(const char *name)
-{
-        return g_list_find_custom(disabled_protocols, name, proto_name_cmp) != NULL;
+        GList *gl;
+
+        for (gl = protocols; gl; gl = gl->next) {
+                struct prpl *proto = gl->data;
+
+                if (g_strcasecmp(proto->name, name) == 0) {
+                        return proto;
+                }
+        }
+
+        return NULL;
 }
 
@@ -135 +132 @@
         extern void twitter_initmodule();
         extern void purple_initmodule();
+        extern void rpc_initmodule();
 
 #ifdef WITH_MSN
@@ -158 +156 @@
 #ifdef WITH_PURPLE
         purple_initmodule();
+#endif
+
+#ifdef WITH_RPC
+        rpc_initmodule();
 #endif
 
@@ -207 +209 @@
         account_t *a;
 
-        if (!ic->bee->ui->log) {
-                return;
-        }
-
         va_start(params, format);
         text = g_strdup_vprintf(format, params);
@@ -229 +227 @@
         /* If we found one, include the screenname in the message. */
         if (a) {
-                ic->bee->ui->log(ic->bee, ic->acc->tag, text);
+                /* FIXME(wilmer): ui_log callback or so */
+                irc_rootmsg(ic->bee->ui_data, "%s - %s", ic->acc->tag, text);
         } else {
-                ic->bee->ui->log(ic->bee, ic->acc->prpl->name, text);
+                irc_rootmsg(ic->bee->ui_data, "%s - %s", ic->acc->prpl->name, text);
         }
 
@@ -313 +312 @@
         }
 
-        if (ic->acc->flags & ACC_FLAG_LOCAL) {
+        if ((ic->acc->flags & ACC_FLAG_LOCAL_CONTACTS) &&
+            !(ic->flags & OPT_LOCAL_CONTACTS_SENT) &&
+            ic->acc->prpl->add_buddy) {
                 GHashTableIter nicks;
-                gpointer k, v;
+                gpointer handle;
                 g_hash_table_iter_init(&nicks, ic->acc->nicks);
-                while (g_hash_table_iter_next(&nicks, &k, &v)) {
-                        ic->acc->prpl->add_buddy(ic, (char *) k, NULL);
+                while (g_hash_table_iter_next(&nicks, &handle, NULL)) {
+                        ic->acc->prpl->add_buddy(ic, (char *) handle, NULL);
                 }
         }
@@ -413 +414 @@
         query_del_by_conn((irc_t *) ic->bee->ui_data, ic);
 
+        /* Throw away groupchats owned by this account. Historically this was only
+           ever done by IM modules which is a bug. But it gives them opportunity
+           to clean up protocol-specific bits as well so keep it that way, just
+           do another cleanup here as a fallback. Don't want to leave any dangling
+           pointers! */
+        while (ic->groupchats) {
+                imcb_chat_free(ic->groupchats->data);
+        }
+
         if (!a) {
                 /* Uhm... This is very sick. */
@@ -498 +508 @@
 }
 
+/* Returns the local contacts for an IM account (based on assigned nicks).
+   Linked list should be freed, the strings themselves not! So look at it
+   like a GSList<const char*> I guess? Empty list means NULL retval (as
+   always with GSList). */
+GSList *imcb_get_local_contacts(struct im_connection *ic)
+{
+        GHashTableIter nicks;
+        GSList *ret = NULL;
+        
+        if (!(ic->acc->flags & ACC_FLAG_LOCAL_CONTACTS)) {
+                /* Only allow protocols that indicate local contact list
+                   support to use this function. */
+                return ret;
+        }
+        
+        g_hash_table_iter_init(&nicks, ic->acc->nicks);
+        gpointer handle;
+        while (g_hash_table_iter_next(&nicks, &handle, NULL)) {
+                ret = g_slist_prepend(ret, (char *) handle);
+        }
+        
+        /* If the protocol asked for the list, assume we won't have to send it
+           anymore in imcb_connected(). */
+        ic->flags |= OPT_LOCAL_CONTACTS_SENT;
+        
+        return ret;
+}
+
 
 struct imcb_ask_cb_data {
@@ -556 +594 @@
         struct imcb_ask_cb_data *cbd = data;
 
-        cbd->ic->acc->prpl->add_buddy(cbd->ic, cbd->handle, NULL);
+        if (cbd->ic->acc->prpl->add_buddy) {
+                cbd->ic->acc->prpl->add_buddy(cbd->ic, cbd->handle, NULL);
+        }
 
         imcb_ask_cb_free(data);
@@ -622 +662 @@
                 GList *m = ic->acc->prpl->away_states(ic);
                 msg = ic->acc->flags & ACC_FLAG_AWAY_MESSAGE ? away : NULL;
-                away = imc_away_state_find(m, away, &msg) ? :
-                       (imc_away_state_find(m, "away", &msg) ? : m->data);
+                away = imc_away_state_find(m, away, &msg) ? : m->data;
         } else if (ic->acc->flags & ACC_FLAG_STATUS_MESSAGE) {
                 away = NULL;
@@ -742 +781 @@
 }
 
-/* Deprecated: using this function resulted in merging several handles accidentally
- * Also the irc layer handles this decently nowadays */
 void imcb_clean_handle(struct im_connection *ic, char *handle)
 {
-}
+        /* Accepts a handle and does whatever is necessary to make it
+           BitlBee-friendly. Currently this means removing everything
+           outside 33-127 (ASCII printable excl spaces), @ (only one
+           is allowed) and ! and : */
+        char out[strlen(handle) + 1];
+        int s, d;
+
+        s = d = 0;
+        while (handle[s]) {
+                if (handle[s] > ' ' && handle[s] != '!' && handle[s] != ':' &&
+                    (handle[s] & 0x80) == 0) {
+                        if (handle[s] == '@') {
+                                /* See if we got an @ already? */
+                                out[d] = 0;
+                                if (strchr(out, '@')) {
+                                        continue;
+                                }
+                        }
+
+                        out[d++] = handle[s];
+                }
+                s++;
+        }
+        out[d] = handle[s];
+
+        strcpy(handle, out);
+}