From 2bceef85412be7465bce8b57da38aba93bb0e9c0 Mon Sep 17 00:00:00 2001 From: alx652 Date: Thu, 7 Mar 2024 19:04:30 -0500 Subject: [PATCH] fix comment 2 --- .../gsrs/autoconfigure/GsrsExportConfiguration.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/gsrs-spring-boot-autoconfigure/src/main/java/gsrs/autoconfigure/GsrsExportConfiguration.java b/gsrs-spring-boot-autoconfigure/src/main/java/gsrs/autoconfigure/GsrsExportConfiguration.java index cd4b4405..b4d27ff6 100644 --- a/gsrs-spring-boot-autoconfigure/src/main/java/gsrs/autoconfigure/GsrsExportConfiguration.java +++ b/gsrs-spring-boot-autoconfigure/src/main/java/gsrs/autoconfigure/GsrsExportConfiguration.java @@ -52,11 +52,6 @@ public class GsrsExportConfiguration { private Map sortOutput = new HashMap<>(); @JsonIgnore - // AW can we remove this if we remove "factories"? - // don't understand how get methods below include ExporterFactories in expList but not in exporters - // see getExporterFor, getAllSupportedFormats - - private Map> exporters = new HashMap<>(); @JsonIgnore @@ -98,8 +93,9 @@ public class GsrsExportConfiguration { // issue. May remove this in the future. AutowireHelper.getInstance().autowire(GsrsExportConfiguration.this); - - // we should not use "factories" config anymore? + // AW: We should not use "factories" config anymore because we can't + // include order, disabled properties with raw Class, right? + // If so delete this block and other related code if(!factoriesMapList.isEmpty()) { //legacy stuff