When I click a part with a click detector, the function doesn't work. Why?

You're very close to the answer with this script, there's just a small error. Basically, rather than assigning >lighton to an integer, you want to create a Boolean (a value that can only be true or false). If we change this, your script should work:

Ok, our script is working. But believe it or not, we can simplify this, which you should always try to do, as it saves you work.

As you can see, there is less to type out now. What i've done is simple, for starters (we'll look at Fig.A) since our >lighton value is now a boolean, it's only ever going to be true or false right? And an if statement is going to execute the code within if the condition >lighton are true, if you look at Fig.B now, you'll see that i've changed your code a bit.

Basically, if >lighton is not true, then do whats within the else statement. Now we're talking. If you really wanted to you could shorten this code even further, but that isn't necessary so i'll leave that for another time.

Useful Links:

Conditional Statements (I recommend looking at Else and ElseIf statements.)

Booleans

Scripts run from the top down, what is happening is when it changes the transparency to 1 and lighton to 1, the next statementif lighton == 1 becomes true and it changes the transparenct back to 0, which is why it looks like its not working but it actually is, the code was just written wrong for what you need

You want to nest the if statement to ensure that the second condition wont run after changing your lighton value.

Here is a shorter way to do it with just 1 if statement

or just 1 line of code using quick maths