Supybot / Patches / #215 Crash for commands with ambiguous getopts shortcuts and no docstring

Crash for commands with ambiguous getopts shortcuts and no docstring

Status: Abandoned

Brought to you by: ddipaolo, jamessan, jemfinch

#215 Crash for commands with ambiguous getopts shortcuts and no docstring

Status: closed

Owner: James McCoy

Labels: None

Priority: 2

Updated: 2015-03-26

Created: 2015-03-03

Creator: Valentin Lorentz

Private: No

An undefined variable was used in this part of the code that is never tested and almost never used in practice.

This patch fixes this issue.

https://github.com/ProgVal/Limnoria/commit/6f9960d7a4ec4ae2e0eb7ebe82006a1d5c91d554

Note: the 'checkDoc' attribute to wrap() has been introduced in Limnoria (setting it to False makes the behavior of wrap() similar to Supybot, ie. allowing commands with no docstring), so you have to remove it before merging the patch.

Discussion

James McCoy - 2015-03-26

assigned_to: James McCoy
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

James McCoy - 2015-03-26

status: open --> accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

James McCoy - 2015-03-26

Applied, along with the subsequent test fix.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

James McCoy - 2015-03-26

status: accepted --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Crash for commands with ambiguous getopts shortcuts and no docstring

Searches

Help

#215 Crash for commands with ambiguous getopts shortcuts and no docstring

Discussion