From 98191056e15e1d6a45e998a859ea927d2607f0d5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guillermo=20Bonveh=C3=AD?= <gbonvehi@gmail.com>
Date: Thu, 16 Apr 2015 01:23:13 -0300
Subject: [PATCH 1/8] Fix rooms grouping examples in docs

---
 docs/source/development.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/source/development.rst b/docs/source/development.rst
index 5be86fba4..118baaf5f 100644
--- a/docs/source/development.rst
+++ b/docs/source/development.rst
@@ -369,7 +369,7 @@ Similar to chats.get API
 
 To open a single multi user chat box, provide the JID of the room::
 
-    converse.room.open('group@muc.example.com')
+    converse.rooms.open('group@muc.example.com')
 
 To return an array of chat boxes, provide an array of JIDs::
 
@@ -377,7 +377,7 @@ To return an array of chat boxes, provide an array of JIDs::
 
 To setup a custom nickname when joining the room, provide the optionnal nick argument::
 
-    converse.room.open('group@muc.example.com', 'mycustomnick')
+    converse.rooms.open('group@muc.example.com', 'mycustomnick')
 
 
 "settings" grouping

From ba85e0733ff1d9cbe36ba9e2dc4e0b1a6f094460 Mon Sep 17 00:00:00 2001
From: JC Brand <jc@opkode.com>
Date: Thu, 16 Apr 2015 08:38:38 +0200
Subject: [PATCH 2/8] Remove anon login settings

---
 dev.html | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/dev.html b/dev.html
