diff --git a/public/src/modules/chat.js b/public/src/modules/chat.js
index 5818c36b03..476718aa9a 100644
--- a/public/src/modules/chat.js
+++ b/public/src/modules/chat.js
@@ -1,3 +1,7 @@
+
+'use strict';
+/* globals app, config, define, socket, translator, templates, utils */
+
define(['taskbar', 'string', 'sound'], function(taskbar, S, sound) {
var module = {};
@@ -5,42 +9,45 @@ define(['taskbar', 'string', 'sound'], function(taskbar, S, sound) {
module.prepareDOM = function() {
// Chats Dropdown
var chatsToggleEl = $('#chat_dropdown'),
- chatsListEl = $('#chat-list'),
- chatDropdownEl = chatsToggleEl.parent();
+ chatsListEl = $('#chat-list');
chatsToggleEl.on('click', function() {
- if (chatDropdownEl.hasClass('open')) {
+ if (chatsToggleEl.parent().hasClass('open')) {
return;
}
socket.emit('modules.chats.list', function(err, chats) {
- var numChats = chats.length,
- chatEl, x, userObj;
+ if (err) {
+ return app.alertError(err.message);
+ }
+
+ var userObj;
+
chatsListEl.empty();
- if (!err && numChats > 0) {
-
- for(x = 0;x < numChats; x++) {
- userObj = chats[x];
- chatEl = $('
')
- .attr('data-uid', userObj.uid)
- .html('
' + userObj.username + '');
-
- chatsListEl.append(chatEl);
- }
- } else {
+
+ if (!chats.length) {
translator.get('modules:chat.no_active', function(str) {
- chatEl = $('')
+ $('')
.addClass('no_active')
- .html('' + str + '');
- chatsListEl.append(chatEl);
+ .html('' + str + '')
+ .appendTo(chatsListEl);
});
+ return;
+ }
+
+ for(var x = 0; x')
+ .attr('data-uid', userObj.uid)
+ .html('
' + userObj.username + '')
+ .appendTo(chatsListEl);
}
});
});
@@ -72,7 +79,7 @@ define(['taskbar', 'string', 'sound'], function(taskbar, S, sound) {
sound.play('chat-incoming');
});
- }
+ };
module.bringModalToTop = function(chatModal) {
var topZ = 0;
@@ -114,7 +121,7 @@ define(['taskbar', 'string', 'sound'], function(taskbar, S, sound) {
module.createModal = function(username, touid, callback) {
templates.preload_template('chat', function() {
- translator.translate(templates['chat'].parse({}), function (chatTpl) {
+ translator.translate(templates.chat.parse({}), function (chatTpl) {
var chatModal = $(chatTpl),
uuid = utils.generateUUID();
diff --git a/public/src/templates.js b/public/src/templates.js
index 2e31e6a4ab..0ee0c13339 100644
--- a/public/src/templates.js
+++ b/public/src/templates.js
@@ -174,7 +174,7 @@
value = $(element).val();
break;
}
-
+
templates.set($(element).attr('template-variable'), value);
});
};