Changes in storage_xml.c [d628339:098a75b]

File:

• storage_xml.c (modified) (6 diffs)

storage_xml.c

@@ -83 +83 @@
 }
 
-/* Use for unsupported/not-found protocols. Save settings as-is but don't allow changes. */
-static void handle_settings_raw(struct xt_node *node, set_t **head)
-{
-        struct xt_node *c;
-
-        for (c = node->children; (c = xt_find_node(c, "setting")); c = c->next) {
-                char *name = xt_find_attr(c, "name");
-
-                if (!name) {
-                        continue;
-                }
-
-                set_t *s = set_add(head, name, NULL, NULL, NULL);
-                set_setstr(head, name, c->text);
-                s->flags |= SET_HIDDEN |
-                            ACC_SET_OFFLINE_ONLY | ACC_SET_ONLINE_ONLY;
-        }
-}
-
 static xt_status handle_account(struct xt_node *node, gpointer data)
 {
@@ -108 +89 @@
         char *pass_b64 = NULL;
         unsigned char *pass_cr = NULL;
-        int pass_len;
+        int pass_len, local = 0;
         struct prpl *prpl = NULL;
         account_t *acc;
@@ -123 +104 @@
                 prpl = find_protocol(protocol);
                 if (!prpl) {
-                        irc_rootmsg(xd->irc, "Warning: Protocol not found: `%s'", protocol);
-                        prpl = (struct prpl*) &protocol_missing;
-                }
+                        irc_rootmsg(xd->irc, "Error loading user config: Protocol not found: `%s'", protocol);
+                        return XT_ABORT;
+                }
+                local = protocol_account_islocal(protocol);
         }
 
@@ -142 +124 @@
                         set_setstr(&acc->set, "tag", tag);
                 }
-                if (prpl == &protocol_missing) {
-                        set_t *s = set_add(&acc->set, "_protocol_name", protocol, NULL, NULL);
-                        s->flags |= SET_HIDDEN | SET_NOSAVE |
-                                    ACC_SET_OFFLINE_ONLY | ACC_SET_ONLINE_ONLY;
+                if (local) {
+                        acc->flags |= ACC_FLAG_LOCAL;
                 }
         } else {
@@ -156 +136 @@
         g_free(password);
 
-        if (prpl == &protocol_missing) {
-                handle_settings_raw(node, &acc->set);
-        } else {
-                handle_settings(node, &acc->set);
-        }
+        handle_settings(node, &acc->set);
 
         for (c = node->children; (c = xt_find_node(c, "buddy")); c = c->next) {
@@ -336 +312 @@
 
                 cur = xt_new_node("account", NULL, NULL);
-                if (acc->prpl == &protocol_missing) {
-                        xt_add_attr(cur, "protocol", set_getstr(&acc->set, "_protocol_name"));
-                } else {
-                        xt_add_attr(cur, "protocol", acc->prpl->name);
-                }
+                xt_add_attr(cur, "protocol", acc->prpl->name);
                 xt_add_attr(cur, "handle", acc->user);
                 xt_add_attr(cur, "password", pass_b64);