Looks good, but here are some things you'd want to consider:


  • I still think silencing timers is a good idea.
  • Your isop check in the on JOIN event is on the wrong line. You can give the message even if you aren't an op. But you can't voice the person if you're not an op. I'd move the isop to the else line to prevent the error there.
  • You don't want to check isop in the DVoice alias at the top of the alias like that. You should be allowed to add someone to the list even if you are not opped. You can't devoice them, but you should still be allowed to add them (useful for permanent de-voice or long-term de-voice). I think the isop should only be on the +v/-v lines and let the /write lines through as shown in my example above.


Invision Support
#Invision on irc.irchighway.net