making pubm work on only one channel? - Linked to loops thrd

[Weirdo]

Code: Select all

#bind pubm - * pubm:pubnotc

proc pubm:pubnotc {nick uhost handle chan text} {
	global chanl network notice active
	if {$active == 1} {
		putnotc $nick $notice(public)
		putlog "Notice sent to $nick" 
	}
}

This is what i have so far. I want to make it so that it doesnt flood out the bot, ie, make a queue, so that multiple notices wont be queued for one person, and so that it will only read the text on the $chanl specified, instead of all channels.

Any way of doing this?

This is the option i have to the other thread i made, the figuring out loops.
the basic outocme of this is a way to tell people in channel, that we are moving networks. Since this is happening tomorrow night, i have to find a way through this pretty quickly

[strikelight]

Code: Select all

#bind pubm - * pubm:pubnotc

proc pubm:pubnotc {nick uhost handle chan text} {
	global chanl network notice active
	if {$active == 1} {
		putnotc $nick $notice(public)
		putlog "Notice sent to $nick" 
	}
}

This is what i have so far. I want to make it so that it doesnt flood out the bot, ie, make a queue, so that multiple notices wont be queued for one person, and so that it will only read the text on the $chanl specified, instead of all channels.

Any way of doing this?

This is the option i have to the other thread i made, the figuring out loops.
the basic outocme of this is a way to tell people in channel, that we are moving networks. Since this is happening tomorrow night, i have to find a way through this pretty quickly

Are you sure you want to send a notice to someone everytime text is typed in the channel? (btw, your bind is commented out, so this won't happen anyways at the moment).

But in any event, I would recommend using 'puthelp "NOTICE $nick :text...' as opposed to putnotc.. I'm not sure, but I believe the putnotc procedure uses the putserv queue which could allow the bot to flood itself off irc.

As for a simple queue... You could either make a list or an array... I'd recommend an array, as it would be easier to maintain (ie. deletion of an element after x amount of time), such as:

Code: Select all

bind pubm - * pubm:pubnotc

proc pubm:pubnotc {nick uhost handle chan text} {
   global chanl network notice active netchange
   if {($active == 1) && (![info exists netchange($uhost)])} {
     puthelp "NOTICE $nick :$notice(public)"
     putlog "Notice sent to $nick"
     set netchange($uhost) 1
    #unset the value after an hour (i find utimers more stable than timers)
     utimer 3600 [list unset netchange($uhost)]
   }
}

Hope this helps

[Papillon]

as for making it only react in one channel.... unless you have multiple channels in $chanl... you could just use

Code: Select all

bind pubm - "#channel *" pubm:pubnotc

[Weirdo]

that works nicely, thank you very much