Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #80191
✓ ANSWER ACCEPTED

Sword keeps breaking after purchase from GUI?

Asked 248 views1 answers

I'm trying to make a game where you can buy a sword w/ the starting cash from some leaderstats. If I put the sword into Starterpack, it works fine. However, once I purchase the sword and it goes into my inventory, it seems to just break the scripts. Here is my current script:

script.Parent.MouseButton1Click:Connect(function()

local PriceOfItem = game.ReplicatedStorage.Buyables.Sword.Price.Value

if game.Players.LocalPlayer.leaderstats.Cash.Value >= PriceOfItem then

local clone = game.ReplicatedStorage.Buyables.Sword:Clone()

clone.Parent = game.Players.LocalPlayer.Backpack

game.Players.LocalPlayer.leaderstats.Cash.Value = game.Players.LocalPlayer.leaderstats.Cash.Value - 50

end

end)

P.S. This is in a LocalScript in a textbutton in a Gui. If you're wondering about the "Buyables" section, it's a folder in ReplicatedStorage.

Anonymous

3 comments

0

Just realized I didn't put the whole thing in a code block. Extremely apologetic, can I edit the post?

Anonymous#
0

How is a local script going to get to the server?

Anonymous#
0

That wouldn't work with FE

Anonymous#

1 answer

✓ Accepted answer
local Player = game:GetService("Players").LocalPlayer
local Cash = Player:WaitForChild("leaderstats"):WaitForChild("Cash")
local ReplicatedStorage = game:GetService("ReplicatedStorage")
local RemoteEvent = ReplicatedStorage:WaitForChild("RemoteEvent")
local Sword = ReplicatedStorage:WaitForChild("Buyables"):WaitForChild("Sword")
local Price = Sword:WaitForChild("Price")

local function MouseButton1Click ()
	if Cash.Value >= Price then
		local NewSword = Sword:Clone()
		NewSword.Parent = Player.Backpack
		RemoteEvent:FireServer(Price)
	end
end

script.Parent.MouseButton1Click:Connect(MouseButton1Click)

Insert a remoteevent in the replicatedstorage then a script in the serverscriptservice then write this in that script.

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

local function OnServerEvent (Player, Price)
	local Cash = Player.leaderstats.Cash
	Cash.Value = Cash.Value - Price
end

ReplicatedStorage.OnClientEvent:Connect(OnServerEvent)

The reason why you should use remote event is because when you change a value in a local script it wont replicate to the server or all the clients, it means that if you change the value of money using a local script to.. example from 120 to 70 the client will see the that the value changed but in the other clients they will still see you have 120 money.

AnonymousEdited Permalink

This discussion is preserved as it was. New answers, comments, and votes are closed.