Thats cool you fixed it. I just wanted to talk real quick about retrieving information.
When storing data I try to store it to make it the quickest and easiest to return. So this script should have a list of nicknames. There are a few ways of doing this. I do not suggest a while loop for this.
You were shown $istok which is a great way to check here.
I can have a variable like.
%badnick = nick1,nick2,nick3,nick4
then I can check
if ($istok(%badnick,$nick,44)) { }
Alot less code and easier on mIRC than a while loop in the join event.
What I was going to suggest though, is that since you already have a hash table, instead of using a while loop to scan through the table, why not set the nicknames as the item names.
So: /hadd badnick $nick $chan
$chan isnt important. I would store any information I may need in my script. If there wasn't any additional info I needed I may just make it $true.
The diffence is, instead of a while loop, all you say is:
if ($hget(badnick,$nick)) { ban --- }
On an aside, your new script here sets the variable at 1, then dec's it. This will only run through the loop once since when it gets to the first time to dec %i it will change to 0 and end. If this is indeed your intention I'd get rid of the loop. Otherwise you probably want the total number of nicknames or something, then dec those. OR start at 1 and INC the variable until it returns a $null value.
Cheers and have fun!
