Why doens´t the money value increase by one when a player clicks the part?

I have recently been making a game, where I need the money value to rise by one each time the player clicks a part however, it doesn´t work and I can figure it out. Help would be appreciated and I will accept a good answer.

local DataStoreService = game:GetService("DataStoreService")

local MyData = DataStoreService:GetDataStore("MyData")

--Getting Data and Creating leaderboard
game.Players.PlayerAdded:Connect(function(player)
	local leaderstats = Instance.new("Folder")
	leaderstats.Name = "leaderstats"
	leaderstats.Parent = player
	
	local Money = Instance.new("IntValue")
	Money.Name = "Money"
	Money.Parent = leaderstats
	
	local Data
	local Sucess, ErroMessage = pcall(function()
		Data = MyData:GetAsync(player.UserId.."-Money")
	end)

	if Sucess == true then
		Money.Value = Data
	else
		print("There was an error")
		warn(ErroMessage)
	end
end)

--Saving Data
game.Players.PlayerRemoving:Connect(function(player)
	local Sucess, ErrorMessage = pcall(function()
		MyData:SetAsync(player.UserId.."-Money", player.leaderstats.Money.Value)
	end)
	
	if Sucess == true then
		print("PlayerData sucessfully saved")
	else 
		print("There was an error")
		warn(ErrorMessage)
	end
end)

local function OnClick()
	local TrashFolder = game.Workspace.TrashFolder
	local TrashFolderChildren = TrashFolder:GetChildren()
	
	for i, v in pairs(TrashFolderChildren)do
		if v.Name == TrashPart then
			v.ClickDetector.MouseClick:Connect(function(player)
				player.leaderstats.Money.Value = player.leaderstats.Money.Value + 1
			end)
		end
	end
end

Hey Andrius, first off. I wouldn't really try, to access a part through a Data Script. Since it's kinda it's own thing. Also, It might be, because when you set the money and where you actually add the money aren't on the same line. Or, when you access TrashPart(47) its as an object, not a string. If your just using the for loop, to find TrashPart directly. Just access it through workspace.TrashFolder directly. Add a WaitForChild() if you need to. This Code Should Work:

local Debounce = false

workspace.Part.ClickDetector.MouseClick:Connect(function(plr)
	if Debounce == false then
		Debounce = true
		plr.leaderstats.Money.Value = plr.leaderstats.Money.Value + 1
	end
	wait(2) -- So They Cant Spam The Click Detector
	Debounce = true
end)

Hope This Helped!

So basically you're using a server script service script, you have to put the touched script in the workspace or make a new script in the workspace and cut the touched function but instead of what you put you have got to do this.

local y = game.Players.LocalPlayer

local x = game.Players:GetChildren() -- This isn't necessary if you just want that player to get it but I'm just going to include it.--

local cash = 1 -- The amount of cash u wanna give--

script.Parent = workspace -- The script's parent can't be the part because it will cause an error in this script.--

local p = workspace:FindFirstChild("BlaBlaBla")
local r = p:FindFirstChildOfClass() -- This is so it looks for the one thing in the part which is going to be click detector. P.S you have to put this script in workspace--

r.MouseButton1Click:Connect(function(mouseClick, recievedData)
if mouseClick and recievedData then return end
       y.leaderstats.Cash.Value = y.leaderstats.Cash.Value + cash
end

This is just one of the many ways this can be fixed, this is just an example so I really have no clue if it will work but I hope it does and if it does, no need to thank me. I will always be here to help! --Kadren aka/ TheQuitzzyKadren