Or statement in C#

I am having a issue that I believe someone can help me with I cant seem to make this work. Its either one or the other.

Code:

if ((!IsAdmin(player.UserIDString)) || (!IsAllowed(player.UserIDString, permRadar))) { }bool IsAdmin(string id) => permission.UserHasGroup(id, "admin");        bool IsAllowed(string id, string perm) => permission.UserHasPermission(id, perm);

 

You actually want a AND in your if-statement as I assume that is where you send the user the message that he is not allowed to use the command so that would be triggered when the user is not an admin AND does not have the permission.

And why not merge IsAdmin with IsAllowed and do something like:

Code:

if (!IsAllowed(player.UserIDString, permRadar) {
    PrintToChat(player, "You are not allowed to use this!");
    return;
}bool IsAllowed(string id, string perm) => permission.UserHasPermission(id, perm) || permission.UserHasGroup(id, "admin");

 

Surprisingly I just did that

Code:

        bool IsAdmin(BasePlayer player) => permission.UserHasGroup(player.UserIDString, "admin") || player.net?.connection?.authLevel >= authLevel;        bool IsAllowed(BasePlayer player, string perm) => permission.UserHasPermission(player.UserIDString, perm) || IsAdmin(player);

[DOUBLEPOST=1469704460][/DOUBLEPOST]works great! Thanks for the advices I always seem to think better after I post in support forums