about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorJames Booth <boothj5@gmail.com>2016-08-28 22:12:50 +0100
committerJames Booth <boothj5@gmail.com>2016-08-28 22:12:50 +0100
commitea98a8b04a0577bd6aacc81f9c98b2b7c4d5cc68 (patch)
treef4d8be4328b8f8d76c47fd8eef9f5698b2c95353
parent6a18dedca8947ea9faa25940ce990268f2661c0a (diff)
downloadprofani-tty-ea98a8b04a0577bd6aacc81f9c98b2b7c4d5cc68.tar.gz
Rename Bookmark->jid to Bookmark->barejid
-rw-r--r--src/ui/console.c8
-rw-r--r--src/xmpp/bookmark.c38
-rw-r--r--src/xmpp/xmpp.h2
-rw-r--r--tests/unittests/test_cmd_bookmark.c14
4 files changed, 31 insertions, 31 deletions
diff --git a/src/ui/console.c b/src/ui/console.c
index 3c104f4d..80f854c3 100644
--- a/src/ui/console.c
+++ b/src/ui/console.c
@@ -698,10 +698,10 @@ cons_show_bookmarks(const GList *list)
 
             theme_item_t presence_colour = THEME_TEXT;
 
-            if (muc_active(item->jid)) {
+            if (muc_active(item->barejid)) {
                 presence_colour = THEME_ONLINE;
             }
-            win_vprint(console, '-', 0, NULL, NO_EOL, presence_colour, "", "  %s", item->jid);
+            win_vprint(console, '-', 0, NULL, NO_EOL, presence_colour, "", "  %s", item->barejid);
             if (item->nick) {
                 win_vprint(console, '-', 0, NULL, NO_DATE | NO_EOL, presence_colour, "", "/%s", item->nick);
             }
@@ -711,8 +711,8 @@ cons_show_bookmarks(const GList *list)
             if (item->password) {
                 win_print(console, '-', 0, NULL, NO_DATE | NO_EOL, presence_colour, "", " (private)");
             }
-            if (muc_active(item->jid)) {
-                ProfWin *roomwin = (ProfWin*)wins_get_muc(item->jid);
+            if (muc_active(item->barejid)) {
+                ProfWin *roomwin = (ProfWin*)wins_get_muc(item->barejid);
                 if (roomwin) {
                     int num = wins_get_num(roomwin);
                     win_vprint(console, '-', 0, NULL, NO_DATE | NO_EOL, presence_colour, "", " (win %d)", num);
diff --git a/src/xmpp/bookmark.c b/src/xmpp/bookmark.c
index 8b9b6758..88b452f2 100644
--- a/src/xmpp/bookmark.c
+++ b/src/xmpp/bookmark.c
@@ -103,7 +103,7 @@ bookmark_add(const char *jid, const char *nick, const char *password, const char
     }
 
     Bookmark *bookmark = malloc(sizeof(Bookmark));
-    bookmark->jid = strdup(jid);
+    bookmark->barejid = strdup(jid);
     if (nick) {
         bookmark->nick = strdup(nick);
     } else {
@@ -171,16 +171,16 @@ bookmark_join(const char *jid)
 
     char *account_name = session_get_account_name();
     ProfAccount *account = accounts_get_account(account_name);
-    if (!muc_active(bookmark->jid)) {
+    if (!muc_active(bookmark->barejid)) {
         char *nick = bookmark->nick;
         if (!nick) {
             nick = account->muc_nick;
         }
-        presence_join_room(bookmark->jid, nick, bookmark->password);
-        muc_join(bookmark->jid, nick, bookmark->password, FALSE);
+        presence_join_room(bookmark->barejid, nick, bookmark->password);
+        muc_join(bookmark->barejid, nick, bookmark->password, FALSE);
         account_free(account);
-    } else if (muc_roster_complete(bookmark->jid)) {
-        ui_room_join(bookmark->jid, TRUE);
+    } else if (muc_roster_complete(bookmark->barejid)) {
+        ui_room_join(bookmark->barejid, TRUE);
         account_free(account);
     }
 
@@ -259,13 +259,13 @@ _bookmark_result_id_handler(xmpp_stanza_t *const stanza, void *const userdata)
             child = xmpp_stanza_get_next(child);
             continue;
         }
-        const char *jid = xmpp_stanza_get_attribute(child, STANZA_ATTR_JID);
-        if (!jid) {
+        const char *barejid = xmpp_stanza_get_attribute(child, STANZA_ATTR_JID);
+        if (!barejid) {
             child = xmpp_stanza_get_next(child);
             continue;
         }
 
-        log_debug("Handle bookmark for %s", jid);
+        log_debug("Handle bookmark for %s", barejid);
 
         char *nick = NULL;
         xmpp_stanza_t *nick_st = xmpp_stanza_get_child_by_name(child, "nick");
@@ -285,13 +285,13 @@ _bookmark_result_id_handler(xmpp_stanza_t *const stanza, void *const userdata)
             autojoin_val = TRUE;
         }
 
-        autocomplete_add(bookmark_ac, jid);
+        autocomplete_add(bookmark_ac, barejid);
         Bookmark *bookmark = malloc(sizeof(Bookmark));
-        bookmark->jid = strdup(jid);
+        bookmark->barejid = strdup(barejid);
         bookmark->nick = nick;
         bookmark->password = password;
         bookmark->autojoin = autojoin_val;
-        g_hash_table_insert(bookmarks, strdup(jid), bookmark);
+        g_hash_table_insert(bookmarks, strdup(barejid), bookmark);
 
         if (autojoin_val) {
             Jid *room_jid;
@@ -302,11 +302,11 @@ _bookmark_result_id_handler(xmpp_stanza_t *const stanza, void *const userdata)
                 nick = account->muc_nick;
             }
 
-            log_debug("Autojoin %s with nick=%s", jid, nick);
-            room_jid = jid_create_from_bare_and_resource(jid, nick);
+            log_debug("Autojoin %s with nick=%s", barejid, nick);
+            room_jid = jid_create_from_bare_and_resource(barejid, nick);
             if (!muc_active(room_jid->barejid)) {
-                presence_join_room(jid, nick, password);
-                muc_join(jid, nick, password, TRUE);
+                presence_join_room(barejid, nick, password);
+                muc_join(barejid, nick, password, TRUE);
             }
             jid_destroy(room_jid);
             account_free(account);
@@ -325,7 +325,7 @@ _bookmark_destroy(Bookmark *bookmark)
         return;
     }
 
-    free(bookmark->jid);
+    free(bookmark->barejid);
     free(bookmark->nick);
     free(bookmark->password);
     free(bookmark);
@@ -354,9 +354,9 @@ _send_bookmarks(void)
         Bookmark *bookmark = curr->data;
         xmpp_stanza_t *conference = xmpp_stanza_new(ctx);
         xmpp_stanza_set_name(conference, STANZA_NAME_CONFERENCE);
-        xmpp_stanza_set_attribute(conference, STANZA_ATTR_JID, bookmark->jid);
+        xmpp_stanza_set_attribute(conference, STANZA_ATTR_JID, bookmark->barejid);
 
-        Jid *jidp = jid_create(bookmark->jid);
+        Jid *jidp = jid_create(bookmark->barejid);
         if (jidp->localpart) {
             xmpp_stanza_set_attribute(conference, STANZA_ATTR_NAME, jidp->localpart);
         }
diff --git a/src/xmpp/xmpp.h b/src/xmpp/xmpp.h
index 6157b648..8dfe7c0e 100644
--- a/src/xmpp/xmpp.h
+++ b/src/xmpp/xmpp.h
@@ -78,7 +78,7 @@ typedef enum {
 } jabber_invite_t;
 
 typedef struct bookmark_t {
-    char *jid;
+    char *barejid;
     char *nick;
     char *password;
     gboolean autojoin;
diff --git a/tests/unittests/test_cmd_bookmark.c b/tests/unittests/test_cmd_bookmark.c
index 759b8238..e321f0db 100644
--- a/tests/unittests/test_cmd_bookmark.c
+++ b/tests/unittests/test_cmd_bookmark.c
@@ -62,7 +62,7 @@ void cmd_bookmark_shows_usage_when_no_args(void **state)
 
 static void _free_bookmark(Bookmark *bookmark)
 {
-    free(bookmark->jid);
+    free(bookmark->barejid);
     free(bookmark->nick);
     free(bookmark);
 }
@@ -70,7 +70,7 @@ static void _free_bookmark(Bookmark *bookmark)
 static gboolean
 _cmp_bookmark(Bookmark *bm1, Bookmark *bm2)
 {
-    if (strcmp(bm1->jid, bm2->jid) != 0) {
+    if (strcmp(bm1->barejid, bm2->barejid) != 0) {
         return FALSE;
     }
     if (strcmp(bm1->nick, bm2->nick) != 0) {
@@ -91,23 +91,23 @@ void cmd_bookmark_list_shows_bookmarks(void **state)
     window.type = WIN_CONSOLE;
 
     Bookmark *bm1 = malloc(sizeof(Bookmark));
-    bm1->jid = strdup("room1@conf.org");
+    bm1->barejid = strdup("room1@conf.org");
     bm1->nick = strdup("bob");
     bm1->autojoin = FALSE;
     Bookmark *bm2 = malloc(sizeof(Bookmark));
-    bm2->jid = strdup("room2@conf.org");
+    bm2->barejid = strdup("room2@conf.org");
     bm2->nick = strdup("steve");
     bm2->autojoin = TRUE;
     Bookmark *bm3 = malloc(sizeof(Bookmark));
-    bm3->jid = strdup("room3@conf.org");
+    bm3->barejid = strdup("room3@conf.org");
     bm3->nick = strdup("dave");
     bm3->autojoin = TRUE;
     Bookmark *bm4 = malloc(sizeof(Bookmark));
-    bm4->jid = strdup("room4@conf.org");
+    bm4->barejid = strdup("room4@conf.org");
     bm4->nick = strdup("james");
     bm4->autojoin = FALSE;
     Bookmark *bm5 = malloc(sizeof(Bookmark));
-    bm5->jid = strdup("room5@conf.org");
+    bm5->barejid = strdup("room5@conf.org");
     bm5->nick = strdup("mike");
     bm5->autojoin = FALSE;