Opened 7 years ago

Closed 6 years ago

#18187 closed Cleanup/optimization (fixed)

In adding options to custom commands the doc stop too early and therefor miss an important information

Reported by: purple@… Owned by: nobody
Component: Documentation Version: 1.4
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


Here: starting here:
"The same closepoll could be easily modified to delete a given poll instead of closing it by accepting additional command line options. These custom options ..."

The documentation doesn't say anywhere that the option will be stored in the dict "options" pass as argument of the method. As obvious as it sounds I've add to google it to find this information and apparently I'm far from being the only one: (it's sad that no one since 2010 has repported this bug :/).

Documentation should always state the obvious.

Kind regards,

Attachments (1)

18187-1.diff (749 bytes) - added by Claude Paroz 7 years ago.
Added paragraph about command options

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by Claude Paroz

Attachment: 18187-1.diff added

Added paragraph about command options

comment:1 Changed 7 years ago by Claude Paroz

Has patch: set
Triage Stage: UnreviewedAccepted
Type: BugCleanup/optimization

See the attached patch. Is it what you were searching for?

comment:2 Changed 7 years ago by purple@…

Yes, thanks!

To be perfect you could add an example but this isn't mandatory.

comment:3 Changed 6 years ago by Claude Paroz

Resolution: fixed
Status: newclosed

In [17930]:

Fixed #18187 -- Completed example and details about custom command options. Thanks purple for the report.

Note: See TracTickets for help on using tickets.
Back to Top