Use splice to avoid leaving empty nulls behind

earldouglas · Jan 7, 2017 · ef4618f · ef4618f
1 parent 312d08d
commit ef4618f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/listeners/message/auto-reply.js b/lib/listeners/message/auto-reply.js
@@ -53,7 +53,7 @@ function messageListener(db, from, channel, message) {
       var index =  findReply(name,replies);
 
       if (index != replies.length) {
-        delete replies[index]
+        replies.splice(index, 1);
       }
 
       replies[index] = { count: 10, name:name, regex: regex, flags: flags, reply: reply, };
@@ -80,7 +80,7 @@ function messageListener(db, from, channel, message) {
         replies[i].count = replies[i].count || 10;
         replies[i].count = replies[i].count - 1;
         if (replies[i].count <= 0) {
-          delete replies[i];
+          replies.splice(i, 1);
         }
       }
     }