commit 32203afc081ad031a80b023c11acb9b0d70174d2
author Fabio Ponciroli <ponch78@gmail.com> Fri Jul 01 10:41:21 2022 +0200
committer Fabio Ponciroli <ponch78@gmail.com> Fri Jul 01 10:08:44 2022 +0000
tree 209894e9a2ff8c93746d3987ba6f85d289f5dd29
parent 53ab60fe31183d32543be60f6f3ec65977d8e4dc

Use original repository to collect stats

Make sure to always work on the original repository, even in case of
DelegateRepository.

Bug: Issue 15997
Change-Id: Ifd09670f3e3a9d529816a0241fc3e81c15387d77

src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/UpdateGitMetricsTask.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/UpdateGitMetricsTask.java b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/UpdateGitMetricsTask.java
index ed120e5..7dc726d 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/UpdateGitMetricsTask.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/UpdateGitMetricsTask.java
@@ -16,6 +16,7 @@
 
 import com.google.common.flogger.FluentLogger;
 import com.google.gerrit.entities.Project;
+import com.google.gerrit.server.git.DelegateRepository;
 import com.google.gerrit.server.git.GitRepositoryManager;
 import com.google.inject.Inject;
 import com.google.inject.assistedinject.Assisted;
@@ -57,7 +58,13 @@
           repository.getIdentifier(), projectName);
       // TODO Loop through all the collectors
       Project project = Project.builder(projectNameKey).build();
-      GitStats gitStats = new GitStats((FileRepository) repository, project);
+
+      Repository unwrappedRepo =
+          repository instanceof DelegateRepository
+              ? ((DelegateRepository) repository).delegate()
+              : repository;
+
+      GitStats gitStats = new GitStats((FileRepository) unwrappedRepo, project);
       Map<String, Long> newMetrics = gitStats.get();
       logger.atInfo().log(
           "Here all the metrics for %s - %s", project.getName(), getStringFromMap(newMetrics));