Opened 9 years ago

Closed 9 years ago

Last modified 3 years ago

#4253 enhancement closed invalid (invalid)

CHANMODES parameter in ISUPPORT is parsed incorrectly

Reported by: ChosenOne Owned by:
Priority: normal Milestone: totally automated release infrastructure
Component: words Keywords: chanmodes, isupport
Cc: Branch:


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.

Attachments (1)

my-twisted-patch.patch (601 bytes) - added by ChosenOne 9 years ago.
patch for, which will fix the mentioned bug

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by ChosenOne

Attachment: my-twisted-patch.patch added

patch for, which will fix the mentioned bug

comment:1 Changed 9 years ago by ChosenOne

Keywords: review added
Owner: Jean-Paul Calderone deleted

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

comment:2 Changed 9 years ago by Jonathan Jacobs

Resolution: invalid
Status: newclosed

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.

comment:3 Changed 8 years ago by <automation>

comment:4 Changed 3 years ago by hawkowl

Keywords: review removed

[mass edit] Removing review from closed tickets.

Note: See TracTickets for help on using tickets.