better way of finding what channels a user is in

Generate the correct list of channels where the user is in (for
commands: QUIT, NICK and KILL)

Author: Andrei Husanu

lib/irc.js

@@ -313,12 +313,14 @@ function Client(server, nick, opt) {
 
                 channels = [];
 
-                // TODO better way of finding what channels a user is in?
+                // finding what channels a user is in
                 Object.keys(self.chans).forEach(function(channame) {
                     var channel = self.chans[channame];
-                    channel.users[message.args[0]] = channel.users[message.nick];
-                    delete channel.users[message.nick];
-                    channels.push(channame);
+                    if (message.nick in channel.users) {
+                        channel.users[message.args[0]] = channel.users[message.nick];
+                        delete channel.users[message.nick];
+                        channels.push(channame);
+                    }
                 });
 
                 // old nick, new nick, channels
@@ -501,8 +503,10 @@ function Client(server, nick, opt) {
                 channels = [];
                 Object.keys(self.chans).forEach(function(channame) {
                     var channel = self.chans[channame];
-                    channels.push(channame);
-                    delete channel.users[nick];
+                    if (nick in channel.users) {
+                        channels.push(channame);
+                        delete channel.users[nick];
+                    }
                 });
                 self.emit('kill', nick, message.args[1], channels, message);
                 break;
@@ -544,11 +548,13 @@ function Client(server, nick, opt) {
 
                 channels = [];
 
-                // TODO better way of finding what channels a user is in?
+                // finding what channels a user is in?
                 Object.keys(self.chans).forEach(function(channame) {
                     var channel = self.chans[channame];
-                    delete channel.users[message.nick];
-                    channels.push(channame);
+                    if (message.nick in channel.users) {
+                        delete channel.users[message.nick];
+                        channels.push(channame);
+                    }
                 });
 
                 // who, reason, channels

package.json

@@ -14,7 +14,8 @@
     "Justin Gallardo <justin.gallardo@gmail.com>",
     "Chris Nehren <cnehren@pobox.com>",
     "Henri Niemeläinen <aivot-on@iki.fi>",
-    "Alex Miles <ghostaldev@gmail.com>"
+    "Alex Miles <ghostaldev@gmail.com>",
+    "Andrei Husanu <node-irc@andreihusanu.ro>"
   ],
   "repository": {
     "type": "git",