Scripting Helpers is winding down operations and is now read-only. More info→

Client script remoteevent problem?

Asked by

hasanchik 49

4 years ago

Thank you for reading this.

So, for my platformer game, i want to make a flagpole at the end something like super mario bros. I have this custom control script and a serverscript.

(Also the purpose of the controlOn variable is for the cutscene to happen without the player moving)

This is the error that pops up:

1  20:19:13.540 - Players.hasanchik.PlayerScripts.ControlScript:46: attempt to index boolean with 'Name'
19:13.540 - Stack Begin
19:13.541 - Script 'Players.hasanchik.PlayerScripts.ControlScript', Line 46
19:13.541 - Stack End

Server script:

01local ts = game:GetService("TweenService")
02local rs = game:GetService("ReplicatedStorage")
03local remote1 = rs:WaitForChild("PlayerControl")
04local debris = game:GetService("Debris")
05local db = false
06local bottom = script.Parent.Flagpole.Bottom
07local top = script.Parent.Flagpole.Top
08local flag = script.Parent.Flagpole.Flag
09local pole = script.Parent.Flagpole.Pole
10function guiBegin(player)
11    local guiCoroutine = coroutine.create(function()
12        local gui = script.FadeOut:Clone()
13        local frame = gui:FindFirstChild("FadeOutFrame")
14        gui.Parent = player.PlayerGui
15        local goal = {BackgroundTransparency = 0}
16        local tweenInfo = TweenInfo.new(1, Enum.EasingStyle.Linear, Enum.EasingDirection.InOut, 0, false, 0)
17        local fadeIn = ts:Create(frame, tweenInfo, goal)
18        fadeIn:Play()
19        wait(2.1)
20        local goal = {BackgroundTransparency = 1}
21        local tweenInfo = TweenInfo.new(1, Enum.EasingStyle.Linear, Enum.EasingDirection.InOut, 0, false, 0)
22        local fadeOut = ts:Create(frame, tweenInfo, goal)
23        fadeOut:Play()
24        wait(1)
25        debris:AddItem(gui, 0.1)
26    end)
27    coroutine.resume(guiCoroutine)
28end
29pole.Touched:Connect(function(part)
30    local HRP = part.Parent:FindFirstChild("HumanoidRootPart")
31    local player = game.Players:GetPlayerFromCharacter(part.Parent)
32    if HRP and player and not db then
33        db = true
34        HRP.Anchored = true
35        local goal = {Position = Vector3.new(HRP.Position.X,bottom.Position,HRP.Position.Z)+Vector3.new(0,3,0)}
36        local tweenInfo = TweenInfo.new(1, Enum.EasingStyle.Sine, Enum.EasingDirection.InOut, 0, false, 0)
37        local Glide = ts:Create(HRP, tweenInfo, goal)
38        Glide:Play()
39        bottom.LevelComplete:Play()
40        wait(1)
41        HRP.Anchored = false
42        db = false
43        remote1:FireClient(player, false)
44    end
45end)
 View all 45 lines...

Client script:

01local player = game.Players.LocalPlayer
02local RunService = game:GetService("RunService")
03local ContextActionService = game:GetService("ContextActionService")
04local rs = game:GetService("ReplicatedStorage")
05local remote1 = rs:WaitForChild("PlayerControl")
06local controlOn = true
07 
08local jumping = false
09local leftValue, rightValue = 0, 0
10 
11local function onLeft(actionName, inputState)
12    if inputState == Enum.UserInputState.Begin then
13        leftValue = 1
14    elseif inputState == Enum.UserInputState.End then
15        leftValue = 0
16    end
17end
18 
19local function onRight(actionName, inputState)
20    if inputState == Enum.UserInputState.Begin then
21        rightValue = 1
22    elseif inputState == Enum.UserInputState.End then
23        rightValue = 0
24    end
25end
26 
27local function onJump(actionName, inputState)
28    if inputState == Enum.UserInputState.Begin then
29        jumping = true
30    elseif inputState == Enum.UserInputState.End then
31        jumping = false
32    end
33end
34 
35local function onUpdate()
36    if player.Character and player.Character:FindFirstChild("Humanoid") and controlOn then
37        if jumping then
38            player.Character.Humanoid.Jump = true
39        end
40        local moveDirection = rightValue - leftValue
41        player.Character.Humanoid:Move(Vector3.new(moveDirection,0,0), false)
42    end
43end
44 
45remote1.OnClientEvent:Connect(function(player2, controlOn2)
46    if player2.Name == player.Name then
47        controlOn = controlOn2
48    end
49end)
50 
51RunService:BindToRenderStep("Control", Enum.RenderPriority.Input.Value, onUpdate)
52 
53ContextActionService:BindAction("Left", onLeft, true, "a", Enum.KeyCode.Left, Enum.KeyCode.DPadLeft)
54ContextActionService:BindAction("Right", onRight, true, "d", Enum.KeyCode.Right, Enum.KeyCode.DPadRight)
55ContextActionService:BindAction("Jump", onJump, true, "w", Enum.KeyCode.Space, Enum.KeyCode.Up, Enum.KeyCode.DPadUp, Enum.KeyCode.ButtonA)
 View all 55 lines...

