Commit 72009672 authored by JC Brand's avatar JC Brand

converse-chatboxes: Don't refetch chatboxes upon reconnection

We don't remove the existing chatboxes and upon refetching they
become stale and get removed (by Backbone) and then the UI for
them becomes unresponsive.
parent c0ef008c
...@@ -1051,7 +1051,10 @@ converse.plugins.add('converse-chatboxes', { ...@@ -1051,7 +1051,10 @@ converse.plugins.add('converse-chatboxes', {
_converse.api.trigger('chatBoxesFetched'); _converse.api.trigger('chatBoxesFetched');
}, },
onConnected () { onConnected (reconnecting) {
if (reconnecting) {
return;
}
const storage = _converse.config.get('storage'); const storage = _converse.config.get('storage');
this.browserStorage = new BrowserStorage[storage]( this.browserStorage = new BrowserStorage[storage](
`converse.chatboxes-${_converse.bare_jid}`); `converse.chatboxes-${_converse.bare_jid}`);
...@@ -1265,7 +1268,7 @@ converse.plugins.add('converse-chatboxes', { ...@@ -1265,7 +1268,7 @@ converse.plugins.add('converse-chatboxes', {
_converse.api.trigger('chatBoxesInitialized'); _converse.api.trigger('chatBoxesInitialized');
}); });
_converse.api.listen.on('presencesInitialized', () => _converse.chatboxes.onConnected()); _converse.api.listen.on('presencesInitialized', (reconnecting) => _converse.chatboxes.onConnected(reconnecting));
_converse.api.listen.on('reconnected', () => _converse.chatboxes.forEach(m => m.onReconnection())); _converse.api.listen.on('reconnected', () => _converse.chatboxes.forEach(m => m.onReconnection()));
/************************ END Event Handlers ************************/ /************************ END Event Handlers ************************/
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment