Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Trying to prompt purchase of a dev product when a brick gets touched?
Hello, here is the script I have so far and im getting this error:
15:48:37.186 - Player is not a valid member of DataModel
Can anybody help me?
Here is my script:
1 | script.Parent.Touched:Connect(function(hit) |
2 | if hit and hit.Parent and game.Players:FindFirstChild(hit.Parent.Name) then |
3 | game:GetService("MarketplaceService"):PromptProductPurchase(game.Player:FindFirstChild(hit.Parent.Name), 408797380) |
4 | end |
5 | end) |
0
game.Players, ah. Also don't use FindFirstChild on Players, just use GetPlayerFromCharacter and do game.Players:GetPlayerFromCharacter(hit.Parent). It's much more reliable, and no need to do if hit and hit.Parent and ... then User#19524 175 — 6y
1 answer
Log in to vote
2
You missed "s" letter on line 3 and all your script is scattered. Here is the right working script
01 | local MPS = game:GetService("MarketplaceService") |
02 | local productId = 408797380 |
03 |
04 | script.Parent.Touched:Connect(function(hit) |
05 | if hit.Parent:FindFirstChild("Humanoid") then |
06 | local player = game.Players:GetPlayerFromCharacter(hit.Parent) |
07 | if player then |
08 | MPS:PromptProductPurchase(player, productId) |
09 | end |
10 | end |
11 | end) |
You can read more here
0
No need for line 5. You're just checking the same thing twice User#19524 175 — 6y
0
But what if not an humanoid will hit that brick? (example the baseplate) HaveASip 494 — 6y
0
Yeah, don't listen to incapaz, you definitely want to check if its a person thats hitting the brick. In fact, I would go the extra mile and do a (hit.Parent:FindFirstChild('Humanoid') or hit.Parent.Parent:FindFirstChild('Humanoid')). OrcaTheFish 95 — 6y
0
Why would you ever check the parent's parent. Every Player Humanoid part that can trigger Touched on BaseParts will be in the hierarchy format to reach Humanoid by simply using FindFirstChild() on Parent. SummerEquinox 643 — 6y