index 82eb6c41e..ac432e88c 100644
--- a/dev.html
+++ b/dev.html
@@ -52,11 +52,6 @@
 <script>
     require(['converse'], function (converse) {
         converse.initialize({
-            auto_login: true,
-            allow_registration: false,
-            allow_logout: false,
-            authentication: "anonymous",
-            jid: "opkode.im",
             bosh_service_url: 'https://conversejs.org/http-bind/', // Please use this connection manager only for testing purposes
             i18n: locales['en'], // Refer to ./locale/locales.js to see which locales are supported
             keepalive: true,

From 292d4a1bc993f30160f6b824c5e909fa09fb4a75 Mon Sep 17 00:00:00 2001
From: JC Brand <jc@opkode.com>
Date: Thu, 16 Apr 2015 09:37:25 +0200
Subject: [PATCH 3/8] Ignore messages sent to a different resource

---
 converse.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/converse.js b/converse.js
index 84b03a997..c7f7343fa 100644
--- a/converse.js
+++ b/converse.js
@@ -3096,7 +3096,13 @@
                 var contact_jid, $forwarded, $received, $sent,
                     msgid = $message.attr('id'),
                     chatbox, resource, roster_item,
-                    message_from = $message.attr('from');
+                    message_from = $message.attr('from'),
+                    message_to = $message.attr('to');
+
+                if(!_.contains([converse.connection.jid, converse.bare_jid], message_to)) {
+                    // Ignore messages sent to a different resource
+                    return true;
+                }
                 if (message_from === converse.connection.jid) {
                     // FIXME: Forwarded messages should be sent to specific resources,
                     // not broadcasted

From 63e3794ba71121b30d27fb60df7c7c5de4ee1aee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20Cargo=C3=ABt?= <florian.cargoet@gmail.com>
Date: Fri, 17 Apr 2015 16:24:02 +0200
Subject: [PATCH 4/8] Fixes #372

---
 converse.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/converse.js b/converse.js
index c7f7343fa..91b5f07a0 100644
--- a/converse.js
+++ b/converse.js
@@ -4003,10 +4003,12 @@
             },
 
             show: function () {
-                // FIXME: There's a bug here, if show_only_online_users is true
-                // Possible solution, get the group, call _.each and check
-                // showInRoster
-                this.$el.nextUntil('dt').addBack().show();
+                this.$el.show();
+                _.each(this.getAll(), function (contactView) {
+                    if (contactView.model.showInRoster()) {
+                        contactView.$el.show();
+                    }
+                });
             },
 
             hide: function () {

From 8b695be4d7837d5f1496529e0c55d021b10c051d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20Cargo=C3=ABt?= <florian.cargoet@gmail.com>
Date: Fri, 17 Apr 2015 17:02:16 +0200
Subject: [PATCH 5/8] Fix group visibility on page load.

Close #374
---
 converse.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/converse.js b/converse.js
index 91b5f07a0..7a97a91e3 100644
--- a/converse.js
+++ b/converse.js
@@ -3978,7 +3978,7 @@
                 if (contact.showInRoster()) {
                     if (this.model.get('state') === CLOSED) {
                         if (view.$el[0].style.display !== "none") { view.$el.hide(); }
-                        if (this.$el[0].style.display === "none") { this.$el.show(); }
+                        if (!this.$el.is(':visible')) { this.$el.show(); }
                     } else {
                         if (this.$el[0].style.display !== "block") { this.show(); }
                     }

From 50ef156a6dae67d5bbcdee09d195a07f47853788 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20Cargo=C3=ABt?= <florian.cargoet@gmail.com>
Date: Fri, 17 Apr 2015 17:04:59 +0200
Subject: [PATCH 6/8] Add changelog entry for #374

---
 docs/CHANGES.rst | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/docs/CHANGES.rst b/docs/CHANGES.rst
index d50501776..08f729e92 100644
--- a/docs/CHANGES.rst
+++ b/docs/CHANGES.rst
@@ -6,6 +6,7 @@ Changelog
 
 * Add the ability to log in anonymously. [jcbrand]
 * Add the ability to log in automatically. [jcbrand]
+* #374 Fix collapsed group visibility on page load. [floriancargoet]
 
 0.9.2 (2015-04-09)
 ------------------
@@ -27,7 +28,7 @@ Changelog
 * Set the JID input field in the login form to ``type=email``. [chatme]
 * New configuration setting `allow_contact_removal <https://conversejs.org/docs/html/configuration.html#allow-contact-removal>`_ [jcbrand]
 * Document that event handlers receive 'event' obj as first arg. [jcbrand]
-* Add a test to check that notifications are played in chat rooms. [jcbrand] 
+* Add a test to check that notifications are played in chat rooms. [jcbrand]
 * #333 Enable automatic reconnection when ``prebind`` and ``prebind_url`` are specified. [jcbrand]
 * #339 Require the JID to be specified when using ``keepalive`` with ``prebind``. Also add a logout API method. [jcbrand]
 * #349 Indicate visitors in chat rooms. [c143]
@@ -214,7 +215,7 @@ Changelog
     to fetch the newest strophe.muc.plugin (for bugfix of #85).
 
     This release contains 3 different builds:
-    - converse.min.js 
+    - converse.min.js
     - converse-no-otr.min.js (Without OTR encryption)
     - converse-no-locales-no-otr.min.js (Without OTR encryption or any translations)
 

From b08776a1ab7cc98e4d059e5608d6aa042b838e52 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guillermo=20Bonveh=C3=AD?= <gbonvehi@mpf.gov.ar>
Date: Tue, 21 Apr 2015 10:22:48 -0300
Subject: [PATCH 7/8] Added form_type setting tests

Two tests were created to see if RegisterPanel.form_type is set
to correctly to legacy or xform when receiving server registration
form.
---
 spec/register.js | 70 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)

diff --git a/spec/register.js b/spec/register.js
index 2e8b11d75..c9aafe22d 100644
--- a/spec/register.js
+++ b/spec/register.js
@@ -160,5 +160,75 @@
             expect(registerview.$('input[type=button]').length).toBe(1);
         }, converse));
 
+        it("will set form_type to legacy", $.proxy(function () {
+            var cbview = this.chatboxviews.get('controlbox');
+            cbview.$('#controlbox-tabs').find('li').last().find('a').click(); // Click the Register tab
+            var registerview = this.chatboxviews.get('controlbox').registerpanel;
+            spyOn(registerview, 'onProviderChosen').andCallThrough();
+            spyOn(registerview, 'getRegistrationFields').andCallThrough();
+            spyOn(registerview, 'onRegistrationFields').andCallThrough();
+            spyOn(registerview, 'renderRegistrationForm').andCallThrough();
+            registerview.delegateEvents();  // We need to rebind all events otherwise our spy won't be called
+            spyOn(this.connection, 'connect').andCallThrough();
+
+            registerview.$('input[name=domain]').val('conversejs.org');
+            registerview.$('input[type=submit]').click();
+
+            var stanza = new Strophe.Builder("stream:features", {
+                        'xmlns:stream': "http://etherx.jabber.org/streams",
+                        'xmlns': "jabber:client"
+                    })
+                .c('register',  {xmlns: "http://jabber.org/features/iq-register"}).up()
+                .c('mechanisms', {xmlns: "urn:ietf:params:xml:ns:xmpp-sasl"});
+            this.connection._connect_cb(test_utils.createRequest(stanza));
+            stanza = $iq({
+                    'type': 'result',
+                    'id': 'reg1'
+                }).c('query', {'xmlns': 'jabber:iq:register'})
+                    .c('instructions')
+                        .t('Please choose a username, password and provide your email address').up()
+                    .c('username').up()
+                    .c('password').up()
+                    .c('email');
+            this.connection._dataRecv(test_utils.createRequest(stanza));
+            expect(registerview.form_type).toBe('legacy');
+        }, converse));
+
+        it("will set form_type to xform", $.proxy(function () {
+            var cbview = this.chatboxviews.get('controlbox');
+            cbview.$('#controlbox-tabs').find('li').last().find('a').click(); // Click the Register tab
+            var registerview = this.chatboxviews.get('controlbox').registerpanel;
+            spyOn(registerview, 'onProviderChosen').andCallThrough();
+            spyOn(registerview, 'getRegistrationFields').andCallThrough();
+            spyOn(registerview, 'onRegistrationFields').andCallThrough();
+            spyOn(registerview, 'renderRegistrationForm').andCallThrough();
+            registerview.delegateEvents();  // We need to rebind all events otherwise our spy won't be called
+            spyOn(this.connection, 'connect').andCallThrough();
+
+            registerview.$('input[name=domain]').val('conversejs.org');
+            registerview.$('input[type=submit]').click();
+
+            var stanza = new Strophe.Builder("stream:features", {
+                        'xmlns:stream': "http://etherx.jabber.org/streams",
+                        'xmlns': "jabber:client"
+                    })
+                .c('register',  {xmlns: "http://jabber.org/features/iq-register"}).up()
+                .c('mechanisms', {xmlns: "urn:ietf:params:xml:ns:xmpp-sasl"});
+            this.connection._connect_cb(test_utils.createRequest(stanza));
+            stanza = $iq({
+                    'type': 'result',
+                    'id': 'reg1'
+                }).c('query', {'xmlns': 'jabber:iq:register'})
+                    .c('instructions')
+                        .t('Using xform data').up()
+                    .c('x', { 'xmlns': 'jabber:x:data', 'type': 'form' })
+                        .c('instructions').t('xform instructions').up()
+                        .c('field', {'type': 'text-single', 'var': 'username'}).c('required').up().up()
+                        .c('field', {'type': 'text-private', 'var': 'password'}).c('required').up().up()
+                        .c('field', {'type': 'text-single', 'var': 'email'}).c('required').up().up()
+            this.connection._dataRecv(test_utils.createRequest(stanza));
+            expect(registerview.form_type).toBe('xform');
+        }, converse));
+
     }, converse, mock, test_utils));
 }));

