CivClassic / Kira

Discord bot for interacting with a Minecraft server using the KiraBukkitGateway plugin.
Other
1 stars 5 forks source link

createrelayhere generates an error #12

Closed wingzero54 closed 3 years ago

wingzero54 commented 3 years ago

image

KaloHG commented 3 years ago

The related line (Line 51, CreateRelayChannelHereCommand.java) fetches the permissions of a user and puts it into an EnumSet. The only way this could be a nullpointer, is if someone attempted to set up a relay in Private Messages (I think?), or if somehow the person is requesting a relay for a channel they are not in.

i.e. gonna need more info on this one wing, it makes literally no sense why this is returning null.

wingzero54 commented 3 years ago

Brit was attempting to create a relay for a citadel group he was in and had the kira permission for, and it was not working and threw this error

KaloHG commented 3 years ago

could you get more information about the channel, as well as the group?

Does this only occur with that specific citadel group? Or only that specific channel.

wingzero54 commented 3 years ago

could you get more information about the channel, as well as the group?

Does this only occur with that specific citadel group? Or only that specific channel.

It appears that any attempt to use "createrelayhere" in any channel or with any group throws this error, the command is totally broken right now.

KaloHG commented 3 years ago

Discord must've changed something related to the API.

wingzero54 commented 3 years ago

This was fixed, I believe by this commit but maybe another one around the same time https://github.com/CivClassic/Kira/commit/cf7cbd47fa7fe766c8bb7607bc455cfeff15b962