It's not clear why you separated this from _form_salutation(). That function could do the checking for whether there is a name. Is there a reason to abstract it?