Why is this script not working?

Help and Feedback Scripting Support
#1

Hello developers,

I typed this script

Schermopname (5)
Schermopname (5)1920×1080 374 KB

and the top lines that should give 100 coins don’t work (see video). I have no idea how to fix this, by the way I didn’t get any error messages. I hope someone can solve this.

robloxapp-20210622-1612565.wmv (685.1 KB)

2 Likes
#2

Try removing the ( and ) at (receiptInfo.ProductId = 12345678)

1 Like
#3

Why does the script below that gives 1000 coins work?

1 Like
#4

ProcessReceipt can only accept 1 function callback. It doesn’t work for 100 because another function overwrote it, put all the code in one ProcessReceipt callback function

Also, some advice

Put the local player line outside of the if statements since you would have duplicated code. Also make sure to check if the player is still in game via game:GetService("Players"):GetPlayerByUserId(receiptInfo.PlayerId), if it returns nil, the player left, so return Enum.ProductPurchaseDecision.NotProcessedYet

Use += value instead of = thing + value

If confused, I would recommend you send the code formatted correctly via 3 backticks (`) before and after your code

1 Like
#5

Sorry I’m not a programmer. Could you maybe show what it should look like?

1 Like
#6

SEnd your code in text and we’ll fix lol

2 Likes
#7

Here,

local mps = game:GetService(“MarketplaceService”)

mps.ProcessReceipt = function(receiptInfo)
if (receiptInfo.ProductId == 1183433727) then
local player = game.Players:GetPlayerByUserId(receiptInfo.PlayerId)
player.leaderstats.Coins.Value = player.leaderstats.Coins.Value + 100
return Enum.ProductPurchaseDecision.PurchaseGranted
end
end

mps.ProcessReceipt = function(receiptInfo)
if (receiptInfo.ProductId == 1183433728) then
local player = game.Players:GetPlayerByUserId(receiptInfo.PlayerId)
player.leaderstats.Coins.Value = player.leaderstats.Coins.Value + 1000
return Enum.ProductPurchaseDecision.PurchaseGranted
end
end

#8

Something like this in relation to your code

MarketplaceService.ProcessReceipt = function(receiptInfo)
	local player = game:GetService("Players"):GetPlayerByUserId(receiptInfo.PlayerId)
	if not player then
		--Player has left, do not say the purchase was successful
		return Enum.ProductPurchaseDecision.NotProcessedYet
	end
	
	if receiptInfo.ProductId == 1183433727 then
		player.leaderstats.Coins.Value += 100
	elseif receiptInfo.ProductId == 1183433728 then
		player.leaderstats.Coins.Value += 1000
	end
	
	return Enum.ProductPurchaseDecision.PurchaseGranted
end

If you are not a scripter I would recommend you spend some time learning it so you can understand simple issues and how to fix them. In this case a trip to the Roblox api would’ve shown you the problem

3 Likes
#9

send a screenshot of the OutPut
there shows the Script errors

#10

It’s already resolved. :grinning:

#11

MarketplaceService.ProcessReceipt is an index that defines a callback (function). You can change this callback but it can only have one function as it’s callback.

I would also be a nice lad and respect any bugs that might happen while setting the coins value, you can always return a non-processed receipt if so.

local Players = game:GetService("Players")
local MarketplaceService = game:GetService("MarketplaceService")

function MarketplaceService.ProcessReceipt(ReceiptObject)
	local UserId = ReceiptObject.PlayerId
	local LocalPlayer = Players:GetPlayerByUserId(UserId)

	if rawequal(ReceiptObject.ProductId, 0) then
		local success, response = pcall(function()
			local Coins = LocalPlayer["leaderstats"].Coins
			Coins.Value = Coins.Value + 1e2
		end)
		if success ~= true then
			return Enum.ProductPurchaseDecision.NotProcessedYet
		end
	elseif rawequal(ReceiptObject.ProductId, 0) then
		local success, response = pcall(function()
			local Coins = LocalPlayer["leaderstats"].Coins
			Coins.Value = Coins.Value + 1e3
		end)
		if success ~= true then
			return Enum.ProductPurchaseDecision.NotProcessedYet
		end
	end
	
	return Enum.ProductPurchaseDecision.PurchaseGranted
end