Why doesn't my remote event work properly?

You made a mistake in firing the event. When you fire an event TO the server, you're not supposed to add the player as a variable because the server already has the player as a variable. Secondly, you're not supposed to add "end" when firing events.

Here's some example code:

LocalScript

local Event = game:GetService("ReplicatedStorage"):WaitForChild("RemoteEvent")

Event:FireServer()

ServerScript

local Event = game:GetService("ReplicatedStorage"):WaitForChild("RemoteEvent")

Event.OnServerEvent:Connect(function(plr)
	--Code
end)

Just a tip: You should always put ALL your code into a code block so it will be easier for people to read your code. To do that, highlight ALL the code and press the blue Lua button. It will add "~" around the code.

EDIT:

Here's a RemoteEvent that tells the server to request new clothing for the character.

LocalScript

local Event = game:GetService("ReplicatedStorage"):WaitForChild("RemoteEvent")

Event:FireServer()

ServerScript

local Event = game:GetService("ReplicatedStorage"):WaitForChild("RemoteEvent")

Event.OnServerEvent:Connect(function(plr)
	if plr.Character then --Checks for the character.
		for i,v in pairs(plr.Character:GetChildren()) do
			if v:IsA("Accoutrement") or v:IsA("ShirtTemplate") or v:IsA("PantsTemplate") then
				v:Destroy() --Removes all clothing.
			end
		end
		
		--Make a folder in ServerStorage with new Accessories and Shirt / Pants.
		for i,v in pairs(game.ServerStorage.Folder:GetChildren()) do
			v:Clone().Parent = plr.Character --Clones new clothing to the character.
		end
	end
end)