Can you help me? Thanks.

See the comment on my answer User#30567 0 — 4y

3 answers

Answered by

DeUltimate23 142

4 years ago

So basically the problem here I think is that your the false argument that u put in the :FireClient event is set as the player2 value in your clientscript. The "Player" argument in your FireClient part is being not sent to the client script resulting into the false argument being the first parameter in your OnClientEvent part. So basically the script is thinking that your first parameter is the boolean false.

My suggestion is that you do this

`1`	`remote1:FireClient(player,player.Name,false)`

then I think its supposed to work.

If this did'nt work then I am will try to look more into it

still, why cant the script set the variable controlOn hasanchik 49 — 4y

i mean the if statement doesent work hasanchik 49 — 4y

on line 46 in the client script hasanchik 49 — 4y

That would return "Players.hasanchik.PlayerScripts.ControlScript:46: attempt to index string with 'Name'" User#30567 0 — 4y

View all comments (3 more)

no both player2.Name and player.Name return hasanchik (my name) do the if statement would have to work hasanchik 49 — 4y

Because you sent player.Name as the argument remember? So instead of doing player2.Name == player.Name you can just do player2 == player.Name. Hopefully this works DeUltimate23 142 — 4y

wait what the nani User#30567 0 — 4y

Answered by

ThatDevTim 188

4 years ago

Either player2 or player is a boolean value(True/False)

Let me take a deeper look into your code and ill edit this when I find the line that the boolean is created.

Happy scripting, Tim

player is the localplayer, and player2 is the 31 line of code and then the player2 is fired at the 43 line in a serverscript hasanchik 49 — 4y

Got it. ThatDevTim 188 — 4y

There must be a problem with the 43 line of code in the serverscript, but the boolean is after the player. the boolean that is fired from the serverscript is in the client script controlOn2 which is on the second place in OnClientEvent:Connect(). controlOn2 is supposed to be a boolean, the first parameter is supposed to be a player object. I have no idea how this would error. hasanchik 49 — 4y

Have you tried using a print(Player2) and a print(Player) right before line 46? MIght be able to figure out the issue from that. ThatDevTim 188 — 4y

View all comments (19 more)

i tried printing, but it just printed hasanchik 49 — 4y

false (for player2 somehow) and nil for controlOn2 hasanchik 49 — 4y

Ok, we found our issue ThatDevTim 188 — 4y

player2 is the boolean value ThatDevTim 188 — 4y

The Server script is parsing player2 as false ThatDevTim 188 — 4y

What how hasanchik 49 — 4y

Ok DeUltimate23 answered the question but the if statement does not wotk hasanchik 49 — 4y

work hasanchik 49 — 4y

ITS DRIVING ME CRAZY hasanchik 49 — 4y

This should be in a comment. You literally translated the error message to english. THAT'S IT User#30567 0 — 4y

what do you mean by that hasanchik 49 — 4y

Try to remove the if statement I think it seems unecessary DeUltimate23 142 — 4y

No, Its for the checking for a specific player. If the statement wasnt there, everbody would freeze for like 5 seconds hasanchik 49 — 4y

nevermind hasanchik 49 — 4y

Have you tried using a print after the if statement to check if it gets past? DeUltimate23 142 — 4y

i figured out something, you didnt need the if statement because the fireclient line of code in the serverscript, the first parameter was the player to which the signal the script it sensed to hasanchik 49 — 4y

why couldnt you say that to me hasanchik 49 — 4y

i like kinda did, but glad to hear that it worked out :) DeUltimate23 142 — 4y

ok i admit i was dumb here hasanchik 49 — 4y

Answered by

User#30567 0

4 years ago

On line 43 of the Script you have:

`1`	`remote1:FireClient(player,` `false)`

On line 45 of the LocalScript you have:

`1`	`remote1.OnClientEvent:Connect(function(player2, controlOn2)`

On line 46 of the LocalScript you have:

`1`	`if` `player2.Name` `==` `player.Name` `then`

Since the first argument to an event is always "Player", the second argument (which is false) would be "player2" and "controlOn2" wouldn't even exist.

This can be fixed easily by replacing line 45 to 49 with

1remote1.OnClientEvent:Connect(function(controlOn2)
2    controlOn = controlOn2
3end)

BUT i have to do the CHECK to make sure the script only targets one player and not all players hasanchik 49 — 4y

IT WILL STILL WORK User#30567 0 — 4y

BUT then everyone wont be able to control their character for like 5 seconds hasanchik 49 — 4y

View all comments (3 more)

DID YOU TRY User#30567 0 — 4y

Ok i will hasanchik 49 — 4y

i tried, but it didnt even work at all hasanchik 49 — 4y

Client script remoteevent problem?

3 answers

Answer this question