From 21b75586a38c8f2e92503394ee1dad059f4f26bd Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Thu, 17 Feb 2022 15:57:06 -0800 Subject: Fix the ${hash} substitution for diff file web links It seems that when we were refactoring this change to address review comments we accidentally swapped out the ${hash} replacement for a revision replacement. Fix this by swapping it back to a hash replacement which was the intended behavior. Note we skip release notes as this addresses a bug in a recent change that should have accurate release notes already. Release-Notes: skip Change-Id: I0b1c56d9aa80b0014e26f2a2385df90b1f5ae413 --- java/com/google/gerrit/server/config/GitwebConfig.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/java/com/google/gerrit/server/config/GitwebConfig.java b/java/com/google/gerrit/server/config/GitwebConfig.java index 40b5a8bbcc..0bc4380319 100644 --- a/java/com/google/gerrit/server/config/GitwebConfig.java +++ b/java/com/google/gerrit/server/config/GitwebConfig.java @@ -321,7 +321,7 @@ public class GitwebConfig { return link( file.replace("project", encode(projectName)) .replace("commit", encode(revision)) - .replace("hash", encode(revision)) + .replace("hash", encode(hash)) .replace("file", encode(fileName)) .toString()); } -- cgit v1.2.3