search

WritheM / Wallace

https://plug.dj/writhem-radio
Other
4 stars 3 forks source link

Bot doesn't seem to like complex names #64

Closed Reanmachine closed 9 years ago

Reanmachine commented 9 years ago

Tried to

!add @§wimmadude66

The bot didn't respond or do anything, I think it's possible the § is fucking with the parsing, the name detection algorithm should probably be improved/looked at.

It seems the bot outright isn't accepting a name parameter, only !add works, but when you add a name it's just not taking it. Same with !remove

ylt commented 9 years ago

hmm, will look into it

Reanmachine commented 9 years ago

Updated ticket.

ylt commented 9 years ago

Think I know why, will make a fix tonight

ylt commented 9 years ago

[2015-08-10 11:09:24.153] [INFO] console - Event handler crashed [TypeError: Can not read property 'room' of undefined] TypeError: Cannot read property 'room' of undefined at Object.messageData.getUser (c:\Users\writhem\Desktop\WritheM Bot\coreplug ins\plug\plug.js:269:43) at PluginInstance.basicBot.events.command_add (c:\Users\writhem\Desktop\Writ heM Bot\plugins\basicBot\cmds\add.js:11:32) at PluginLoader.fireEvent (c:\Users\writhem\Desktop\WritheM Bot\core\PluginL oader.js:137:47) at PluginManager.fireEvent (c:\Users\writhem\Desktop\WritheM Bot\core\Plugin Manager.js:137:38) at EventProxy.chat (c:\Users\writhem\Desktop\WritheM Bot\coreplugins\plug\pl ug.js:274:37) at Plugin.<anonymous> (c:\Users\writhem\Desktop\WritheM Bot\coreplugins\plug \plug.js:125:44) at Plugged._emitChat (c:\Users\writhem\Desktop\WritheM Bot\coreplugins\plug\ node_modules\plugged\plugged.js:848:10) at Plugged._wsaprocessor (c:\Users\writhem\Desktop\WritheM Bot\coreplugins\p lug\node_modules\plugged\plugged.js:572:22) at WebSocket._receivedMessage (c:\Users\writhem\Desktop\WritheM Bot\coreplug ins\plug\node_modules\plugged\plugged.js:385:18) at WebSocket.emit (events.js:110:17)

TODO: https://github.com/WritheM/Wallace/blob/master/coreplugins/plug/plug.js#L257 add .bind(this)

ylt commented 9 years ago

Fixed in a7aac97, http://i.imgur.com/EY9Z4uy.png