why is my remote event not working (somewhat solved)

[mortal moat]

here is the local script: ```local Button = game.StarterGui.ScreenGui.TextButton
local Player = game.Players.LocalPlayer

Button.MouseButton1Click:Connect(function()
game.ReplicatedStorage.DropName:FireServer(Player)
end)Here is the regular script:game.ReplicatedStorage.DropName.OnServerEvent:Connect(function(Player)
print("Test")
end)```

[rapid snow]

u dont need to pass the player in the fireserver, it already know whos the one who started the event

and make sure is everything where it need to be

[mortal moat]

what

im just trying to print test to see if its passing on

but it aint printing

when i click the gui button

please help

why is my remote event not working (still unsolved help)

[dawn tinsel]

If possible, dont use remote events

[mortal moat]

so what do i do

[mossy wraith]

** You are now Level 3! **

[mortal moat]

im making a cash system and im trying to make it so a player can drop cash they have by using clone and then assiging the value

im currently trying to get it to clone first

[dawn tinsel]

Looking at your script, I see that you want to make a button to be clickable

Just use a server script, easy as that

[mortal moat]

you mean

i can use server script with guis?

[dawn tinsel]

Yea, 100%

[mortal moat]

how do i track who clicks the button then

[dawn tinsel]

And tbh its better to let the server handle it

[dawn tinsel]

Just get the player

[mortal moat]

local Player = game.Players.LocalPlayer

?

[dawn tinsel]

No... It wont work

Depends where your script is located

[mortal moat]

in the button

[dawn tinsel]

LocalPlayer works only on client

[mortal moat]

ive only been scripting for half a week

[dawn tinsel]

You know how StarterGui works, right?

[mortal moat]

yeah

it displays a gui

[dawn tinsel]

No worries, everyone starts somewhere

[dawn tinsel]

Yea but you know that it replicates to clients (players)?

[mortal moat]

keep going

tell me more

[dawn tinsel]

When a player joins, everything inside StarterGui gets cloned and parented to the player's gui (PlayerGui inside the player)

[mortal moat]

i see

[dawn tinsel]

So if we know that, we could get the player if the script is anywhere in the PlayerGui by just getting the first ascensor that is a player class

[mortal moat]

so... if the button is directly in a screen gui script.parent.parent

the third parent would be the player?

[mortal moat]

?

[dawn tinsel]

I pretty sure that would be the PlayerGui, not the player itself

[rapid snow]

i dont think a script in playergui its good.. pretty exploitable

[mortal moat]

idc

its just exploits

[rapid snow]

im not talking to you, im talking to the one is saying that

[dawn tinsel]

But a easy way of getting the player, no matter wher inside of PlayerGui the script is and without having to use .Parent multiple times, is to use FindFirstAncestorOfClass

[mortal moat]

i see

so

let me try it

[dawn tinsel]

Hackers could expoilt no matter where the scripts are tbh

[mortal moat]

ill be back in momento

[rapid snow]

yeah but putting that there make it easier lol, just expect your game being hacked for all sides

[mortal moat]

im not making a game

im practasing

[dawn tinsel]

Unless its a competitve game, make your game fun to play and dont fixate on making it un-hackable

[mortal moat]

for my dream game

[rapid snow]

seeing that is probably the first or one of the first yes

[dawn tinsel]

He could use the script in ServiceScriptService, that would be safer I guess, but he would have more things to do, if you wanna go thru with him its fine by me, but I wont

[mortal moat]

i will prob use it for something but i dont worry about exploits (i used to be an exploiter shhhh) byfron is here and synapse never gave me my money back and have partnered with rblx soooo

i dont worry about exploiters

[dawn tinsel]

Anyways, where were we?

[mortal moat]

yup, hold up, ill try to do what you told me

ill ping you

[dawn tinsel]

You could try and make it hard for them, but dont waste all your time making the game un-hackable and forget about the quaility of the game itself

[mortal moat]

im not even gonne try make the game unhackable

not worth my time and roblox is winning the war agaisnt exploiters

did i put the find first ancestor in the correct place?

[rapid snow]

use :

[dawn tinsel]

Like instead of using a RemoteEvent that could be easily exploited, use a server script but dont go the extra mile to make it imposible to cheat. If you are able to make a really fun game, few people would cheat, but if your game is not that fun, way more people would cheat. Lets not even talk about how many people dont even know to write a code and still use cheates that other people made, so if your game is not that big, few people would care to make a cheat

local plr = script:FindFirstAncestorOfClass("Player")

[rapid snow]

and do FindFirstAncestorOfClass("Player")

https://tenor.com/view/crunch-cat-luna-eat-gif-3997701764820700221

[mortal moat]

now its red

[dawn tinsel]

And dont use the RemoteEvent, that could easily be used by cheaters

[mortal moat]

ok

[rapid snow]

FindFirstAncestorOfClass("Player")

[dawn tinsel]

You still need to add ( ) and arguments

[mortal moat]

although i will use it sometimes

[mortal moat]

also when i click the button it doesnt print anything

[dawn tinsel]

Well yea, sometimes you actually have no other way but using it

