No, my code is correct. The reason that you still use the $user_id instead of $uid is because of how the block calls the information from the DB. It is calling the $uid and setting $user_id equal to it.