Changeset ce402b2 for protocols/twitter/twitter_lib.c

Timestamp:

2015-01-26T07:50:54Z (9 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

12fe5ea

Parents:

dfaa46d

git-author:

William Pettersson <william.pettersson@…> (11-01-15 05:08:37)

git-committer:

dequis <dx@…> (26-01-15 07:50:54)

Message:

Twitter format strings

Allow users to specify how tweets should be displayed

3 new settings are available to set how tweets are displayed:

• twitter_format_string for normal tweets
• retweet_format_string for retweets
• reply_format_string for replies

For full documentation see the help files

File:

• protocols/twitter/twitter_lib.c (modified) (5 diffs)

protocols/twitter/twitter_lib.c

@@ -63 +63 @@
         guint64 reply_to;
         gboolean from_filter;
+        struct twitter_xml_status *rt;
 };
 
@@ -88 +89 @@
         g_free(txs->text);
         txu_free(txs->user);
+        txs_free(txs->rt);
         g_free(txs);
 }
@@ -489 +491 @@
                 if (rtxs) {
                         g_free(txs->text);
-                        txs->text = g_strdup_printf("RT @%s: %s", rtxs->user->screen_name, rtxs->text);
+                        txs->text = g_strdup(rtxs->text);
                         txs->id = rtxs->id;
-                        txs_free(rtxs);
+                        txs->rt = rtxs;
                 }
         } else if (entities) {
@@ -609 +611 @@
 }
 
+/**
+ * Function to properly format a tweet as per the users configuration.
+ */
+static char *twitter_msg_get_text(struct im_connection *ic, int log_id, int reply_to,
+                                struct twitter_xml_status *txs, const char *prefix) {
+        gchar * format = set_getstr(&ic->acc->set, "format_string");
+        GString * text = g_string_new(NULL);
+
+        gchar *c;
+        if (reply_to != -1)
+                format = set_getstr(&ic->acc->set, "reply_format_string");
+        if (txs->rt)
+                format = set_getstr(&ic->acc->set, "retweet_format_string");
+
+        for (c = format; *c ; c++) {
+                if (!(*c == '%' && *(c+1))) {
+                        text = g_string_append_c(text, *c);
+                        continue;
+                }
+                c++; // Move past the %
+                switch (*c) {
+                        case 'i':
+                                g_string_append_printf(text, "%02x", log_id);
+                                break;
+                        case 'r':
+                                if (reply_to != -1) // In case someone does put %r in the wrong format_string
+                                g_string_append_printf(text, "%02x", reply_to);
+                                break;
+                        case 'a':
+                                if (txs->rt) // In case someone does put %a in the wrong format_string
+                                        text = g_string_append(text, txs->rt->user->screen_name);
+                                break;
+                        case 'c':
+                                text = g_string_append(text, txs->text);
+                                break;
+                        default:
+                                text = g_string_append_c(text, *c);
+                }
+        }
+        text = g_string_prepend(text, prefix);
+        return g_string_free(text, FALSE);
+}
+
 /* Will log messages either way. Need to keep track of IDs for stream deduping.
    Plus, show_ids is on by default and I don't see why anyone would disable it. */
@@ -647 +692 @@
                 td->log[td->log_id].id = txs->rt_id;
         
-        if (set_getbool(&ic->acc->set, "show_ids")) {
-                if (reply_to != -1)
-                        return g_strdup_printf("\002[\002%02x->%02x\002]\002 %s%s",
-                                               td->log_id, reply_to, prefix, txs->text);
-                else
-                        return g_strdup_printf("\002[\002%02x\002]\002 %s%s",
-                                               td->log_id, prefix, txs->text);
-        } else {
-                if (*prefix)
-                        return g_strconcat(prefix, txs->text, NULL);
-                else
-                        return NULL;
-        }
+        return twitter_msg_get_text(ic, td->log_id, reply_to, txs, prefix);
 }