diff options
Diffstat (limited to 'fun/clbot/clbot.go')
| -rw-r--r-- | fun/clbot/clbot.go | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fun/clbot/clbot.go b/fun/clbot/clbot.go index 317d99e22f05..ee250f706c8c 100644 --- a/fun/clbot/clbot.go +++ b/fun/clbot/clbot.go @@ -156,11 +156,11 @@ func runIRC(ctx context.Context, ircCfg irc.ClientConfig, sendMsg <-chan string) } } -func username(p gerritevents.PatchSet) string { +func username(a gerritevents.Account) string { options := []string{ - p.Uploader.Username, - p.Uploader.Name, - p.Uploader.Email, + a.Username, + a.Name, + a.Email, } for _, opt := range options { if opt != "" { @@ -243,13 +243,13 @@ func main() { if e.Change.Project != *notifyRepo || !notifyBranches[e.Change.Branch] || e.PatchSet.Number != 1 { continue } - user := username(e.PatchSet) + user := username(e.PatchSet.Uploader) parsedMsg = nopingAll(user, fmt.Sprintf("CL/%d proposed by %s - %s - %s", e.Change.Number, user, e.Change.Subject, patchSetURL(e.Change, e.PatchSet))) case *gerritevents.ChangeMerged: if e.Change.Project != *notifyRepo || !notifyBranches[e.Change.Branch] { continue } - owner := username(e.PatchSet) + owner := username(e.Change.Owner) submitter := e.Submitter.Username url := patchSetURL(e.Change, e.PatchSet) |