egghelp.org community

simo · Owner Joined: 22 Mar 2015 Posts: 941

i also found if timer is running and nick is gone from channel it doesnt kill the running timer

CrazyCat · Posted: Sat Dec 11, 2021 7:57 pm Post subject:

Why do you add your if {[set thetimer ....} { killtimer xxxxx } in all procs ?
Just keep the if {[info exists ::ish($nick)]} { return } in proc auto:announcement, it will be ok when pple come out and in and regain hop.

And concerning the person out of the chan when timer runs... why catch unset ? The timer is killed if $nick is not on chan OR $nick is not halfop. With your catch, you first unset the "memory" timer ID and then try to use this variable to kill the timer.

I resume your code:

Arnold_X-P · Posted: Sun Dec 12, 2021 12:09 am Post subject:

hi simo

remove:
catch { unset ::ish($nick) }

and look for this :
killutimer $::ish($nick)

change it to:
unset -nocomplain -- ::is($nick)
_________________
Very Happy

thanks to that they help, that others learn Very Happy

simo · Owner Joined: 22 Mar 2015 Posts: 941

CrazyCat · Posted: Sun Dec 12, 2021 11:55 am Post subject:

I understand, but it was already managed.
If you want to kill the previous timer and start a new one (to reinit the 10 minutes delay), just unset ::ish($target) if it exists.

Note that in your code (proc auto:announcement), you have an error:

simo · Owner Joined: 22 Mar 2015 Posts: 941

oh true thanks for the correction this is what i have so far

simo · Owner Joined: 22 Mar 2015 Posts: 941

now i also get error:

SpiKe^^ · Posted: Sun Dec 12, 2021 3:31 pm Post subject:

simo · Owner Joined: 22 Mar 2015 Posts: 941

ive loaded it and changed to utimer to test with not to have to wait minutes to see results

but when nick is de-halfopped or left channel the timer doesnt get removed tho

simo · Owner Joined: 22 Mar 2015 Posts: 941

the repeat announcement doesnt seem to know what nicks to check for as it doesnt get feeded the nick after the first time nick is halfopped

perhaps if the nick is stored to be used by repeat annoucer it can do the checks

simo · Owner Joined: 22 Mar 2015 Posts: 941

perhaps the halfopped nick can be stored seperate in a var as well to be used by repeat announcement proc

simo · Owner Joined: 22 Mar 2015 Posts: 941

thanks for trying Spike^^ CrazyCat much apreciated

SpiKe^^ · Posted: Sun Dec 12, 2021 4:22 pm Post subject:

Maybe it's just a case issue with the array element name?
one last try...

simo · Owner Joined: 22 Mar 2015 Posts: 941

tested it same result thanks for the efforts u put into it tho Spike^^ and CrazyCat

SpiKe^^ · Posted: Sun Dec 12, 2021 5:51 pm Post subject:

OK, I bet I know what is going on:)

Let us assume that the recently added, infinitely repeating timer Does Not function as one might Hope and Expect it to....