commit 7d91ddbceb1782a1ff4639ea75b721f08af6592b
author Nasser Grainawi <nasser@codeaurora.org> Fri May 29 15:50:36 2020 -0600
committer Nasser Grainawi <nasser@codeaurora.org> Fri May 29 15:53:32 2020 -0600
tree d02e3e44d124dffa14a94593302fd05320f0d4e5
parent e131af8325d1d3ae27af684013073af737c4f9b4

Fix getRefsByPrefix usage

Ie1b9ebd194a4ca2542d9aaf5c59c71dfbb361676 removed usage of the
deprecated getRefs(PREFIX) API, but missed that the returned key was
being used and the key had the prefix removed. Get ls-batches working
correctly again by stripping the prefix from the returned Ref.

Change-Id: I86784fdeb3a39e6d4ad44a75dcb57dd14c91db18

src/main/java/com/googlesource/gerrit/plugins/batch/BatchStore.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/batch/BatchStore.java b/src/main/java/com/googlesource/gerrit/plugins/batch/BatchStore.java
index 9c0db7b..d582af4 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/batch/BatchStore.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/batch/BatchStore.java
@@ -62,8 +62,9 @@
     List<Batch> batches = new ArrayList<>();
     try (Repository repo = repoManager.openRepository(project)) {
       for (Ref ref : repo.getRefDatabase().getRefsByPrefix(BATCHES_REF)) {
+        String id = ref.getName().substring(BATCHES_REF.length());
         try {
-          batches.add((includeBatchInfo == true) ? read(ref.getName()) : new Batch(ref.getName()));
+          batches.add((includeBatchInfo == true) ? read(id) : new Batch(id));
         } catch (NoSuchBatchException e) {
           continue;
         }