opendev/gerrit
Code Issues Proposed changes

Merge "Include change number in change email footers"

Browse Source
This commit is contained in:
David Pursehouse 2016-11-09 01:22:00 +00:00 committed by Gerrit Code Review
commit ae0c067545
3 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-server/src/main/java/com/google/gerrit/server/mail/send/ChangeEmail.java
View File

@ -446,6 +446,7 @@ public abstract class ChangeEmail extends NotificationEmail {
changeData.put("subject", change.getSubject());
changeData.put("originalSubject", change.getOriginalSubject());
changeData.put("ownerEmail", getNameEmailFor(change.getOwner()));
changeData.put("changeNumber", Integer.toString(change.getChangeId()));
soyContext.put("change", changeData);
String subject = change.getSubject();

1
gerrit-server/src/main/resources/com/google/gerrit/server/mail/ChangeFooter.soy
View File

@ -47,6 +47,7 @@
Gerrit-MessageType: {$messageClass}{\n}
Gerrit-Change-Id: {$changeId}{\n}
Gerrit-Change-Number: {$change.changeNumber}{\n}
Gerrit-PatchSet: {$patchSet.patchSetId}{\n}
Gerrit-Project: {$projectName}{\n}
Gerrit-Branch: {$branch.shortName}{\n}

1
gerrit-server/src/main/resources/com/google/gerrit/server/mail/ChangeFooterHtml.soy
View File

@ -47,6 +47,7 @@
<p style="{$footerStyle}">
Gerrit-MessageType: {$messageClass}<br/>
Gerrit-Change-Id: {$changeId}<br/>
Gerrit-Change-Number: {$change.changeNumber}<br/>
Gerrit-PatchSet: {$patchSet.patchSetId}<br/>
Gerrit-Project: {$projectName}<br/>
Gerrit-Branch: {$branch.shortName}<br/>