MineColonies

MineColonies

57M Downloads

Permission nodes for all /mc commands.

Wissi opened this issue · 5 comments

commented

No specific nodes exist for the Minecolonies commands to be set server side in permission systems.

For example:
minecolonies.command.* (wildcard) will allow ALL /mc perms

But no specific node for each command.
/mc rtp (needs a node) for example: minecolonies.command.rtp
/mc colonies list it could be: minecolonies.command.colonies.list

Atm no variants work only the wildcard, but is not recommended because it gives ALL /mc commands and some should only be Admin/OP.

commented
commented

We need permission nodes on our commands to use the permission system if it is supported

commented

I guess it's two separate settings:
Config file side for SMP play in LAN ?

But definitely nodes required for server side permission systems.

commented

@Kostronor might that resolve our problem?

 return FMLCommonHandler.instance().getMinecraftServerInstance().getPlayerList()
                    .canSendCommands(((EntityPlayer) sender).getGameProfile());

 public boolean canSendCommands(GameProfile profile)
    {
        return this.ops.hasEntry(profile) || this.mcServer.isSinglePlayer() && this.mcServer.worlds[0].getWorldInfo().areCommandsAllowed() && this.mcServer.getServerOwner().equalsIgnoreCase(profile.getName()) || this.commandsAllowedForAll;
    }
commented

I think that resolved it