CHANMODES parameter in ISUPPORT is parsed incorrectly

[ChosenOne]

The CHANMODES parameter is currently parsed incorrectly by isupport_CHANMODES. The modeline is a comma-seperated list of letters, which gives you modes who require: - an address on setting and unsetting - a parameter on setting and unsetting - a parameter only on setting - no parameter at all.

(i.e. ('addressModes', 'param', 'setParam', 'noParam'))

Currently isupport_CHANMODES will disregard the comma and stuff all the modes into a dict which results to wrong output.

Instead, we should split the CHANMODES parameter at the comma and then build a dict according to the elements we get.

[ChosenOne]

patch for irc.py, which will fix the mentioned bug

[ChosenOne]

The patch will assign the parameters its correct key in the returned dict.

[jonathanj]

This patch seems like it would introduce an AttributeError attempting to find an attribute 'split' on a list instance. The splitting this patch attempts to do is already handled by ServerSupportedFeatures.parse.

After discussing this with the original submitter, it seems that there was simply some confusion between attempting to implement CHANMODES handling themselves and their understand of Twisted's handling of it.