also it would have been easier (as I mentioned before) to just change player to my. player is built-in pointer. As you have already wizard pointer created, you don't have to attach this action to player pointer anymore.
If you choose to use player then replace wizard with player. But I guess you use wizard pointer somewhere else, so this wouldn't be a good idea!

Cheers!