Changeset 05aba55

Timestamp:

2015-12-01T04:55:21Z (9 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

48b5fef

Parents:

03df717

Message:

purple: Implement PurpleNotifyUiOps.notify_message

Which has no connection context. Luckily local_bee exists, and libpurple
only supports only one irc user per process.

This sucks.

And yesterday I was naively thinking (again) that local_bee might not be
needed, that maybe we can do things properly. Of course it only took a
look at that one reverted commit (56985aa) to remember that life is
unfair, and that, under Moloch, everyone is irresistably incentivized to
ignore the things that unite us in favor of forever picking at the
things that divide us in exactly the way that is most likely to make
them more divisive. That being said, I think all these hacks are going
to look nicer once I sandbox the whole thing in a separate process with
one IM account per process, as opposed to one irc use per process. Then
we'll be able to rely on global state exclusively, which is saner.

File:

• protocols/purple/purple.c (modified) (2 diffs)

protocols/purple/purple.c

@@ -1288 +1288 @@
 };
 
+/* Absolutely no connection context at all. Thanks purple! brb crying */
+static void *prplcb_notify_message(PurpleNotifyMsgType type, const char *title,
+                                   const char *primary, const char *secondary)
+{
+        char *text = g_strdup_printf("%s%s - %s%s%s",
+                (type == PURPLE_NOTIFY_MSG_ERROR) ? "Error: " : "",
+                title,
+                primary ?: "",
+                (primary && secondary) ? " - " : "",
+                secondary ?: ""
+        );
+
+        if (local_bee->ui->log) {
+                local_bee->ui->log(local_bee, "purple", text);
+        }
+
+        g_free(text);
+
+        return NULL;
+}
+
 static void *prplcb_notify_email(PurpleConnection *gc, const char *subject, const char *from,
                                  const char *to, const char *url)
@@ -1348 +1369 @@
 static PurpleNotifyUiOps bee_notify_uiops =
 {
-        NULL,
+        prplcb_notify_message,
         prplcb_notify_email,
         NULL,