[SCRIPT] for(int) [SOLVED]

[Mudbill]

SetLightVisible does not return a boolean, so it cannot be used in a boolean-asking if-statement. That function will simply change an actual light, not tell you what it is.

Setter scripts can very rarely be used in if-statements. This is a setter, hence the SetLightVisible. Getter scripts are made for questioning. If there was one called GetLightVisible, it would likely return true if the light specified was visible (just like your original setup).

Only scripts that return a specific value can be used in if-statements. This is because you're comparing two values. For example you can compare a string with a string, an int with an int or a boolean with a boolean. Comparing cross-values can be tricky, but whenever you see that a script has the type "void," that means it's not returning anything. It's like comparing nothing to something; doesn't work.

What you can do here instead is to use local variables. Whenever you enable a light, use SetLocalVarInt("Lamp", 1);
Then in the question, use if(GetLocalVarInt("Lamp") == 1) instead.