checking subscription on Who is messed

ID:263490
 
Feb 17 2007, 4:42 pm
Code:
Who()
            for(var/mob/M in world)
                if(M.key)
                    if(client.CheckPassport("14c8d1b50fba994b"))
                        usr << "<font color=red>~Name:[M.name] Key:[M.key]~</font>"
                        counter += 1
                    else
                        usr << "<font color=red>Name:[M.name] Key:[M.key]</font>"
                        counter += 1

            usr << "<br><u>Players Online</u>: <b>[counter]"
            counter = 0
            return ..()


Problem description: it says everyone is a subscriber, weird thing is i use the same if statement in my login and it works correctly on stating if you are a subscriber or not.

Feb 17 2007, 4:50 pm
if(client.CheckPassport()) is the same as if(src.client.CheckPassport())

You need to check the client of each mob instead of src's client.
Feb 17 2007, 5:09 pm
In response to Jon88
wow i dont believe i didnt catch that...duh

thanks a lot
Feb 18 2007, 5:41 am
In response to Chase_Hammer
Heh... you don't seem to catch far more obvious and visible errors. :O