why isn't this working?

[Devil]

Can someone help me out on this one? the bot returns this error.

Code: Select all

 
Tcl error [intro]: wrong # args: should be "intro from keyword arg"

and here's how the tcl is now.

Code: Select all

bind raw - "NOTICE" intro

set intro "being"
set intro1 "introduced"

proc intro {from keyword arg} {
global intro intro1
if {[string match *$intro* $arg]} {
if {[string match *$intro1* $arg]} {
putserv "PRIVMSG #sysops :SRV_INTRODUCED! [strftime \[%H:%M\]] $arg"
}
}
}

[DarkJFMan]

bind raw - "NOTICE" intro

set intro "being"
set intro1 "introduced"

proc intro {from keyword arg} {
global intro intro1
if {[string match *$intro* $arg]} {
if {[string match *$intro1* $arg]} {
putserv "PRIVMSG #sysops :SRV_INTRODUCED! [strftime \[%H:%M\]] $arg"
}
}
}

That's wrong.

[Devil]

yeah, but i got that form awyeah in some other help topic I cant remember now... It's pretty strange hehe, can you tell me how to get it to work?

[Alchera]

Post the original, unmodified, procedure.

[awyeah]

Well, why don't you just use "bind notc" instead of binding raw to NOTICE. Directly binding to notice would be easier, if you are only considering binding for PRIVATE NOTICES, CHANNEL NOTICES and *NOT* SERVER NOTICES.

Note:

Code: Select all

if {[string match *$intro* $arg]} {
if {[string match *$intro1* $arg]} {

Can be simplied further to:

Code: Select all

if {([string match *$intro* $arg]) && ([string match *$intro1* $arg])} {

[user]

Code: Select all

if {([string match *$intro* $arg]) && ([string match *$intro1* $arg])} {

Why do you use so many redundant parentheses? I've noticed you do that alot lately. ()'s are only needed when there's several parts of an expression that needs to be "grouped". The result of 'string match' is a single integer and to put parentheses around it make no sense.

[awyeah]

Actually I don't know but I like to group them togehter. Like when you have a long line of multiple functions, putting them into one line with ()'s in between makes them easier to seperate from each other.

To me they look nice when grouped; well not nice but they are easier for me to read, nothing else.

[Devil]

Guys, thank you very much for helping me, I really apreciate it