Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[permissions] Clearing rules, doesn't allow command to be used #3037

Closed
kennnyshiwa opened this issue Oct 8, 2019 · 0 comments · Fixed by #3041
Closed

[permissions] Clearing rules, doesn't allow command to be used #3037

kennnyshiwa opened this issue Oct 8, 2019 · 0 comments · Fixed by #3041
Labels
Status: In Progress Type: Bug

Comments

@kennnyshiwa
Copy link
Contributor

@kennnyshiwa kennnyshiwa commented Oct 8, 2019

Command bugs

Command name

clearglobalrules or clearserverrules after setting a default server or default global rule

What cog is this command from?

permissions

What were you expecting to happen?

After clearing the rules for the command to work as intended

What actually happened?

Command was still denied as if the permissions were in place

How can we reproduce this issue?

Deny a command with permissions such as ping with [p]permissions setdefaultglobalrule deny ping

Then clear the rules with clearglobalrules

Try and have a user use the denied command, after the permissions were cleared and it will still be blocked, after a bot restart it will become available.

Only seems to occur if you use setdefaultglobalrule or setdefaultserverrule when setting the permissions as a normal server or global rule clears and allows the command to work,

Might just want to send a message after clearing rules to reboot the bot, or there could be another way of handling the clearing of permissions

@Flame442 Flame442 added the Type: Bug label Oct 8, 2019
@mikeshardmind mikeshardmind added the Status: In Progress label Oct 18, 2019
@Kowlin Kowlin closed this in #3041 Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: In Progress Type: Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants