Skip to content

Commit

Permalink
fixed NPE from new global config
Browse files Browse the repository at this point in the history
Note, that I'm not sure if it is fix or workaround.
Not, that it misbehaved for me completly - the items added in config
could not be removed in  2.434(.1) STS jenkins
renamed settings to jtreg as it should be
  • Loading branch information
judovana committed Dec 4, 2023
1 parent 81426dc commit 0293cb5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
import org.kohsuke.stapler.DataBoundSetter;
import org.kohsuke.stapler.StaplerRequest;

import java.util.ArrayList;
import java.util.List;
import java.util.logging.Logger;

Expand All @@ -18,8 +19,8 @@ public class JenkinsReportJckGlobalConfig extends GlobalConfiguration {
private static Logger logger = Logger.getLogger(JenkinsReportJckGlobalConfig.class.getName());

String diffToolUrl;
List<ComparatorLinksGroup> comparatorLinksGroups;
List<ConfigItem> configItems;
List<ComparatorLinksGroup> comparatorLinksGroups = new ArrayList<>();
List<ConfigItem> configItems = new ArrayList<>();

public static JenkinsReportJckGlobalConfig getInstance() {
return GlobalConfiguration.all().get(JenkinsReportJckGlobalConfig.class);
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?jelly escape-by-default='true'?>
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
<f:section title="Jenkins Report Jck Plugin">
<f:section title="Jenkins Report Jtreg Plugin">
<f:entry title="Diff-tool url (eg http://server.my:9090)" field="diffToolUrl">
<f:textbox/>
</f:entry>
Expand Down

0 comments on commit 0293cb5

Please sign in to comment.