Color3 value not being updated correctly when used in a variable?

Asked by

7 years ago

Edited 7 years ago

I'm currently working on a system whereby a player selects a colour in a GUI, which is then sent to the server to change the Color3 value of a certain group of parts using a RemoteEvent. I'm experiencing a problem whereby, when called in a variable named 'colour', the Color3 value of the parts is set to 0,0,0.

There are no errors in the Output Log.

Here is the associated LocalScript (I have annotated so it is clearer what I am doing and if I've made an error somewhere):

01local replicatedStorage = game:GetService("ReplicatedStorage")
02local updateColourEvent = replicatedStorage:WaitForChild("UpdateColourEvent")
03-- (Above) Setting up RemoteEvent
04 
05wait(1)
06 
07local hilt = game.Workspace:WaitForChild("Hilt1") -- The model that holds the parts
08local forward = script.Parent.Forwards -- The button pressed to trigger the colour change
09local back = script.Parent.Backwards -- Not used yet
10colour = script.Parent.Colour.BackgroundColor3 -- The current colour the client has selected
11 
12colours = { -- Dictionary to hold the available colours
13    ["Red"] = Color3.new(1,0,0),
14    ["Green"] = Color3.new(0.4,1,0.5),
15    ["Blue"] = Color3.new(0.2,0.2,1),
16    ["Purple"] = Color3.new(0.6,0,1),
17    ["Cyan"] = Color3.new(0,1,1),
18    ["Orange"] = Color3.new(1,0.4,0),
19    ["Pink"] = Color3.new(1,0,0.9)
20}
21 
22 
23 
24forward.MouseButton1Click:connect(function()
25    print("Setting colour...")
26    print(colour)
27    if colour == colours["Red"] then
28        print("Current colour is red, setting to green.")
29        script.Parent.Colour.BackgroundColor3 = Color3.new(0.4,1,0.5)
30        colour = script.Parent.Colour.BackgroundColor3
31    end
32    updateColourEvent:FireServer(colour)
33end)
34 
35-- (Above) Function called when player clicks button to update colour. A test is done to find the current selected colour, and then uses that to switch to the next colour in the list. The selected colour is updated in the variable 'colour', and sent to the Server with the RemoteEvent.
 View all 35 lines...

And here is the associated Server Script (also annotated):

01--LukeSmasher
02 
03local replicatedStorage = game:GetService("ReplicatedStorage")
04local updateColourEvent = Instance.new("RemoteEvent", replicatedStorage)
05local hilt = script.Parent
06 
07updateColourEvent.Name = "UpdateColourEvent"
08 
09updateColourEvent.OnServerEvent:connect(function(player,colour)
10    print(colour) -- Prints value correctly
11    for i, child in pairs(hilt:GetChildren()) do
12        if child.Name == "Primary" then -- Parts named 'Primary need to be re-coloured to the value of x
13            child.Color = Color3.new(colour) -- Parts' colour given value of 0,0,0
14            print(colour) -- 'colour' still has correct value when printed
15        end
16    end
17end)

As described by my annotations, when the players clicks on the button to switch colour, a variable named 'colour' is updated and sent to the Server with the RemoteEvent (in ReplicatedStorage). The Server Script takes the Color3 value in the variable and sets all parts named "Primary" in the model to have that colour.

Instead of setting to the value of 'colour', however, the parts get the Color3 Value, 0,0,0.

Any ideas as to what I've done wrong would be hugely appreciated. I am more than happy to provide more information on my set-up if necessary.

Color3 value not being updated correctly when used in a variable?

1 answer

Answer this question