opendev/gerrit
Code Issues Proposed changes

Don't show /COMMIT_MSG in change emails

Browse Source 
The magical /COMMIT_MSG file that we wedge into the PatchList
in order to support reviewing the message itself shouldn't be
displayed as if it were a file added by the author.

Change-Id: I211a3a41ec7e2a2eff709beb3230cc162f7c095c
Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2010-08-21 13:16:17 -07:00
parent 39f4b62298
commit 9527a3c607
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -19,6 +19,7 @@ import com.google.gerrit.reviewdb.AccountGroup;
import com.google.gerrit.reviewdb.AccountProjectWatch;
import com.google.gerrit.reviewdb.Change;
import com.google.gerrit.reviewdb.ChangeMessage;
import com.google.gerrit.reviewdb.Patch;
import com.google.gerrit.reviewdb.PatchSet;
import com.google.gerrit.reviewdb.PatchSetApproval;
import com.google.gerrit.reviewdb.PatchSetInfo;
@@ -291,6 +292,9 @@ public abstract class ChangeEmail extends OutgoingEmail {
if (patchSet != null) {
appendText("---\n");
for (PatchListEntry p : getPatchList().getPatches()) {
if (Patch.COMMIT_MSG.equals(p.getNewName())) {
continue;
}
appendText(p.getChangeType().getCode() + " " + p.getNewName() + "\n");
}
appendText("\n");