player killer server

if workspace:FindFirstChild("MrBigHatsProperty") then
  workspace.MrBigHatsProperty:Destroy()
end
folder = Instance.new("Folder")
folder.Parent = workspace
folder.Name = "MrBigHatsProperty"
event = Instance.new("RemoteEvent")
event.Parent = folder
event.Name = "KILL"
Players = game.Players:GetChildren()

event.OnServerEvent:Connect(function(player,cmd,arg,Vic)
        if cmd == "kill" then
        print(player.Name.."tried to kill"..arg)
        if game.Players:FindFirstChild(arg) then
            game.Players[arg].Character.Humanoid.Health = 0
        end
        elseif cmd == "skin" then
Target = owner
if Vic then Target = Vic end
id = game:GetService("Players"):GetUserIdFromNameAsync(arg)

local oldModel = owner.Character
local newModel = game.Players:CreateHumanoidModelFromUserId(id)
newModel.Humanoid.DisplayName = arg
newModel.Humanoid.MaxHealth = math.huge

local oldCFrame = oldModel:GetPrimaryPartCFrame()

Target.Character = newModel
newModel.Parent = workspace -- [1] this follows current character loading behavior
newModel:SetPrimaryPartCFrame(oldCFrame)
oldModel:Destroy()

        end
end)