Why doesnt this kill all the players in the game?

local script:

local player = game.Players.LocalPlayer

player.Chatted:Connect(function(msg)

if msg:lower():match("killall") then

game:GetService("ReplicatedStorage"):WaitForChild("killall"):FireServer()

end

end)

server script:

local killall = remote:WaitForChild("killall")

local players = game.Workspace:GetChildren()

killall.OnServerEvent:Connect(function()

for i, v in pairs(players) do

if v:FindFirstChild("Humanoid") then

v.Humanoid.Health = 0

print(v.Name)

print(v.Humanoid.Health)

end

end

end)

output: klepolek version 3

0

MR. Pirana

0

baldi

0

but i didnt die, why when the others die(they are npc's with a humanoid inside of them)

You used "workspace:GetChildren()" and not "game:GetService("Players"):GetPlayers()". If you do workspace:GetChildren() you'll get stuff such as the Baseplate and models and not the actual players.

Instead, we'll be looping through all of the players, getting the characters, and modifying their health to 0.

local KillAll = game:GetService("ReplicatedStorage").killall

KillAll.OnServerEvent:Connect(function()
	for i,v in pairs(game:GetService("Players"):GetPlayers()) do
		local Character = v.Character
		if Character then
			Character.Humanoid.Health = 0
		end
	end
end)

In the code above we're looping through all of the players. We then make a variable of their character. If their character does exist we change their humanoid's health to 0. If their character does not exist we do not do anything and move on the the next player.