How to fix this piece of code?

Help and Feedback Scripting Support
1

I am making a nuke developer product, When a player purchases it, a event fires to all clients to say the name of the player and a warning message with the name of player, But apparently it gives out the own client’s name not the buyer. How do i fix this?
Here is the Server Script:

local function FiringAll()
	CS:FireAllClients(28)
	AT:FireAllClients(Player)
	BM:FireAllClients(Player)
	RR:FireAllClients()
	BGAB:FireAllClients()
	wait(16)
	ROF:FireAllClients()
	wait(7)
	L:FireAllClients()
	wait(5)
	Explosion:FireAllClients()
--Not full code

Local Script:

AT.OnClientEvent:Connect(function()
	local Purchaser = Player.PlayerGui.Nuke.Purchaser
	AnimatedTextOfPurchase(Purchaser, Player.Name.." Has purchased nuke! ")
end)
--Other message
local function BoughtMessage() 
	game.StarterGui:SetCore("ChatMakeSystemMessage",
		{
			Text = "[Server]:"..Player.Name.." Has bought nuke!",
			Size = 20,
			Color = Color3.new(1, 0.333333, 0),
			Font = Enum.Font.Creepster
		}
	)
end
2

you sure you got the player from the remotes,
maybe show more code?

3

I changed it a bit
Anyways here is the events

BM.OnClientEvent:Connect(function(player)
	game.StarterGui:SetCore("ChatMakeSystemMessage",
		{
			Text = "[Server]:"..player.Name.." Has bought nuke!",
			Size = 20,
			Color = Color3.new(1, 0.333333, 0),
			Font = Enum.Font.Creepster
		}
	)
end)

AT.OnClientEvent:Connect(function(player)
	local Purchaser = Player.PlayerGui.Nuke.Purchaser
	AnimatedTextOfPurchase(Purchaser, player.Name.." Has purchased nuke! ")
end)

Still sadly doesn’t work

4

few questions

  • what do these 2 letter remotes mean?

  • what are you passing through :FireClient()

  • what is player

i know what some your code does
i dont know where you Connected it

5

It’s connected to the server script
The player is the localplayer
I don’t quite understand what you mean by 2 letter remotes mean
I am passing the 2 messages when a player buys the nuke

6

here is the problem
try getting the player from OnClientEvent

i will quote it

end

tip for next time
try posting the Connect() functions in your code

7

Alright will put it there
the 2 letter remotes are shortcuts instead of writing them all
here is what they mean

local RR = RS.RisingRocket
local ROF = RS.RotationOfNuke
local BM = RS.BoughtMessage
local L = RS.Landing
local AT = RS.AnimatedText
local CS = RS.CameraShake
local BGAB = RS.Buttongoneandback

I will make sure to add the connect(), thx for that tip

Summary

This text will be hidden

8

Also How do I get the player from onclientevent? It’s already there in the photo I sent I am not quite sure
Where else do I put it?

9

send the player as second argument in FireClient
it will be the First argument in

which? pls link

im confused

10

image
image
For Both of them

11

Both got player as first parameter or argument as u said

12

Can you show what player is and if you are firing remotes with the player in a script or localscript?

13

They are all fired in the server script
player is inside the function Onclientevent

14

What if you fire the remote with a table with the player’s userid or username instead of firing with the player object?

Edit: Tested sending a player object though a remote and it worked, this is not needed.

15

Firing it from the server script?

16

Can you share what Player is when you call the FiringAll function?

17

Also How would i add player object in a table I am not quite certain how to do that