From 158abd7526e596e6b76f5bf8ac45cb2422ac0e70 Mon Sep 17 00:00:00 2001
From: Mattias Erming <mattias@mattiaserming.com>
Date: Sat, 15 Mar 2014 02:19:50 +0100
Subject: [PATCH] Fixed issues with NICK events

---
 client/js/chat.js |  2 +-
 lib/server.js     | 39 +++++++++++++++++++++++++++------------
 2 files changed, 28 insertions(+), 13 deletions(-)

diff --git a/client/js/chat.js b/client/js/chat.js
index a2f3fbc..6b6e5e9 100644
--- a/client/js/chat.js
+++ b/client/js/chat.js
@@ -160,7 +160,7 @@ $(function() {
 	};
 
 	$.fn.scrollToBottom = function() {
-		this.scrollTop(1e10);
+		this.scrollTop(this.prop("scrollHeight"));
 	};
 
 	$.fn.isScrollAtBottom = function() {
diff --git a/lib/server.js b/lib/server.js
index ca3a000..8bb9953 100644
--- a/lib/server.js
+++ b/lib/server.js
@@ -362,22 +362,29 @@ function handleEvent(argv) {
 		break;
 
 	case "NICK":
-		var message = new models.Message({
-			user: argv.nick,
-			text: "changed name to " + argv.args[0],
-			type: "nick"
-		});
-
 		channels.each(function(channel) {
-			var user = channel.get("users").findWhere({name: argv.nick});
+			var users = channel.get("users");
+			var messages = channel.get("messages");
+
+			var user = users.findWhere({name: argv.nick});
 			if (typeof user !== "undefined") {
 				user.set("name", argv.args[0]);
-				channel.get("messages").add(message);
+
+				users.sort();
+				users.trigger(
+					"change", {}, users
+				);
+
+				messages.add(new models.Message({
+					user: argv.nick,
+					text: "changed name to " + argv.args[0],
+					type: "nick"
+				}));
 			}
 		});
 
 		if (argv.nick == network.get("nick")) {
-			network.set("nick", argv.args[0]);
+			network.set({nick: argv.args[0]}, {silent: true});
 		}
 		break;
 
@@ -491,14 +498,21 @@ function handleEvent(argv) {
 		if (typeof rpl_namreply === "undefined") {
 			channel.get("users").reset({silent: true});
 		}
+		
+		// This variable is global but will be deleted when
+		// rpl_endofnames is triggered..
 		rpl_namreply = true;
 
 		var names = argv.args[3].split(' ');
 		for (var i in names) {
+			var mode = (names[i].match(/^\W/) || "").toString();
+			var user = new models.User({
+				mode: mode,
+				name: names[i].replace(mode, '')
+			});
 			users.add(
-				new models.User({name: names[i]}), {
-					silent: true
-				}
+				user,
+				{silent: true}
 			);
 		}
 		break;
@@ -507,6 +521,7 @@ function handleEvent(argv) {
 		var channel = channels.findWhere({name: argv.args[1]});
 		var users = channel.get("users");
 		
+		// .. here!
 		delete rpl_namreply;
 		users.trigger(
 			"add", {}, users