Merge pull request #754 from nextcloud/bugfix/noid/correct-roomlist-sorting

Sort correctly room collection.

js/models/roomcollection.js

@@ -28,8 +28,15 @@
 
 	var RoomCollection = Backbone.Collection.extend({
 		model: OCA.SpreedMe.Models.Room,
-		comparator: function(model) {
-			return [model.get('active') ? -1 : 0, -(model.get('lastPing'))];
+		comparator: function(modelA, modelB) {
+			var	activeA = modelA.get('active'),
+				activeB = modelB.get('active');
+
+			if (activeA !== activeB) {
+				return activeB - activeA;
+			}
+
+			return modelB.get('lastPing') - modelA.get('lastPing');
 		},
 		url: OC.linkToOCS('apps/spreed/api/v1', 2) + 'room',
 		/**