tree 06883a473a62b31c4fcfb53058888f79ddb0fe0b
parent 0e4e41a160afb3553ea0a895500770b290ab569b
author David Pursehouse <dpursehouse@collab.net> 1504050020 +0900
committer David Pursehouse <dpursehouse@collab.net> 1504054070 +0900

HookArgs: Use IdentifiedUser.getNameEmail to format account name

In most of the hooks, the account is being formatted as either:

  Name Surname (email@domain.com)

when the account has an email address, or:

  Name Surname

when the account does not have an email address. If the account name
is empty the anonymous coward name is used.

Instead of implementing this logic in the hook, defer it to the
existing implementation in IdentifiedUser which properly formats
the name when there is an email address:

  Name Surname <email@domain.com>

and also handles the cases where the account does not have an email
address but instead has a username, or falls back to the account Id.

The names used in the ref-update and commit-received hooks were already
formatted like this, so this change makes the rest of the hooks
consistent.

Change-Id: I99eeab1759dcedfdcc8c689ee6ac4599e3b58325