Forums Scripts & Popups Auto Slap

Auto Slap
.On:{ if (%aslap = $null) { set %aslap 1 | echo -a Auto Slap Is On } }
.Off:{ if (%aslap = 1) { set %aslap | echo -a Auto Slap Is Off } }

on *:action:slaps *:#:{
if (%aslap = 1) { describe # slaps $nick around a bit with a large trout
}
else {
halt
}
}

Can I do this more shorter?

Well, ignoring unncessary { }, you could shorten the On/Off popup:

.On:{ set %aslap 1 | echo -a Auto Slap Is On }
.Off:{ unset %aslap | echo -a Auto Slap Is Off }

You don't need the if statement, because if you set it 'On' then %aslap will have the value of '1', and if you set it off, %aslap no longer exists.

You can also shorten the on action event:

on *:action:slaps *:#:{
if (%aslap = 1) {
describe # slaps $nick around a bit with a large trout
}
}

You don't need to tell the script to halt, it will only trigger if %aslap is '1' anyway. So if it isn't 1, it won't do anything.

Regards,

Thanks

SOrry for dubble post but how can I make it so it just is for me?

change the second # to $chan. otherwise you'll be doing it on whatever channel is active

Not really, in a remote, # is equal to $chan, not to the active channel