Kick On Idle

[Cult]

on *:text:!icheck &:#Leb:{ 
  if $nick isop # { 
    if $2 == on { 
      .timericheck 0 60 icheck 
      msg # IdleCheck enabled 
    } 
    elseif $2 == off { 
      .timericheck off 
      msg # IdleCheck disabled 
    } 
  } 
} 

alias icheck { 
  var %i = 1 
  while $nick(#Leb,%i) { 
    if (($v1 isreg #Leb) && ($nick(#Leb,%i).idle > 600)) { 
      ban -k #Leb $nick(#Leb,%i) 2 No Idling. Thank you. 
      inc %i 
    } 
    inc %i 
  } 
}

[ZAcK]

Don't you think you should get the code to check if you are an OP or not? I think the way you have it will attempt to kick even if you are not OPed
I would add two checks, one in the event and the other same line with the while loop (just before the mode/kick line), so the code wouldnt run the check if you are not opped and if an OP deoped you for some reason, the loop would halt.
I might be wrong, but I thought I should share my thoughts

Happy New Year everyone

[Cult]

sorry for undeleting your post bro well it was a pleasure to undelete it, coz it`s a forum in here and we shall discuss stuff, and if i have something wrong and someone was trying to fix it, that is good
i`ll try to fix it bro.
happy new year u too bro..
wish u all luck in the new year.

[anmlhckr]

I dont think theres need to add WHILE LOOPS , thx.

[ZAcK]

No need for new while loops! actually it is so simple, a @ and && ($me isop #leb) would do the trick.

on @*:text:!icheck &:#Leb:{ 
  ; i just added the @ to the previous line
  if ($nick isop #) { 
    if ($2 == on) { 
      .timericheck 0 60 icheck 
      msg # IdleCheck enabled 
    } 
    elseif ($2 == off) { 
      .timericheck off 
      msg # IdleCheck disabled 
    } 
  } 
} 

alias icheck { 
  var %i = 1 
  while $nick(#Leb,%i) && ($me isop #leb) { 
    ; and && ($me isop $chan) to this line.
    if (($v1 isreg #Leb) && ($nick(#Leb,%i).idle > 600)) { 
      ban -k #Leb $nick(#Leb,%i) 2 No Idling. Thank you. 
      inc %i 
    } 
    inc %i 
  } 
}

I didnt test it, but i guess it should work fine.

Cult, next time please email or send me a messege before undeleting/editing my posts, I deleted it for a reason. No worries this time.

Have fun all

[Cult]

Originally Posted by ZAcK

No need for new while loops! actually it is so simple, a @ and && ($me isop #leb) would do the trick.

on @*:text:!icheck &:#Leb:{ 
  ; i just added the @ to the previous line
  if ($nick isop #) { 
    if ($2 == on) { 
      .timericheck 0 60 icheck 
      msg # IdleCheck enabled 
    } 
    elseif ($2 == off) { 
      .timericheck off 
      msg # IdleCheck disabled 
    } 
  } 
} 

alias icheck { 
  var %i = 1 
  while $nick(#Leb,%i) && ($me isop #leb) { 
    ; and && ($me isop $chan) to this line.
    if (($v1 isreg #Leb) && ($nick(#Leb,%i).idle > 600)) { 
      ban -k #Leb $nick(#Leb,%i) 2 No Idling. Thank you. 
      inc %i 
    } 
    inc %i 
  } 
}

I didnt test it, but i guess it should work fine.

Cult, next time please email or send me a messege before undeleting/editing my posts, I deleted it for a reason. No worries this time.

Have fun all

thanx for the fixing bro