Ban not adding.

[StaTiiKxKALEB]

I'm trying to get this code to work, but it's not adding the ban when they disconnect. Any ideas?

Code: [Select]

hook.Add("PlayerDisconnect", "BanForCheaterDisconnect", function(ply)
local plysteamid = ply:SteamID()
local plyname = ply:Nick()
local time = 1
local admin = "CONSOLE"
local reason = "(CHEATING) Don'try to cheat."

if ply:IsUserGroup("cheater") then
ULib.addBan(plysteamid, time, reason, plyname, admin)
end
end)


[Bytewave]

Presumably you should be using ULib.ban(ply[, time][, reason][, admin]) instead of ULib.addBan(), but I'm not sure.

[JamminR]

PlayerDisconnectED ?
Notice my emphasis on 'ed'.
You seem to be using an incorrect hook name.

Errors are likely appearing in console when this first gets loaded that would indicate this.

[JamminR]

ByteWave, now that you point yours out, I'm not sure honestly.
Ban looks for the player and removes them forcefully.
I'm pretty sure addBan could be used since they're already leaving.

[StaTiiKxKALEB]

PlayerDisconnectED ?
Notice my emphasis on 'ed'.
You seem to be using an incorrect hook name.

Errors are likely appearing in console when this first gets loaded that would indicate this.

Thanks, didn't realize. Simple mistake. Thanks for the help man. Also, thanks for the information @Bytewave. Also, how would I correctly add a user to a group? For right now, I'm using RunConsoleCommand( "ulx", "adduserid", ply:SteamID(), "cheater" )

[JamminR]

Well, that should work.
However, since we didn't really dev ULX to be built on (NOT that we mind it), if you're developing only relying on ULib (and not ULX), see
ULib.ucl.addUser

[MrPresident]

The first argument of a hook.Add is just a string identifier for that particular hook, if it doesn't exist it won't throw an error, it'll just never get called.

[JamminR]

Thanks MrP.
(Seems silly, would be easier to troubleshoot if they did)
"Unknown hook - nonexistent at line blah"

[Bytewave]

Thanks MrP.
(Seems silly, would be easier to troubleshoot if they did)
"Unknown hook - nonexistent at line blah"

Well, when you consider the lack of "registering" events and instead just running hook.Call with an arbitrary event name, that's not really feasible. It would be helpful, though, just breaking. :p