egghelp.org community

[MIODude]

I just want a simple procedure to only be invoked if the user isn't an op or doesn't have voice...

[arfer]

Logically you need && instead of || because you want to trigger if user is not op AND is not voice.

You are also using braces to seperate the two commands determining the logical result instead of parenthesis. Braces generally hinder variable, command and backslash substitution.

I'm not entirely sure what putchan means. Perhaps you mean one of the Eggdrop Tcl output commands putserv or putquick.

The argument 'args' has special meaning, do not use it in this context.

Generally I would use return 0 or even simply return at the conclusion of a proc, though there are special circumstances where you may want to deliberately return 1. Refer to 'return values' in tcl-commands.doc. In any case you don't need two such statements as per your code.

[MIODude]

Thanks! that did it..

I'm not sure about the putchan myself - it came with the original script that i was modifying

[nml375]

The putchan command is provided by the alltools.tcl script provided with eggdrops.

As for the logics, you don't need those parenthesis, as the not-operator (!) has higher priority than either and-operator (&&) or or-operator (||). No harm in including them though.
_________________
NML_375, idling at #eggdrop@IrcNET