on join bind help

[evotech]

Hi, trying to get this script to work and send a notice on join. as far as i can tell it's according to the documentation.

Code: Select all

bind pub - .streams cmd:stream
bind pub - !test cmd:test
bind pub - !whoburnedthehousedown
bind join - * cmd:stream

proc cmd:stream { nick uhost handle chan text } {
   catch { exec cat /home/evotech/eggdrop/scripts/urls | perl /home/evotech/eggdrop/scripts/streams.pl } streams
   putnotc $nick "$streams"
}

It works if called, but not on join, whats wrong with the 'bind join - * cmd:stream' line?

Probably something minor

thanks

[Trixar_za]

Try

Code: Select all

bind join - "#* *!*@*" cmd:stream

What your missing is the channel and user mask part of the bind.

[nml375]

The bind is just fine, "*" is a wide enough mask to match pretty much anyone joining the channel. However, the argument list of cmd:stream does not match the required parameters of the join binding; it only passes on 4 (nickname, user@host, handle, and channel) while your cmd:stream proc has 5, as you already use it with pub bindings.

There are two fixes for this;
1. use a different proc for your join binding

Code: Select all

bind join - * join:stream
proc join:stream {nick host handle channel} {
  catch {exec cat /home/evotech/eggdrop/scripts/urls | perl /home/evotech/eggdrop/scripts/streams.pl} streams
  putnotc $nick $streams
}

2. Make the last argument (text) optional by assigning it a default value:

Code: Select all

bind join - * cmd:stream
proc cmd:stream {nick uhost handle chan {text ""}} {
  catch {exec cat /home/evotech/eggdrop/scripts/urls | perl /home/evotech/eggdrop/scripts/streams.pl} streams
  putnotc $nick "$streams"
}

[caesar]

bind join <flags> <mask> <proc>
procname <nick> <user@host> <handle> <channel>

the join procedure expects 4 arguments while you have 5. You can trick it by replacing:

Code: Select all

proc cmd:stream { nick uhost handle chan text } {

with:

Code: Select all

proc cmd:stream {nick uhost handle chan {text ""}} {

[evotech]

cool!

idk if i even need the text paramenter, ill try just taking it out first
edit: without it the public call doesnt work so i used the second option and it works great

But yeah, many thanks!