[UNSOLVED] Animation won't stop working even after i unequipped the tool?

--//Services
local UIS = game:GetService('UserInputService')

--//Variables
local player = game.Players.LocalPlayer
local char = player.Character
local tool = script.Parent


--//Use this since it's a local script inside a tool
player.CharacterAdded:Connect(function(character)
char = character
local hum = char:WaitForChild('Humanoid')


--//Animation
local animation = Instance.new('Animation')
animation.AnimationId = "rbxassetid://2680328238"
local spinSlash = hum:LoadAnimation(animation)

--//Equipped
tool.Equipped:Connect(function()
	
--//Input
local debounce = false
UIS.InputBegan:Connect(function(input)
	if input.KeyCode == Enum.KeyCode.X and not debounce then
		debounce = true
		spinSlash:Play()
		wait(2)
		debounce = false
		end
	end)
end)

--//Unequipped
tool.Unequipped:Connect(function()
	spinSlash:Stop()
	end)
end)

This is an animation of a sword slashing so it's not a looped animations. When the tool is equipped, x can be pressed in order to play the slashing animation but when i unequip it, x can still be pressed to play the animation again. How can i fix this so that x can only be pressed when the tool is equipped?

You are loading another animation when you are unequipping the tool. Fixed script:

local UIS = game:GetService('UserInputService')
local player = game.Players.LocalPlayer
local char = player.Character
local tool = script.Parent
--//Use this since it's a local script inside a tool
player.CharacterAdded:Connect(function(character)
char = character
local hum = char:WaitForChild('Humanoid')
 local meme = hum:LoadAnimation(tool.SpinSlash)
--//Equipped
tool.Equipped:Connect(function()
 
 
--//Input
local debounce = false
UIS.InputBegan:Connect(function(input)
    if input.KeyCode == Enum.KeyCode.X and not debounce then
        debounce = true
 
        meme:Play()
        wait(2)
        debounce = false
        end

    end)

end)

 

--//Unequipped

tool.Unequipped:Connect(function()

    meme:Stop()
    end)
end)

NVM, go with the other answer!

It is because you have

UIS.InputBegan:Connect(function(input)

as a form of global function. It isn't put in any other bounds of another function (as a local) like within the function, "equipped" but is defined as soon as the script is ran. So this means once you load in, you could press "X" by theory and the script would run again.

Since I don't exactly understand how you could make it local within equipped with it working properly and don't feel for researching at the moment, I will provide a top-of-the-head solution.

Have,

UIS.InputBegan:Connect(function(input)

As a defined function like:

function inputActivated(input)
	-- code
end

UIS.InputBegan:Connect(inputActivated)

Then when unequipped occurs, do this:

tool.Unequipped:Connect(function()
	inputActivated = nil
	hum:LoadAnimation(tool.SpinSlash):Stop()
	end
end)

You also had an additional "end)" that MIGHT'VE caused a problem.

tool.Unequipped:Connect(function()
	hum:LoadAnimation(tool.SpinSlash):Stop()
	end) -- Here
end)

Sorry for my laziness, but hopefully that helps.