ayust / kitnirc

Python IRC bot framework.
MIT License
11 stars 7 forks source link

Problems related to treating channel names as if they are case-sensitive, or something like that #16

Closed flexd closed 10 years ago

flexd commented 10 years ago

This suddenly happens, when kicking someone, but things have been only semi-working recently. I am not sure what happened on the server but the channel name suddenly seems to be all capitalised, while in my configs it is not capitalised, and that is causing stuff to break. Error below:

ERROR 2014-08-12 13:08:29,918 kitnirc.client:0222 - Error while processing event 'LINE': KeyError('#yamc',)
Traceback (most recent call last):
  File "/home/kristoffer/.virtualenvs/yamcbot/src/kitnirc/kitnirc/client.py", line 217, in dispatch_event
    if handler(self, *args):
  File "/home/kristoffer/.virtualenvs/yamcbot/src/kitnirc/kitnirc/client.py", line 590, in on_line
    parser(client, command, actor, args)
  File "/home/kristoffer/.virtualenvs/yamcbot/src/kitnirc/kitnirc/client.py", line 708, in _parse_kick
    channel = client.server.channels[channel]
KeyError: '#yamc'
ayust commented 10 years ago

I think this (and other similar issues) should be resolved by 21a951954ba1a1b5d9acd5bcd740e46bb3a6b392.

flexd commented 10 years ago

Sort of working, but this might be remaining:

ERROR 2014-08-13 12:23:35,984 kitnirc.client:0237 - Error while processing event 'LINE': KeyError('#YAMC',)
Traceback (most recent call last):
  File "/home/kristoffer/.virtualenvs/yamcbot/local/lib/python2.7/site-packages/kitnirc/client.py", line 232, in dispatch_event
    if handler(self, *args):
  File "/home/kristoffer/.virtualenvs/yamcbot/local/lib/python2.7/site-packages/kitnirc/client.py", line 599, in on_line
    parser(client, command, actor, args)
  File "/home/kristoffer/.virtualenvs/yamcbot/local/lib/python2.7/site-packages/kitnirc/client.py", line 862, in _parse_mode
    user = client.server.channels[channel].members[argument]
KeyError: '#YAMC'
ayust commented 10 years ago

8cd88213e2a44c73c17ab0c7023c5408b255c596 should resolve that.

flexd commented 10 years ago

Seems to work. :+1: