opendev/gerrit
Code Issues Proposed changes

Merge "Fix name of method in WebLinks class that returns file web links"

Browse Source
This commit is contained in:
David Pursehouse
2014-09-25 01:00:04 +00:00
committed by Gerrit Code Review
parent 2f1a22acba ef6f5f15c2
commit 1d613e231d
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/WebLinks.java
View File

@@ -52,7 +52,7 @@ public class WebLinks {
return links;
}
public Iterable<WebLinkInfo> getPatchLinks(String project, String revision,
public Iterable<WebLinkInfo> getFileLinks(String project, String revision,
String file) {
List<WebLinkInfo> links = Lists.newArrayList();
for (FileWebLink webLink : fileLinks) {

2
gerrit-server/src/main/java/com/google/gerrit/server/change/GetDiff.java
View File

@@ -206,7 +206,7 @@ public class GetDiff implements RestReadView<FileResource> {
private List<WebLinkInfo> getFileWebLinks(Project project, String rev,
String file) {
List<WebLinkInfo> fileWebLinks = new ArrayList<>();
for (WebLinkInfo link : webLinks.get().getPatchLinks(project.getName(),
for (WebLinkInfo link : webLinks.get().getFileLinks(project.getName(),
rev, file)) {
if (!Strings.isNullOrEmpty(link.name) && !Strings.isNullOrEmpty(link.url)) {
fileWebLinks.add(link);