Fix bugs caused by Map#set(k,v) returning undefined instead of v

lib/channel.js

@@ -120,7 +120,8 @@ function channel(name) {
     return cache.get(cid);
   }
   const chan = new Channel(name);
-  return cache.set(chan.id, chan);
+  cache.set(chan.id, chan);
+  return chan;
 }
 
 /** For joining channels

lib/irc.js

@@ -454,8 +454,11 @@ function matchPred(pred, handler) {
 
 function matchType(type, handler) {
   const handlers = this.handlers;
-  const arr = handlers.has(type) ? handlers.get(type) :
-    handlers.set(type, []);
+  let arr = handlers.get(type);
+  if (!arr) {
+    arr = [];
+    handlers.set(type, arr);
+  }
   arr.push(handler);
   return this.ignore.bind(this, type, handler);
 }

lib/person.js

@@ -45,7 +45,7 @@ Person.prototype.toString = function() {
        + this.user : "") + "@" + this.host : "")
 }
 
-property(Person.prototype, "id", function() { return id(this.nick) })
+property(Person.prototype, "id", function() { return id(this.nick) });
 
 /** Make a Person object
  *  @throws {Error} if no matching signature was found
@@ -55,13 +55,16 @@ property(Person.prototype, "id", function() { return id(this.nick) })
  *  @return {Person}
  */
 function person(nick, user, host) {
-  if (arguments.length === 0 || arguments.length > 3)
-    throw new Error("No matching signature")
-  const pid = id(nick)
-  if (personCache.has(pid))
-    return personCache.get(pid)
-  const p = new Person(nick, user || null, host || null)
-  return personCache.set(p.id, p)
+  if (arguments.length === 0 || arguments.length > 3) {
+    throw new Error("No matching signature");
+  }
+  const pid = id(nick);
+  if (personCache.has(pid)) {
+    return personCache.get(pid);
+  }
+  const p = new Person(nick, user || null, host || null);
+  personCache.set(p.id, p);
+  return p;
 }
 
 /** Send a {@link Message} to a {@link Person}.

lib/server.js

@@ -53,7 +53,8 @@ function server(name, port) {
     return serverCache.get(sid);
   }
   const server = new Server(name, port ? port : DEFAULT_PORT);
-  return serverCache.set(server.id, server);
+  serverCache.set(server.id, server);
+  return server;
 }
 
 exports.Server  = Server;