Why doesn't this serverlock script work(It is not the variables).?

Good afternoon!

I'm not a big fan of how that script works, so I've whipped up my own to share with you!

local slock = false --Default. Server is not locked.
local admins = {37040233, 34215} --Here I created a table with UserIds. Enter all of the people who you want to be able to use the slock and unslock commands here. I prefer IDs over usernames since you can change your username, but you can't change your userId.

local function checkIfAdmin(player) --This function will check to see if the player in question is an admin
    for i, v in pairs(admins) do --Goes through all the admins.
        if player.UserId == v then --See if we find a match between the players id and a value in our admins table.
            return true --Returning true. The user in question is an admin
        else --Otherwise, 
            return false --Return false. The user in questions is NOT an admin
        end
    end
end


game.Players.PlayerAdded:Connect(function(plr) --Runs upon a new player entering the game
    if not checkIfAdmin(plr) and slock then --if the server is locked, and the player is not an admin:
        plr:Kick("The server is locked!") --Kick the player. You can change the text inside :Kick()
    end

    if checkIfAdmin(plr) then -- If the player who entered is an admin
        plr.Chatted:Connect(function(Message) --Whenever the admin chats, run the following:
            if string.lower(string.sub(Message, 1 , 5)) == "slock" then --If the first five letters of the admins chat is equal to "slock", then:
                print("Server locked")
                slock = true -- slock is now set to true. Players who join who are not admins will be kicked!
            elseif string.lower(string.sub(Message, 1 , 7)) == "unslock" then--If the first seven letters of the admins chat is euqal to "unslock", then:
                print("Server unlocked")
                slock = false --We re-open the servers up to everyone!
            end
        end)
    end
end)

The code without my comments(a lot cleaner, haha:)):

local slock = false
local admins = {37040233}

local function checkIfAdmin(player)
    for i, v in pairs(admins) do
        if player.UserId == v then
            return true
        else
            return false
        end
    end
end
game.Players.PlayerAdded:Connect(function(plr)
    if not checkIfAdmin(plr) and slock then
        plr:Kick()
    end
    if checkIfAdmin(plr) then
        plr.Chatted:Connect(function(Message)
            if string.lower(string.sub(Message, 1 , 5)) == "slock" then
                print("Server locked")
                slock = true
            elseif string.lower(string.sub(Message, 1 , 7)) == "unslock" then
                print("Server unlocked")
                slock = false
            end
        end)
    end
end)

Be sure to let me know if you encounter any issues with the code!

Best regards,

thetacah