This sort of code breaks if you ever move components in and out of containers or rename them.
Consider using sendMessage() instead and adding a message handler on the target field to set focus on itself. This is a practice worth adopting and whoever has to maintain your code will appreciate it.
Also, please see Wiki - how to post code on this forum.