Custom Check Job

[Darkblizzard]

Hello,
I wanted to know how do I add to check two groups in ulx that are being inherited by others? As of currently this code is not working.
Thank you.

	    customCheck = function(ply) 
        if ply:IsValid() then return ply:CheckGroup("VIP") or ply:CheckGroup("VIP Moderator") end 
    end,
	CustomCheckFailMsg = "You do not VIP."

[c👌de_g🎷]

Try IsUserGroup instead of CheckGroup

[GoldTrigger]

	customCheck = function(ply) return table.HasValue({"gold", "diamond", "admin", "superadmin", "communitydirector", "founder"},ply:GetUserGroup()) end,

I use the above and it works very well. I prefer to check for every group instead of inherited ones. Just replace my ranks with yours :P

[Jeezy]

[QUOTE=GoldTrigger;47427852]

	customCheck = function(ply) return table.HasValue({"gold", "diamond", "admin", "superadmin", "communitydirector", "founder"},ply:GetUserGroup()) end,

I use the above and it works very well. I prefer to check for every group instead of inherited ones. Just replace my ranks with yours :P[/QUOTE]

No need to loop over the table like that.
[lua]
local ranks = { ["gold"] = true, ["diamond"] = true, ["admin"] = true, ["superadmin"] = true, ["communitydirector"] = true, ["found"] = true }
customCheck = function( ply ) return ( ranks[ply:GetUserGroup( )] ) end
[/lua]

[GoldTrigger]

[QUOTE=Jeezy;47427873]No need to loop over the table like that.
[lua]
local ranks = { ["gold"] = true, ["diamond"] = true, ["admin"] = true, ["superadmin"] = true, ["communitydirector"] = true, ["found"] = true }
customCheck = function( ply ) return ( ranks[ply:GetUserGroup( )] ) end
[/lua][/QUOTE]

Well I guess you learn something everyday. Also, question: Would looping over the table cause noticeable lag?

[c👌de_g🎷]

[QUOTE=GoldTrigger;47427938]Well I guess you learn something everyday. Also, question: Would looping over the table cause noticeable lag?[/QUOTE]

Depends on how many elements you are looping over and your sorting

[Lolcats]

No not in that function, but everyone here will tell you that you will reach [URL="http://en.wikipedia.org/wiki/Big_O_notation"]O( n ).[/URL] In layman's terms, you shouldn't. It's bad practice, especially when you start having larger tables/start using loops in functions that are called more rapidly.

[Darkblizzard]

[QUOTE=GoldTrigger;47427852]

	customCheck = function(ply) return table.HasValue({"gold", "diamond", "admin", "superadmin", "communitydirector", "founder"},ply:GetUserGroup()) end,

I use the above and it works very well. I prefer to check for every group instead of inherited ones. Just replace my ranks with yours :P[/QUOTE]

This works extremely well, thank you very much!