Problem with events

RazoTheCreator · 2019-06-07T13:48:25.947Z

Hey. I’m a newbie “scripter”, trying to use some events.

Here’s the part of the client-side script:

RegisterCommand(“test”,function(source)
local asd = “asd”
TriggerServerEvent(“testEvent”,source,asd)
end)

And there’s the server-side:

RegisterNetEvent(“testEvent”)
AddEventHandler(“testEvent”,function(source,asd)
TriggerClientEvent(‘chat:addMessage’, {
color = { 255, 255, 0},
multiline = true,
args = {“test”}
})
end)

I’ve also tried to put the RegisterNetEvent to the client side instead, didn’t work either.
Any tips would be appreciated!

TheIndra · 2019-06-07T13:54:41.954Z

RazoTheCreator:

TriggerClientEvent(‘chat:addMessage’, {

Second parameter is who you want to send to event to (so -1 for everyone or source id)

SpikE_Odets · 2019-06-07T13:59:12.690Z

TheIndra:

Second parameter is who you want to send to event to (so -1 for everyone or source id)

As someone new to lua as well. Make sure you add.

local _source = source.

And then pass -1 or _source as who to send it to

SpikE_Odets · 2019-06-07T14:05:20.575Z

Also look at the script as you are passing a string from client to server that has no use.

asd is set in client and sent but never used in server script. Not code breaking but something to think about.

RazoTheCreator · 2019-06-07T14:09:49.404Z

Yeah, that’s something I’m actually aware of.

Also, I’m not sure that the event gets “called” at all, that’s why I’m asking this.

And what do you mean TheIndra, where does that second parameter go?

SpikE_Odets · 2019-06-07T14:24:22.883Z

If you haven’t checked out

docs.fivem.net

TriggerClientEvent

Triggers an event on the specified client(s), and passes on any additional arguments. Syntax TriggerClientEvent(string eventName, int playerId[, ...]) Required arguments eventName: A string representing the event name to call on the client....

That will tell you what parameters you can work with. For the native your looking at

the first param is he client event name…
the second is who you want to send it to. Can use -1. To send it to everyone on the server or source to send it to only the person that send the server trigger
The thrid is what you want to pass from the client to the server

Hopefully that helps make a bit of sense for you

RazoTheCreator · 2019-06-07T16:15:30.017Z

Thanks, I’ve written the missing parameter, but nothing happens, still.
The main point of this question is that maybe you can find something about the events that I’m not doing right.

stemcmyk · 2019-06-07T16:21:40.813Z

Its RegisterNetEvent in client side
and you cant register commands in client side
(You did a typo in your post? )

TheIndra · 2019-06-07T16:27:20.938Z

stemcmyk:

Its RegisterNetEvent in client side

Why he wants to call a event on server?

stemcmyk:

cant register commands in client side

Yes you can

RazoTheCreator · 2019-06-07T16:31:21.010Z

Alright, so the problem is a bit more complex.

This is the line in the client-side script that matters now:

TriggerServerEvent(“onSkinChange”,source,skin)

And this is the server-side code:

RegisterNetEvent(“onSkinChange”)
AddEventHandler(“onSkinChange”,function(source,skinstr)
local pnev = GetPlayerName(source)
TriggerClientEvent(‘chat:addMessage’, -1, {
color = { 255, 255, 0},
multiline = true,
args = {“osc " … skinstr}
})
MySQL.Async.fetchAll(“UPDATE fivem SET skin=@skin WHERE name=@nev”,
{[”@skin"] = skinstr, [“@nev”] = pnev},
function(result)
end)
end)

Nothing happens, that’s my problem.

stemcmyk · 2019-06-07T16:34:45.161Z

When you make a client sided event you do

RegisterNetEvent("event")
AddEventHandler("event", function(source)
end)

And to trigger a client event you do “TriggerClientEvent(event)”
--------------------------------.
And when you create a event server-side

RegisterServerEvent("event")
AddEventHandler("event", function(source)
end)

And to trigger a server event you do “TriggerServerEvent(event)”

TheIndra · 2019-06-07T16:43:20.780Z

stemcmyk:

When you make a client sided event you do

No shit but what has that todo with the original topic? this guy has no clientsided event

RazoTheCreator:

AddEventHandler(“onSkinChange”,function(source,skinstr)

Is this is to change people their own skin, don’t let people send source as you can never trust the client and they might send other people’s source, there’s a global source variable which can be used

RazoTheCreator:

args = {“osc " … skinstr}

This should probably be {“osc”, skinstr}

stemcmyk · 2019-06-07T17:00:04.216Z

RazoTheCreator:

RegisterNetEvent(“onSkinChange”)

TheIndra, you dont use netevent on server-side

TheIndra · 2019-06-07T17:03:54.933Z

stemcmyk:

TheIndra, you dont use netevent on server-side

Uh yes you do? how is the client otherwise supposed to call it? I suggest you take a look at Listening for events - Cfx.re Docs

RazoTheCreator · 2019-06-07T18:03:10.656Z

RazoTheCreator:

local pnev = GetPlayerName(source)

This line caused all of my problems. Appearently it is faulty, and it caused the code to stop there, without doing anything after it. I gave the trigger the name itself as a parameter instead of the source, and now it works. Thanks for everything.