opendev/gerrit
Code Issues Proposed changes

ChangeScreen2: Do not display 'M' next to modified files

Browse Source 
Most files are modified. Its the added/deleted/renamed/copied files
that need to be highlighted as outside of the typical review flow.

Avoid UI clutter by only displaying 'ADRCW' status codes, or any
other codes the server returns in the REST API.

Change-Id: I332152c3d6f8c6416649e3f7efe6a3a141f28536
This commit is contained in:
Shawn Pearce
2013-10-07 11:00:36 -07:00
parent 9c7ac31c72
commit 31b0ee2c36
3 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-server/src/main/java/com/google/gerrit/server/change/FileInfoJson.java
View File

@@ -56,7 +56,8 @@ public class FileInfoJson {
Map<String, FileInfo> files = Maps.newTreeMap();
for (PatchListEntry e : list.getPatches()) {
FileInfoJson.FileInfo d = new FileInfoJson.FileInfo();
d.status = e.getChangeType().getCode();
d.status = e.getChangeType() != Patch.ChangeType.MODIFIED
? e.getChangeType().getCode() : null;
d.oldPath = e.getOldName();
if (e.getPatchType() == Patch.PatchType.BINARY) {
d.binary = true;