Error: invalid command name ""

[ache]

I have one more problem.
I get this error:

Code: Select all

Tcl error [pub:checkgreet]: invalid command name ""

This is the code i have for the pub:checkgreet proc atm:

Code: Select all

proc pub:checkgreet { nick uhost handle channel arguments } {
  list arguments $arguments
  set greetnick [lindex $arguments 0]
  set greetchannel [lindex $arguments 1]
  set greettext [lrange $arguments 2 end]
  if {([string index $greetchannel 0]=="#")} {
    [pub:addgreet $nick $greetnick $greetchannel $greettext]
	}	elseif {[string range $greetchannel 0 2]=="all"} {
    [pub:addgreet $nick $greetnick $greetchannel $greettext]
  } else {
    putquick "NOTICE $nick : Please state a channel to greet in. \026!chelp !addgreet\026 for more info."
    return
  }
}

And this is the code i have for the pub:addgreet proc:

Code: Select all

proc pub:addgreet {nick greetnick greetchannel greettext} {
  set greetdb "scripts/charlie.tcl/greets.db"
  set greet "$greetchannel $greetnick $greettext"
  set searchString "$greetchannel $greetnick"
  set greetData [main:filetodata $greetdb]
  set greetIndex [lsearch $greetData $searchString*]
  if {($greetIndex >= "0")} {
    set greetData [lreplace $greetData $greetIndex $greetIndex $greet]
    putquick "NOTICE $nick : Changed greet message for $greetnick on $greetchannel."
  } else {
    lappend greetData "$greet"
    putquick "NOTICE $nick : Added greet message for $greetnick on $greetchannel."
  }
  set greetChannel [open $greetdb "w"]
  set greetData [lsort $greetData]
  set greetData [join $greetData "\n"]
  puts -nonewline $greetChannel "$greetData\n"
  close $greetChannel
}

thanks for any help.

[Alchera]

Try changing:

Code: Select all

elseif {[string range $greetchannel 0 2]=="all"} {

To:

Code: Select all

elseif {([string range $greetchannel 0 2] == "all")} {

[ache]

Tried it but with no luck.
still the same error..

[Sir_Fz]

remove the brackets from

Code: Select all

[pub:addgreet $nick $greetnick $greetchannel $greettext]

(both)

[ache]

worked like charm!
thanks Sir_Fz

[Alchera]

Damn, I must be blind.

[ache]

Damn, I must be blind.

exactly what i though when i got the solution.
err not that you would be blind, but me.
so simple.. that will probably never happen again

[Alchera]

Sometimes you just cannot see for the trees. LOL