I made a sword script and I have this enemy but he is not taking any damage? Why?

This is the script im using

local sword = script.Parent
local canAttack = true
local canDamage = false

function onTouch(part)

	local character = part.Parent
	local player = game.Players:GetPlayerFromCharacter(character)
	
	
	if player then
		if canDamage == true then
			character.BoarHumanoid:TakeDamage(2)
		end
	end
	
end
sword.Blade.Touched:Connect(onTouch)

function Attack()
	
	local attack = sword.Parent.Humanoid:LoadAnimation(script.SwordSlash)
	if canAttack == true then
		attack:Play()
		canAttack = false
		
		canDamage = true
		wait(1)
		
		canDamage = false
		wait(0.25)
		canAttack = true
	end
	
end
sword.Activated:Connect(Attack)

I noticed two major mistakes in your code:

• You're checking an NPC to see if it is a player, which will never happen.
• The Tool is not actually parented to a player's Backpack, therefore Activated fails to work. This may also be the reason you're getting an error.

Solutions:

1. Allow the player to use the tool. Putting the tool in StarterPack or assigning its parent to the player's Backpack both work. If you parent the tool to the player's Character, the player will automatically equip the tool.

2. Check for not game.Players:GetPlayerFromCharacter(character). I think you're getting confused with this function, but adding not to it is telling the script to check to see if there is no player associated with character.

I fixed it so thanks for everyone who helped