Unless /Player is set with /mob as a parent type, that won't work.
*points to the code in the OP*
It is :P
I'd be remiss not to point out that mob/var/list/names[] is a bad idea. The /list and [] together are redundant, but I'd take off the [] and just not initialize the list until it's needed. You should avoid initializing a list for every single mob if you can help it.
Lummox JR
OK, done. And I went ahead and made names a var only for /Player as well (changing it to mob was my attempt at making it work)
*Edit, okay so Jamesburrow's eye still has a lid and its not exactly on fire. But still, I like making outdated LOTR movie references.