Skip to content

Superstarify Not Appropriately Capturing Reason String #731

@sco1

Description

@sco1

The superstarify command function definition isn't correctly capturing the (optional) reason string, causing the command set the first word as the reason, if specified, or raising an error if the first word contains an apostrophe:

@command(name="superstarify", aliases=("force_nick", "star"))
async def superstarify(
self,
ctx: Context,
member: Member,
duration: Expiry,
reason: str = None
) -> None:

Changing to:

    @command(name="superstarify", aliases=("force_nick", "star"))
    async def superstarify(
        self,
        ctx: Context,
        member: Member,
        duration: utils.Expiry,
        *,
        reason: str = None
    ) -> None:

Should capture the entire reason string & resolve the issue.

Metadata

Metadata

Assignees

No one assigned

    Labels

    a: moderationRelated to community moderation functionality: (moderation, defcon, verification)p: 2 - normalNormal Priorityt: bugSomething isn't working

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions