erronious bot logic

[c0ncrete]

i've come across a few places in the bot code where the decision making process is checking IsEngaged() on clients. IsEngaged() will ALWAYS return false on a client (unless that client is AI controlled), as it only checks for a populated hate list. as a result, GetFirstIncomingMobToMez() and other methods/functions like it are probably using more cycles than necessary due to checks failing where they should not. all instances where the mob in question is a client (and not AI controlled) should be checking the value of AggroCount. if the group member is a bot, IsEngaged() is correct.