Consider what you have in your IF/ELSEIF/ELSE statements...
You are checking if the person is already opped and then if they are, you are opping them. The same for voice/halfop. I have a feeling you don't mean it that way.
Decide what exactly you want to happen. You don't need to op someone who is already opped, but you aren't going to op everyone who comes in. How do you know whether to op someone? You will need some kind of check in there to determine who should be opped, who voiced, etc.
Last edited by Riamus2; 28/03/11 10:24 PM.