self:Team() isn't actually checking

[Triple-X]

[LUA]if PS.Config.PointsOverTime then
timer.Create('PS_PointsOverTime_' .. self:UniqueID(), PS.Config.PointsOverTimeDelay * 60, 0, function()
if !IsValid(self) or !self:Team() == TEAM_HUNTERS or !self:Team() == TEAM_PROPS then return end
self:PS_GivePoints(PS.Config.PointsOverTimeAmount)
self:PS_Notify("You've been given ", PS.Config.PointsOverTimeAmount, " ", PS.Config.PointsName, " for playing on the server!")
end)
end[/LUA]

Am I using an incorrect notation or something? Players that are not TEAM_HUNTERS or TEAM_PROPS are still getting points. Thanks.

[NiandraLades]

Have you tried adding print statements to see where the issue lies

[Triple-X]

[LUA]if !IsValid(self) or !self:Team() == TEAM_HUNTERS or !self:Team() == TEAM_PROPS then return end[/LUA]

Added a print on this line and got nothing back.

[Mista Tea]

Pretty sure you shouldn't be doing

!this == that

You should be doing

this != that

or

!(this == that)

[Triple-X]

[QUOTE=Mista Tea;50491446]Pretty sure you shouldn't be doing

!this == that

You should be doing

this != that

or

!(this == that)

[/QUOTE]

Made that change in my code and now no-one gets points :pudge:

[TW1STaL1CKY]

It should be:

if !IsValid(self) or (self:Team() != TEAM_HUNTERS and self:Team() != TEAM_PROPS) then return end

or

if !IsValid(self) or !(self:Team() == TEAM_HUNTERS or self:Team() == TEAM_PROPS) then return end

In your original code, you're doing !self:Team() which gets the team ID and then turns it to false (because the 'not' of something that isn't nil or false is false). Then you are checking if that is equal to TEAM_HUNTERS (and TEAM_PROPS) which are numbers, but TEAM_HUNTERS and TEAM_PROPS aren't false, so the if statement isn't successful and continues to give the points.

[Triple-X]

[QUOTE=TW1STaL1CKY;50492371]It should be:

if !IsValid(self) or (self:Team() != TEAM_HUNTERS and self:Team() != TEAM_PROPS) then return end

or

if !IsValid(self) or !(self:Team() == TEAM_HUNTERS or self:Team() == TEAM_PROPS) then return end

In your original code, you're doing !self:Team() which gets the team ID and then turns it to false (because the 'not' of something that isn't nil or false is false). Then you are checking if that is equal to TEAM_HUNTERS (and TEAM_PROPS) which are numbers, but TEAM_HUNTERS and TEAM_PROPS aren't false, so the if statement isn't successful and continues to give the points.[/QUOTE]

Works perfect, thanks mate.