Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Double Coins Gamepass Wrong Values?
Hello! In my game, I am trying to make a double coins gamepass. Each round, the player should earn 5 coins (so double coins would give them 10). The gamepass works, and 10 coins are rewarded. However, if you don't own the gamepass, you sometimes receive 5 coins like you are supposed to and other times you receive 15 coins.
- No errors are given
- This has only been tested in singleplayer
- The script only happens once, because the Wins go up only 1. (This is the purpose of having some of the variables) -
Thanks to anyone who attempts to help!
01 | script.Parent.Touched:Connect(function(p) |
02 | local char = p.Parent.Name |
03 | local player = game.Players:FindFirstChild(char) |
04 | local numWins = player.leaderstats.Wins.Value |
05 | local numCoins = player.leaderstats.Coins.Value |
06 | local CoinsEarned = 5 |
07 |
08 | player.Character.HumanoidRootPart.CFrame = game.Workspace.LOBBYTELE.CFrame |
09 |
10 | wait(2) |
11 | local service = game:GetService("MarketplaceService") |
12 | if service:UserOwnsGamePassAsync(player.userId,8519306) == true then |
13 | CoinsEarned = 10 |
14 | end |
15 | player.leaderstats.Coins.Value = numCoins + CoinsEarned |
16 | player.leaderstats.Wins.Value = numWins + 1 |
17 | end) |
1 answer
Log in to vote
2
This is just a cleanup
01 | local MpS = game:GetService ("MarketplaceService") |
02 | local Gamepass = 8519306 |
03 |
04 | local Part = script.Parent |
05 |
06 | local function PlayerWonAction(Target) |
07 | if not (Target) then return end |
08 | local Player = game.Players:GetPlayerFromCharacter(Target.Parent) |
09 | if (Player and Player.Character) then |
10 | local Leaderstats = Player:FindFirstChild("leaderstats") |
11 | if (Leaderstats and Leaderstats.Coins and Leaderstats.Wins) then |
12 | local CoinsEarned = MpS:UserOwnsGamepassAsync(Player.UserId, Gamepass) and 10 or 5 |
13 | Leaderstats.Coins.Value = (Leaderstats.Coins.Value + CoinsEarned) |
14 | Leaderstats.Wins.Value = (Leaderstats.Wins.Value + 1) |
15 | end |
16 | Character:SetPrimaryPartCFrame(workspace.LOBBYTELE.CFrame) |
17 | end |
18 | end |
19 |
20 | Part.Touched:Connect(PlayerWonAction) |
0
Thank you! This definitely easier to read than what I had lol :) Amanda314159 291 — 5y