Remote events broken, 1 click fires twice

Help and Feedback Scripting Support
, ,
1

Hey there, the problem is:
I click the button once and it fires the button code ONCE but the event fires TWICE

Client

local player = game.Players.LocalPlayer
local ReplicatedStorage = game:GetService("ReplicatedStorage")

local cooldown = false

script.Parent.MouseButton1Up:Connect(function()
	print("click")
	if not cooldown then
		cooldown = true
		ReplicatedStorage:FindFirstChild("EquipSword"):FireServer(tostring(script.Parent.Name))
		wait(1)
		cooldown = false
	end
end)

Server

--Equip a sword frm your inventory
EquipSwordRE.OnServerEvent:Connect(function(player, Sword)
	print(Sword.." equipped")
	if player.Swords:FindFirstChild(Sword) then
		if player.Backpack:FindFirstChildOfClass("Tool") then
			if player.Backpack:FindFirstChild(Sword) then
				player.Backpack:FindFirstChildOfClass("Tool"):Destroy()
				player.checks.Equipped.Value = " "
			else
				player.Backpack:FindFirstChildOfClass("Tool"):Destroy()
				ServerStorage.Swords:FindFirstChild(Sword):Clone().Parent = player.Backpack
				player.checks.Equipped.Value = Sword
			end
		elseif workspace:FindFirstChild(player.Name):FindFirstChildOfClass("Tool") then
			if workspace:FindFirstChild(player.Name):FindFirstChild(Sword) then
				workspace:FindFirstChild(player.Name):FindFirstChildOfClass("Tool"):Destroy()
				player.checks.Equipped.Value = " "
			else
				workspace:FindFirstChild(player.Name):FindFirstChildOfClass("Tool"):Destroy()
				ServerStorage.Swords:FindFirstChild(Sword):Clone().Parent = player.Backpack
				player.checks.Equipped.Value = Sword
			end
		else
			ServerStorage.Swords:FindFirstChild(Sword):Clone().Parent = player.Backpack
			player.checks.Equipped.Value = Sword
		end
	end
end)

Console print:

click
Sword equipped
Sword equipped

The issue is not in my code, since it worked fine the other day, so it’s most likely caused by a new Roblox update.

I have the same issue with ALL of my remote events.
ALL of them.

2

dont use mousebutton1up, use mousebutton1click
mousebutton1up work when u not hold mousebutton1 so when u click it will thought u hold 2 times

3

Well, the fact of the matter is that there is indeed no issue in MouseButton1Up. It is something with your code as “click” is only printed once.

Can you check if your game has any other client scripts related to that button that fire the remote “EquipSword”, that is most likely your issue (overlapping buttons, overlapping scripts)