Streetlight script not working at all

[2jammers]

I was trying to make a street light script for my game, but for some reason its just not working at all:

local Lighting = game:GetService("Lighting")
local LightGroup = script.Parent:FindFirstChild("Light")
local Light = LightGroup:FindFirstChild("Light")
local PrimaryLightColor

local IsLightOn = script:FindFirstChild("IsLightOn")
local IsLightOrange = script:FindFirstChild("IsLightOrange")

local SpotLight = Light:WaitForChild("SpotLight")

if IsLightOrange.Value == true then
	PrimaryLightColor = Color3.fromRGB(181, 122, 47)
else
	PrimaryLightColor = Color3.fromRGB(255, 255, 255)
end

Lighting:GetPropertyChangedSignal("ClockTime"):Connect(function()
	if (Lighting.ClockTime > 6.25) and (Lighting.ClockTime < 17.5) then
		print("es")
		IsLightOn.Value = false
		SpotLight.Enabled = false
		Light.Material = Enum.Material.Glass
		Light.Color = Color3.fromRGB(68, 65, 67)
		Light.Transparency = 0.35
	elseif (Lighting.ClockTime < 6.25) and (Lighting.ClockTime > 17.5) then
		print("e")
		IsLightOn.Value = true
		SpotLight.Enabled = true
		Light.Material = Enum.Material.Neon
		Light.Color = PrimaryLightColor
		SpotLight.Color = PrimaryLightColor
		Light.Transparency = 0
	end
end)

I already tried print debugging, it didn’t print anything in either of them.
Thanks in advance! (I believe the problem is in the property change signal)

[daisytheghostchild98 (bcman)]

You don’t need to use parenthesis in your if statements.
From what I’ve researched, it looks like you called it properly.

Lighting:GetPropertyChangedSignal("ClockTime"):Connect(function()
	if Lighting.ClockTime > 6.25 and Lighting.ClockTime < 17.5 then
		print("es")
		IsLightOn.Value = false
		SpotLight.Enabled = false
		Light.Material = Enum.Material.Glass
		Light.Color = Color3.fromRGB(68, 65, 67)
		Light.Transparency = 0.35
	elseif Lighting.ClockTime < 6.25 and Lighting.ClockTime > 17.5 then
		print("e")
		IsLightOn.Value = true
		SpotLight.Enabled = true
		Light.Material = Enum.Material.Neon
		Light.Color = PrimaryLightColor
		SpotLight.Color = PrimaryLightColor
		Light.Transparency = 0
	end
end)

[daisytheghostchild98 (bcman)]

Also, have you checked to see if the function gets called?

[2jammers]

Yeah for some reason the function is just not being called.

[daisytheghostchild98 (bcman)]

Try using this.

while wait() do
	if Lighting.ClockTime > 6.25 and Lighting.ClockTime < 17.5 then
		print("es")
		IsLightOn.Value = false
		SpotLight.Enabled = false
		Light.Material = Enum.Material.Glass
		Light.Color = Color3.fromRGB(68, 65, 67)
		Light.Transparency = 0.35
	elseif Lighting.ClockTime < 6.25 and Lighting.ClockTime > 17.5 then
		print("e")
		IsLightOn.Value = true
		SpotLight.Enabled = true
		Light.Material = Enum.Material.Neon
		Light.Color = PrimaryLightColor
		SpotLight.Color = PrimaryLightColor
		Light.Transparency = 0
	end
end

[2jammers]

It still doesn’t work for some reason. Im not really sure why though, for some reason when the time changes to night it still prints ‘es’

[daisytheghostchild98 (bcman)]

It might be an issue with variables then. Have you made sure that everything is in the right place?

[2jammers]

All of the variables are in the correct place!

[daisytheghostchild98 (bcman)]

Have you tried checking to see if the change actually happened in the instances?
e.g. IsLightOn shows true or false

[SillyMeTimbers (Silly)]

If you are still having issues with this I made a detailed explanation on a night-light system similar to this.

Simple Street Light Script! [Beginners Tutorial]

Although this is a very neat tutorial I personally don’t suggest doing this method entirely such as While true do loops You aren’t checking if the time is +18 or -6, Basically telling the script to only run when the time strikes 12 - 6 am Instead of easily updating all the objects with one script you have every object contain a script, I personally don’t suggest having any scripts in the workspace but that’s just how I was taught by my brother. What you’ll need For my suggested tutorial o…

[2jammers]

Alright, I found the issue. I wasn’t using a daylight cycle which set GetMinutesAfterMidnight(). Instead I was changing the time manually which obviously did not work.