commit 49469de1938091312137814bfb0e1e73c19325ba
author Martin Fick <mfick@codeaurora.org> Fri Sep 10 12:03:22 2021 -0600
committer Martin Fick <mfick@codeaurora.org> Mon Sep 20 01:21:57 2021 +0000
tree 96de041e838b25b4a28021f202bef8ded0239a44
parent d4ff08a50c8226d35fa7403b03c582550e1138ec

fixup! Add support for tasks-factory and names-factory keywords

No longer shadow name in TaskConfig.Task (it should have been removed
from TaskBase).

Change-Id: Iadc9f544cc99286a96b0fb2d63031dc9418a2b23

src/main/java/com/googlesource/gerrit/plugins/task/TaskConfig.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/task/TaskConfig.java b/src/main/java/com/googlesource/gerrit/plugins/task/TaskConfig.java
index c5ecc6a..8cd8022 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/task/TaskConfig.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/task/TaskConfig.java
@@ -58,7 +58,6 @@
     public String fail;
     public String failHint;
     public String inProgress;
-    public String name;
     public String pass;
     public String preloadTask;
     public Map<String, String> properties;
@@ -79,7 +78,6 @@
       fail = getString(s, KEY_FAIL, null);
       failHint = getString(s, KEY_FAIL_HINT, null);
       inProgress = getString(s, KEY_IN_PROGRESS, null);
-      name = s.subSection;
       pass = getString(s, KEY_PASS, null);
       preloadTask = getString(s, KEY_PRELOAD_TASK, null);
       properties = getProperties(s, KEY_PROPERTIES_PREFIX);
@@ -133,7 +131,7 @@
 
     public Task(SubSection s, boolean isVisible, boolean isTrusted) {
       super(s, isVisible, isTrusted);
-      name = getString(s, KEY_NAME, s.subSection);
+      name = s.subSection;
     }
 
     protected Task(TaskBase base) {