As the purpose of the function is to get the user display name, it would improve the flow of the method to test for false in the if statement and return null there, lose the else and end with return $this->data->user_display_name (IMO)