ircd.js
ircd.js copied to clipboard
Fix: Don't crash on kick without user
If you do "/KICK " and don't specify a user the server crashes
home/ben/workspace/ircd.js/lib/commands.js:123
userNames = users.split(','),
^
TypeError: Cannot call method 'split' of undefined
at Object.Commands.KICK (/home/ben/workspace/ircd.js/lib/commands.js:123:27)
at Object.Server.respondToMessage (/home/ben/workspace/ircd.js/lib/server.js:171:36)
at Object.Server.respond (/home/ben/workspace/ircd.js/lib/server.js:181:14)
at Object.Server.data (/home/ben/workspace/ircd.js/lib/server.js:320:10)
at Carrier.
Doesn't happen anymore with this fix and just sends an "needs more arguments"
Good catch, thanks for the fix. Your patch seems to alter the indentation of several lines. Could you please clean that up?
Just seems like it. The whole part between l137-160 is now within the "else"-block and therefore the indentation moved a little.
I (well, my editor :P) also removed the trailing whitespaces (l336 & l498).
Hope that makes sense
Can you merge this please???? @sespindola
lol is there a way i can participate in this? otherwise i'll gladly fork this as i've been looking for a node.js implementation of irc