commit 91c8c1e35eaa31ba71ec8cfc7438efb0f7f48db3
author Shawn Pearce <sop@google.com> Mon Jun 06 12:41:56 2016 -0700
committer Shawn Pearce <sop@google.com> Mon Jun 06 12:41:56 2016 -0700
tree 7f884665cc15c6f12482dbd86f6e51069e06e806
parent 00fc28b622d615d26c0c94a267796fdf638d6b93

Don't format webLinks for parent commits

Post fd6b98143759ed38d9682b4127903def4e09f541 ("Don't show
PatchSetWebLink for parent commits") the web UI does not use patch
set web links for the parent commits. These are parent commits,
they are not necessarily a patch set under review, and should not
get links formatted by the PatchSetWebLinks.

Change-Id: Ib2babb8a4c9599fda1d6282cb0c6efdc349c0ae5

gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java
index df23e9a..c9a0d11 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java
@@ -1004,11 +1004,6 @@
       CommitInfo i = new CommitInfo();
       i.commit = parent.name();
       i.subject = parent.getShortMessage();
-      if (addLinks) {
-        FluentIterable<WebLinkInfo> parentLinks =
-            webLinks.getPatchSetLinks(project, parent.name());
-        i.webLinks = parentLinks.isEmpty() ? null : parentLinks.toList();
-      }
       info.parents.add(i);
     }
     return info;