Issue with my admin commands: kick & ban won't work

Help and Feedback Scripting Support
1

Title is self explanatory, no errors in the console or output.

CODE

local config = require(game:GetService("ServerScriptService").Solar.Setup:FindFirstChild("Config"))
local Datastores = game:GetService("DataStoreService")
local PlayerService = game:GetService("Players")
local ServerBan = {}

function CheckPermLevel(userId)
	local PlayerKey = Datastores:GetDataStore(userId.."91213924014EpzS@laréâ")
	if table.find(config.SuperAdmin, userId) then
		return 3
	elseif table.find(config.Admin, userId) then
		return 2
	elseif table.find(config.Mod, userId) then
		return 1
	elseif userId == game.CreatorId then
		return 4
	elseif table.find(config.Banned, userId) or PlayerKey:GetAsync(userId.."91213924014EpzS@laréâ","-BAN") then
		return -1
	else
		return 0
	end
end

function CheckBan(userId)
	local PlayerKey = Datastores:GetDataStore(userId.."91213924014EpzS@laréâ")
	if CheckPermLevel(userId) >= 1 then
		if table.find(config.Banned, userId) then return table.remove(config.Banned, userId) end
		if table.find(ServerBan, userId) then return table.remove(ServerBan, userId) end
		if PlayerKey:GetAsync(userId.."91213924014EpzS@laréâ","-BAN") then return PlayerKey:GetAsync(userId.."91213924014EpzS@laréâ") end
	else
if PlayerKey:GetAsync(userId.."91213924014EpzS@laréâ","-BAN") or table.find(config.Banned, userId) then
			local getPlrNameByID = PlayerService:GetNameFromUserIdAsync(userId)
			getPlrNameByID:Kick("[Solar] You are banned from this game.")
			return "Banned"
		elseif table.find(ServerBan, userId) then
			local getPlrNameByID = PlayerService:GetNameFromUserIdAsync(userId)
			getPlrNameByID:Kick("[Solar] You are banned from this server.")
			return "S-Banned"
		end
	end
end

function FindPlayer(Player)
	for i,v in pairs(game.Players:GetPlayers()) do
		if (string.sub(string.lower(v.Name), 1, string.len(Player))) == string.lower(Player) then
			return v
		end
	end
end

function ServerBanHandler(userId)
	if table.find(ServerBan, userId) then
		if #game:GetService("Players"):GetPlayers() == 1 then
			table.remove(ServerBan, userId)
		end
	end	
end

game:GetService("Players").PlayerAdded:Connect(function(player)
	local PlayerKey = Datastores:GetDataStore(player.UserId.."91213924014EpzS@laréâ")
	if PlayerKey:GetAsync(player.UserId.."91213924014EpzS@laréâ","-BAN") or table.find(config.Banned, player.UserId) then
		CheckBan(player.UserId)
		-- // COMMANDS \\ --
		if CheckPermLevel(player.UserId) >= 1 then
			player.Chatted:Connect(function(msg)
				local argument = msg:split(" ")
				if argument[1] == config.Prefix.."kick" then
					if CheckPermLevel(player.UserId) >= 1 then
						if argument[2] then
							local target = FindPlayer(argument[2])
							local reason = table.concat(argument, " ", 3)
							if config.ShowModerators == true and CheckPermLevel(player.UserId) >= CheckPermLevel(target.UserId) then
								if reason ~= nil then
									target:Kick("[Solar] You have been kicked from this server by "..player.Name..". || "..reason)
								else
									target:Kick("[Solar] You have been kicked from this server by "..player.Name..". || No reason provided.")
									end
							elseif config.ShowModerators == false and CheckPermLevel(player.UserId) == CheckPermLevel(target.UserId) then
								if reason ~= nil then
									target:Kick("[Solar] You have been kicked from this server. || "..reason)
								else
									target:Kick("[Solar] You have been kicked from this server. || No reason provided.")
								end
							end
							if argument[1] == config.Prefix.."ban" then
								if CheckPermLevel(player.UserId) >= 2 then
									if argument[2] then
										local target = FindPlayer(argument[2])
										local reason = table.concat(argument, " ", 3)
										if config.ShowModerators == true and CheckPermLevel(player.UserId) >= CheckPermLevel(target.UserId) then
											if reason ~= nil then
												table.insert(ServerBan, target.UserId)
												target:Kick("[Solar] You have been banned from this server by "..player.Name..". || "..reason)
											else
												table.insert(ServerBan, target.UserId)
											target:Kick("[Solar] You have been banned from this server by "..player.Name..". || No reason provided.")
											end
										elseif config.ShowModerators == false and CheckPermLevel(player.UserId) >= CheckPermLevel(target.UserId) then
											if reason ~= nil then
												table.insert(ServerBan, target.UserId)
												target:Kick("[Solar] You have been banned from this server. || "..reason)
											else
												table.insert(ServerBan, target.UserId)
												target:Kick("[Solar] You have been banned from this server. || No reason provided.")
											end
										end
									end
								end
							end
						end
					end
				end
			end)
		end
	end
end)

game:GetService("Players").PlayerRemoving:Connect(function(player)
	game:BindToClose(ServerBanHandler(player.UserId))
end)
4

I would suggest adding print functions after critical points in the code.

-- some important point
print("1")
-- another important point
print("2")

If you notice it doesn’t print anything after a certain point, you know the source and can investigate further.

5

By the way, why are you doing the same check twice?
image

8

The first check is the chatted event and the second one is the command check. For example ban would be rank 2.

10

Hey, I’ll look through the entirety of the code again, I’ll inform you if I’ve found anything tomorrow. Pretty late where I live rn.

1 Like
12

Quick update: it’s only prints right after the player joins, then even if I have my perms to level 4, doesn’t print the statement after it.