You do not want parenthesis around the commands. If you have more than 1 command to execute only if the 'if' statement is 'true', you must have curly braces around them, but with 1 command here, they are optional.
old:
if ($1 == !o) ( echo -a 0,4 mode $chan +o $2 )
new:
if ($1 == !o) { echo -a 0,4 mode $chan +o $2 }
or
if ($1 == !o) echo -a 0,4 mode $chan +o $2
Your code is TOO good, because it does more than it should do
Your code is assuming that everyone in channel is friendly and will not abuse this power.
Your code does not limit which channels this script will perform actions in, except is limited to 100% of all the channels where you are OP.
Your code does not limit which nicks can make these !requests.
Your code can send mode commands that do not need to be sent. You don't verify that $2 is even a nick in the channel, and does not verify that $2 now has the mode being removed. You do not even check if there is a $2 word.
For example, if I came into your channel and typed:
!deop XGamerAMD
Your script would react by removing OP from yourself
If you want to have people receive OP or Voice status as soon as they enter the channel, without using a !trigger, if their address matches the expected address, you might want to look at the control tab in address book:
/help Control
I only fixed some of the problems, because it helps to have more information about your final goal. So far I only suggest that you limit the trigger to action in 1 channel, and make sure $2 is a nick in channel, and if using !deop or !dv it verifies they have the mode being taken away. But more information is needed if you do not want to use the Control in the address book
on @:TEXT:*:#channelname:{
Tokenize 32 $strip($1-)
var %nick $2 | if (!$2) return | if (%nick isnum) return
if (!$nick($chan,$2)) return
if ($1 == !o) echo -a mode $chan +o $2
if ($1 == !deop) {
if (!$nick($chan,$2,o)) return
echo -a mode $chan -o $2
}
if ($1 == !v) echo -a mode $chan +v $2
if ($1 == !dv) {
if (!$nick($chan,$2,v)) return
echo -a mode $chan -v $2
}
}