From 8934043d792bf3d942e42a56365d8ebba30c3dcc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guillermo=20Bonveh=C3=AD?= <gbonvehi@mpf.gov.ar>
Date: Tue, 21 Apr 2015 11:05:03 -0300
Subject: [PATCH 8/8] Submit registration form as xform or legacy

Support to send the registration form as legacy or xform, we
use the same variable form_type that was already being used when
calling render
Added submission tests
---
 converse.js      | 16 ++++++++++++----
 spec/register.js | 31 +++++++++++++++++++++++++++++--
 2 files changed, 41 insertions(+), 6 deletions(-)

diff --git a/converse.js b/converse.js
index 7a97a91e3..76ab57c7c 100644
--- a/converse.js
+++ b/converse.js
@@ -5060,11 +5060,19 @@
                 var $inputs = $(ev.target).find(':input:not([type=button]):not([type=submit])'),
                     iq = $iq({type: "set"})
                         .c("query", {xmlns:Strophe.NS.REGISTER})
-                        .c("x", {xmlns: Strophe.NS.XFORM, type: 'submit'});
 
-                $inputs.each(function () {
-                    iq.cnode(utils.webForm2xForm(this)).up();
-                });
+                if (this.form_type == 'xform') {
+                    iq.c("x", {xmlns: Strophe.NS.XFORM, type: 'submit'});
+                    $inputs.each(function () {
+                        iq.cnode(utils.webForm2xForm(this)).up();
+                    });
+                } else {
+                    $inputs.each(function () {
+                        var $input = $(this);
+                        iq.c($input.attr('name'), {}, $input.val());
+                    });
+                }
+
                 converse.connection._addSysHandler(this._onRegisterIQ.bind(this), null, "iq", null, null);
                 converse.connection.send(iq);
                 this.setFields(iq.tree());
