Scripting Helpers is winding down operations and is now read-only. More info→

How would i unban a player if they werent in the game?

Asked by

Zrxiis 26

7 years ago

Im trying to make a script so i can ban, kick, and unban players. But I'm not able to unban if the players arent in the game.

01local admins = {
02    "FangBlade16";
03}
04 
05local banlist = game:GetService("DataStoreService"):GetDataStore("BanList")
06 
07game.Players.PlayerAdded:connect(function(player)
08    local playerId = player.UserId
09    local check = banlist:GetAsync(playerId)
10    if check == nil then
11        banlist:SetAsync(playerId,"Clear")
12    elseif check == "Banned" then
13        player:Kick("You are banned from this game")
14    elseif check == "Clear" then
15        print("You are clear!")
16    end
17 
18    for i, admins in ipairs(admins) do
19        if player.Name == admins then
20            player.Chatted:connect(function(msg)
21                for i,players in ipairs(game.Players:GetChildren()) do
22                    if msg == ":kick "..players.Name then
23                        players:Kick("You have been kicked by: "..player.Name)
24                    elseif msg == ":ban "..players.Name then
25                        banlist:SetAsync(players.UserId,"Banned")
26                        players:Kick("You have been banned by: "..player.Name)
27--This part below wont work.
28                    elseif msg == ":unban "..players.Name then
29                        local pardonedId = game.Players:GetUserIdFromNameAsync(players.Name)
30                        banlist:SetAsync(pardonedId,"Clear")
31 
32                    end
33                end    
34            end)
35        end
36    end
37end)
 View all 37 lines...

1 answer

Answered by

WillieTehWierdo200 966

7 years ago

01...
02 
03player.Chatted:connect(function(msg)
04    if string.sub(msg, 1, 6) == ":unban" then
05        local playerName = string.sub(msg, 8)
06        -- first look for the player in the game
07        for _, player in pairs(game.Players:GetPlayers()) do
08            if player.Name == playerName then
09                banlist:SetAsync(player.UserId, "Clear")
10                return
11            end
12        end
13        -- if we get to this line, it means the player isn't in this game
14        local success, pardonedId = pcall(function()
15            return game.Players:GetUserIdFromNameAsync(playerName)
16        end)
17        if success then
18            banlist:SetAsync(pardonedId, "Clear")
19        else
20            print("User '"..playerName.."' does not exist")
21        end
22    else
23        -- this is your existing code for :kick and :ban
24        for _, player in pairs(game.Players:GetPlayers()) do
25            if msg == ":kick "..player.Name then
26                player:Kick("You have been kicked by: "..player.Name)
27            elseif msg == ":ban "..player.Name then
28                banlist:SetAsync(player.UserId,"Banned")
29                player:Kick("You have been banned by: "..player.Name)
30            end
31        end  
32    end 
33end)
34 
35...
 View all 35 lines...

Oops I didn't realize that `player` was already defined as the chatting player, so it will be overwritten in the second half of your script block and the kick messages will tell the kicked players that they were kicked by themselves! If you want to fix this, rename the `player` variable from line 24 to the end to something else, like `suspect` or `troublemaker` WillieTehWierdo200 966 — 7y

Thank you it worked. Zrxiis 26 — 7y

How would i unban a player if they werent in the game?

1 answer

Answer this question