Other than that you're sending the "text on" and "text off" to the channel/query when you could just use a command to enable/disable it. Beyond that, using groups would probably be a better method, though that really isn't so important.

That and someone may manage to write a line that starts with "text". Your check will always turn off text if $2 isn't equal to "on" and even if you checked $2 for off, you could still have someone manage to write a line starting with "text on" or "text off" since you're not limiting the text to 2 words.

Invision Support
#Invision on irc.irchighway.net