Line 1

[mortal moat]

im currently trying to get the gui to work and then i can do the rest

[dawn tinsel]

You are using "StarterGui"

[mortal moat]

until i do smth stupid

[dawn tinsel]

Instead of taking the gui from the player

[mortal moat]

oh i see

[dawn tinsel]

First you need to get the player

local plr = script:FindFirstAncestorOfClass("Player")

Then you need to find the button

local button = plr.PlayerGui.ScreenGui.TextButton

[mortal moat]

local Player = script:FindFirstAncestorOfClass("Player")
local Button = Player.ScreenGui.TextButton

Button.MouseButton1Click:Connect(function()
print("Hewo :3")
end)

oh

ok

[rapid snow]

or script.Parent no?

[dawn tinsel]

Hm yea, or that

[dawn tinsel]

Close enough, you also need to add PlayerGui

[mortal moat]

local Player = script:FindFirstAncestorOfClass("Player")
local Button = script.Parent

Button.MouseButton1Click:Connect(function()
print("Hewo :3")
end)

[dawn tinsel]

Does it works now?

[mortal moat]

one sec

[mossy wraith]

** You are now Level 4! **

[mortal moat]

https://www.youtube.com/watch?v=1rd4P7uMvvQ

my gift to u

i listen to this while scripting

now lemme check

[dawn tinsel]

https://tenor.com/view/cat-22-cat-22-gif-1269004317513322794

[mortal moat]

yup

its printing!!!!

[dawn tinsel]

Good

[mortal moat]

ok wait

one more thing

how do i check what value a player types into a text box so i know how much the player wants to drop

[dawn tinsel]

Now, if you want to make it a bit more secure, you could add the script in ServiceScriptService but you would have to change a few things

[dawn tinsel]

Here you would need a TextBox and a remote event

[mortal moat]

oh

remote event

[dawn tinsel]

I listen to metal/rock haha

[mortal moat]

why is my remote event not working (somewhat solved)

[dawn tinsel]

Some of the things are client sided only, like TextBox, mouse location etc

[dawn tinsel]

And this is where you need a remote event which could be easily used by exploiters

[mortal moat]

hold up im still scripting the button

wat

i dont have a humanoid?

[rapid snow]

u need to go to char first

Player.Character

[mortal moat]

ayyyy

im a money machine now

although if multible players join does that mean it will also spawn on them?

[mortal moat]

or do you think this is multiplayer compatible

[mortal moat]

i wonder if i can make the money the player drops spawn infront of him instead of in him, do you know how?

eh

thats something for another time

[mortal moat]

alright ill do the text box thingy now

[dawn tinsel]

Change the CFrame

[mortal moat]

nvm that we are doing text box

i have my lil text box

[mortal moat]

what now

[mortal moat]

hlep

@dawn tinsel

local Player = script:FindFirstAncestorOfClass("Player")
local TextBox = script.Parent
local Amount = script.Parent.Amount.Value

if TextBox:IsFocused() == false then
local Amount = TextBox.Text
end

how do i fix this mess

😭

[dawn tinsel]

Man Im pretty sure IsFocused is a event...

[rapid snow]

do

TextBox.FocusLost

this gets two parameters

the first that is if the enter was pressed and the second that is the inputObject

TextBox.FocusLost:Connect(function(enterPressed, inputObject)

Amount = TextBox.Text

end)

dont do another local, just change the local u declared

you can if u want do only if enterpressed if u want to the value set only if the enter was pressed

thats just your likings

[mortal moat]

still not working 😦

i dont think the amount is getting changed

waiitttt

i did smth wrong

[mossy wraith]

** You are now Level 5! **

[mortal moat]

im tryna to debug this but it just wont work

whats wrong with this

[rapid snow]

Amount.Value

stop

wati a min

you need to declare the object of the value

local Amount = script.Parent.Amount

and then in the textbox you change the value

Amount.Value = TextBox.Text

[mortal moat]

oh i see

thank you

[mortal moat]

[mortal moat]

still wont work

local TextBox = script.Parent
local Amount = script.Parent.Amount

TextBox.FocusLost:Connect(function(enterPressed, inputObject)

        Amount.Value = TextBox.Text
    
end)```

local Button = script.Parent
local Money = workspace.Money
local Worth = script.Parent.Parent.TextBox.Amount.Value


Button.MouseButton1Click:Connect(function()
    Worth = script.Parent.Parent.TextBox.Amount.Value
    local position = Player.Character.HumanoidRootPart.Position
    local newObject = Money:Clone()
    newObject.Parent = workspace
    newObject.Position = position
    newObject.Name = "Money"
    newObject.Amount.Value = Worth
end)```

what is wrong with me

why am i so stupid

[rapid snow]

its this a localscript??

[mortal moat]

no

[rapid snow]

this needs to be one

[mortal moat]

but then it cant change the value

[rapid snow]

this is where u need to do a remoteevent

[mortal moat]

: (

[mortal moat]

ok

can you teach me how to do remote events? i am just copy pasting em rn having no idea how they work

oh.. well i need to go now

i will try fix it tommorow

thank you for ur help