Bug - Fixed Buffbot buff casting weirdness

[starkid]

Sorry about the double post. We caused a new error, yay! err, I mean... booo.

So issue, when you try to change the target name to "yourself" it thinks you want to buff the player named yourself (and incidentally there is one), and not you.

Suggestion:

change line 296 to this.target = null;

 

[Bale]

The good news is that "yourself" is in hardcore so nobody is losing MP by buffing him. Yay!

 

[Grotfang]

Wow. Yourself could have been a very lucky player

r8647

Hopefully that should work. It worked fine for me, but we'll see. If null screws us over at some point, an alternative fix would be to assign "me" to the target string and check for that along with null and 0 length strings when assigning names. Since names must be 3 characters or over, me would also be unambiguous.

I'm really sorry about that -- I should have spotted that it would become a problem.

Thanks for the code, starkid.

 

[starkid]

Thanks for fixing the bug! =D