Bugfix. Only unauthorise user once removed from the roster.

47380d08 · JC Brand · 588825e0 · 47380d08
Commit 47380d08 authored Oct 19, 2012 by JC Brand
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 4 deletions

converse.js converse.js +5 -4

No files found.
--- a/converse.js
+++ b/converse.js
@@ -954,9 +954,10 @@
                    "Remove": function() {
                        var bare_jid = that.model.get('bare_jid');
                        $(this).dialog( "close" );
-                        xmppchat.connection.roster.unauthorize(that.model.get('jid'));
+                        xmppchat.connection.roster.remove(bare_jid, function (iq) {
-                        xmppchat.roster.remove(bare_jid);
+                            xmppchat.connection.roster.unauthorize(bare_jid);
-                        xmppchat.connection.roster.remove(bare_jid);
+                            xmppchat.roster.remove(bare_jid);
+                        });
                    },
                    "Cancel": function() {
                        $(this).dialog( "close" );
@@ -970,7 +971,7 @@
            xmppchat.connection.roster.authorize(jid);
            xmppchat.connection.roster.add(jid, this.model.get('fullname'), [], function (iq) {
                xmppchat.connection.roster.subscribe(jid);
-            }, this);
+            });
        },
        declineRequest: function () {