diff --git a/spec/register.js b/spec/register.js
index c9aafe22d..df2410225 100644
--- a/spec/register.js
+++ b/spec/register.js
@@ -160,7 +160,7 @@
             expect(registerview.$('input[type=button]').length).toBe(1);
         }, converse));
 
-        it("will set form_type to legacy", $.proxy(function () {
+        it("will set form_type to legacy and submit it as legacy", $.proxy(function () {
             var cbview = this.chatboxviews.get('controlbox');
             cbview.$('#controlbox-tabs').find('li').last().find('a').click(); // Click the Register tab
             var registerview = this.chatboxviews.get('controlbox').registerpanel;
@@ -192,9 +192,22 @@
                     .c('email');
             this.connection._dataRecv(test_utils.createRequest(stanza));
             expect(registerview.form_type).toBe('legacy');
+
+            registerview.$('input[name=username]').val('testusername');
+            registerview.$('input[name=password]').val('testpassword');
+            registerview.$('input[name=email]').val('test@email.local');
+
+            spyOn(converse.connection, 'send');
+
+            registerview.$('input[type=submit]').click();
+
+            expect(converse.connection.send).toHaveBeenCalled();
+            var $stanza = $(converse.connection.send.argsForCall[0][0].tree());
+            expect($stanza.children('query').children().length).toBe(3);
+            expect($stanza.children('query').children()[0].tagName).toBe('username');
         }, converse));
 
-        it("will set form_type to xform", $.proxy(function () {
+        it("will set form_type to xform and submit it as xform", $.proxy(function () {
             var cbview = this.chatboxviews.get('controlbox');
             cbview.$('#controlbox-tabs').find('li').last().find('a').click(); // Click the Register tab
             var registerview = this.chatboxviews.get('controlbox').registerpanel;
@@ -228,6 +241,20 @@
                         .c('field', {'type': 'text-single', 'var': 'email'}).c('required').up().up()
             this.connection._dataRecv(test_utils.createRequest(stanza));
             expect(registerview.form_type).toBe('xform');
+
+            registerview.$('input[name=username]').val('testusername');
+            registerview.$('input[name=password]').val('testpassword');
+            registerview.$('input[name=email]').val('test@email.local');
+
+            spyOn(converse.connection, 'send');
+
+            registerview.$('input[type=submit]').click();
+
+            expect(converse.connection.send).toHaveBeenCalled();
+            var $stanza = $(converse.connection.send.argsForCall[0][0].tree());
+            expect($stanza.children('query').children().length).toBe(1);
+            expect($stanza.children('query').children().children().length).toBe(3);
+            expect($stanza.children('query').children().children()[0].tagName).toBe('field');
         }, converse));
 
     }, converse, mock, test_utils));