Ticket #4253 enhancement closed invalid

Opened 4 years ago

Last modified 4 years ago

CHANMODES parameter in ISUPPORT is parsed incorrectly

Reported by: ChosenOne Owned by:
Priority: normal Milestone: totally automated release infrastructure
Component: words Keywords: chanmodes isupport review
Cc: Branch:
Author: Launchpad Bug:

Description

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

my-twisted-patch.patch Download (0.6 KB) - added by ChosenOne 4 years ago.
patch for irc.py, which will fix the mentioned bug

Change History

Changed 4 years ago by ChosenOne

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

1

Changed 4 years ago by ChosenOne

  • owner exarkun deleted
  • keywords review added

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

2

Changed 4 years ago by jonathanj

  • status changed from new to closed
  • resolution set to invalid

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.

3

Changed 3 years ago by <automation>

Note: See TracTickets for help on using tickets.