Moved Helper to separate class

Author: fabapp2

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/SpringBootUpgradeReportSection.java

@@ -47,37 +47,16 @@
 @Setter
 public class SpringBootUpgradeReportSection {
 
-    private static final String ls = System.lineSeparator();
-    private List<String> projects;
-
-    /**
-     * Helper acting as {@link Condition} and data provide for a {@link SpringBootUpgradeReportSection}.
-     * @deprecated Use {@link AbstractHelper} instead
-     */
-    @Deprecated(forRemoval = true)
-    public interface Helper<T> extends Condition {
-        /**
-         * @return {@code Map<String, T>} the model data for the template.
-         */
-        Map<String, T> getData();
-    }
-
-    public static abstract class AbstractHelper<T> implements Helper<T> {
-
-        @Override
-        public String getDescription() {
-            return "";
-        }
-    }
-
     public static final String CHANGE_HEADER = "What Changed";
     public static final String AFFECTED = "Why is the application affected";
     public static final String REMEDIATION = "Remediation";
-
-    public boolean shouldRender(ProjectContext context) {
-        return helper.evaluate(context);
-    }
-
+    private static final String ls = System.lineSeparator();
+    /**
+     * The spring project(s)/modules this change comes from.
+     *
+     * e.g. {@code spring-boot} and {@code actuator}
+     */
+    private List<String> projects;
     /**
      * Section title
      */
@@ -109,15 +88,22 @@ public boolean shouldRender(ProjectContext context) {
     @NotNull
     private Set<String> contributors;
 
-    @JsonIgnore
-    private Helper<Object> helper;
+    @NotNull
+    private SpringBootUpgradeReportSectionHelper<?> helper;
+
     @JsonIgnore
     @Autowired
     private SpringBootUpgradeReportFreemarkerSupport freemarkerSupport;
 
+    public boolean shouldRender(ProjectContext context) {
+        return helper.evaluate(context);
+    }
+
+
+
     public String render(ProjectContext context) {
         if (getHelper().evaluate(context)) {
-            Map<String, Object> params = getHelper().getData();
+            Map<String, ?> params = getHelper().getData();
 
             try (StringWriter writer = new StringWriter()) {
                 String templateContent = buildTemplate();
@@ -138,7 +124,7 @@ public String render(ProjectContext context) {
         throw new IllegalArgumentException("Could not render Section '"+ getTitle()+"', evaluating the context returned false");
     }
 
-    private void renderTemplate(Map<String, Object> params, StringWriter writer, String templateContent) throws IOException, TemplateException {
+    private void renderTemplate(Map<String, ?> params, StringWriter writer, String templateContent) throws IOException, TemplateException {
         String templateName = getTitle().replace(" ", "") + UUID.randomUUID();
         freemarkerSupport.getStringLoader().putTemplate(templateName, templateContent);
         Template t = freemarkerSupport.getConfiguration().getTemplate(templateName);
@@ -208,28 +194,20 @@ private void renderLineBreak(StringBuilder sb) {
         sb.append(ls);
     }
 
-    private void renderGitHubInfo(StringBuilder sb) {
-        sb.append("****").append(ls);
-        sb.append("[%hardbreaks]").append(ls);
+    void renderGitHubInfo(StringBuilder sb) {
         if(gitHubIssue != null) {
-            sb.append("**Issue:** https://github.com/spring-projects-experimental/spring-boot-migrator/issues/").append(gitHubIssue).append("[#").append(gitHubIssue).append("^, role=\"ext-link\"] ").append(ls);
+            sb.append("**Issue:** https://github.com/spring-projects-experimental/spring-boot-migrator/issues/").append(gitHubIssue).append("[#").append(gitHubIssue).append("^, role=\"ext-link\"] ").append(" + ").append(ls);
         }
-//        if(contributors != null && gitHubIssue != null) {
-//            sb.append(", ");
-//        } else {
-//            sb.append(ls);
-//        }
         if(contributors != null) {
             List<Author> authors = getAuthors();
             sb.append("**Contributors:** ");
             String authorsString = authors.stream().map(a -> "https://github.com/" + a.getHandle() + "[@" + a.getHandle() + "^, role=\"ext-link\"]").collect(Collectors.joining(", "));
-            sb.append(authorsString).append(ls);
+            sb.append(authorsString).append(" + ").append(ls);
         }
         if(projects != null){
             String projectsList = projects.stream().collect(Collectors.joining(", "));
             sb.append("**Projects:** ").append(projectsList).append(ls);
         }
-        sb.append("****").append(ls);
     }
 
     private void renderSectionTitle(StringBuilder sb) {

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/SpringBootUpgradeReportSectionHelper.java

@@ -0,0 +1,37 @@
+/*
+ * Copyright 2021 - 2022 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.springframework.sbm.boot.upgrade_27_30.report;
+
+import org.springframework.sbm.engine.recipe.Condition;
+
+import java.util.Map;
+
+/**
+ * Helper base class which is {@link Condition} and data provider for a {@link SpringBootUpgradeReportSection}.
+ *
+ * @author Fabian Krüger
+ */
+public abstract class SpringBootUpgradeReportSectionHelper<T> implements Condition {
+    /**
+     * @return {@code Map<String, T>} the model data for the template.
+     */
+    public abstract Map<String, T> getData();
+
+    @Override
+    public String getDescription() {
+        return "";
+    }
+}

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/ActuatorEndpointsSanitizationHelper.java

@@ -16,6 +16,7 @@
 package org.springframework.sbm.boot.upgrade_27_30.report.helper;
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.build.api.BuildFile;
 import org.springframework.sbm.build.api.Module;
 import org.springframework.sbm.engine.context.ProjectContext;
@@ -29,7 +30,7 @@
 /**
  * @author Fabian Krüger
  */
-public class ActuatorEndpointsSanitizationHelper extends SpringBootUpgradeReportSection.AbstractHelper<List<BuildFile>> {
+public class ActuatorEndpointsSanitizationHelper extends SpringBootUpgradeReportSectionHelper<List<BuildFile>> {
 
     private static final String ACTUATOR_GROUP_ID = "org.springframework.boot";
     private static final String ACTUATOR_ARTIFACT_ID = "spring-boot-actuator";

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/AddSpringBootRepositoriesHelper.java

@@ -17,18 +17,14 @@
 package org.springframework.sbm.boot.upgrade_27_30.report.helper;
 
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.build.migration.conditions.NoPluginRepositoryExistsCondition;
 import org.springframework.sbm.build.migration.conditions.NoRepositoryExistsCondition;
 import org.springframework.sbm.engine.context.ProjectContext;
 
 import java.util.Map;
 
-public class AddSpringBootRepositoriesHelper implements SpringBootUpgradeReportSection.Helper<String>{
-    @Override
-    public String getDescription() {
-        return null;
-    }
-
+public class AddSpringBootRepositoriesHelper extends SpringBootUpgradeReportSectionHelper<String> {
     @Override
     public boolean evaluate(ProjectContext context) {
         return new NoRepositoryExistsCondition().evaluate(context) && new NoPluginRepositoryExistsCondition().evaluate(context);

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/BannerSupportHelper.java

@@ -18,6 +18,7 @@
 
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.project.resource.RewriteSourceFileHolder;
 
@@ -26,7 +27,7 @@
 import java.util.Map;
 import java.util.stream.Collectors;
 
-public class BannerSupportHelper implements SpringBootUpgradeReportSection.Helper<List<String>> {
+public class BannerSupportHelper extends SpringBootUpgradeReportSectionHelper<List<String>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
 

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/ChangesToDataPropertiesHelper.java

@@ -21,6 +21,7 @@
 import org.springframework.sbm.boot.properties.api.SpringBootApplicationProperties;
 import org.springframework.sbm.boot.properties.search.SpringBootApplicationPropertiesResourceListFilter;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.build.migration.conditions.NoDependencyExistMatchingRegex;
 import org.springframework.sbm.engine.context.ProjectContext;
 
@@ -34,7 +35,7 @@
 /**
  * @author Fabian Krüger
  */
-public class ChangesToDataPropertiesHelper implements SpringBootUpgradeReportSection.Helper<List<ChangesToDataPropertiesHelper.Match>> {
+public class ChangesToDataPropertiesHelper extends SpringBootUpgradeReportSectionHelper<List<ChangesToDataPropertiesHelper.Match>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
 

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/CommonsMultipartResolverHelper.java

@@ -20,14 +20,15 @@
 import org.springframework.sbm.boot.common.finder.MatchingMethod;
 import org.springframework.sbm.boot.common.finder.SpringBeanMethodDeclarationFinder;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.stream.Collectors;
 
-public class CommonsMultipartResolverHelper implements SpringBootUpgradeReportSection.Helper<List<String>>{
+public class CommonsMultipartResolverHelper extends SpringBootUpgradeReportSectionHelper<List<String>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     private static final String COMMONS_MULTIPART_RESOLVER_CLASS = "org.springframework.web.multipart.commons.CommonsMultipartResolver";

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/ConstructorBindingHelper.java

@@ -24,6 +24,7 @@
 import org.openrewrite.java.tree.J;
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.engine.recipe.OpenRewriteSourceFilesFinder;
 import org.springframework.sbm.project.resource.RewriteSourceFileHolder;
@@ -34,7 +35,7 @@
 import java.util.stream.Collectors;
 
 
-public class ConstructorBindingHelper implements SpringBootUpgradeReportSection.Helper<List<String>> {
+public class ConstructorBindingHelper extends SpringBootUpgradeReportSectionHelper<List<String>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     private List<String> constructorBindingFiles;

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/LoggingDateFormatHelper.java

@@ -18,6 +18,7 @@
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.filter.LoggingDateFormatPropertyFinder;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.properties.api.PropertiesSource;
 
@@ -27,7 +28,7 @@
 /**
  * @author Fabian Krüger
  */
-public class LoggingDateFormatHelper implements SpringBootUpgradeReportSection.Helper<List<? extends PropertiesSource>> {
+public class LoggingDateFormatHelper extends SpringBootUpgradeReportSectionHelper<List<? extends PropertiesSource>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     private List<? extends PropertiesSource> propertiesSources;

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/PagingAndSortingHelper.java

@@ -22,6 +22,7 @@
 import org.openrewrite.java.tree.J;
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.project.resource.RewriteSourceFileHolder;
 import org.springframework.sbm.support.openrewrite.GenericOpenRewriteRecipe;
@@ -31,7 +32,7 @@
 import java.util.Map;
 import java.util.stream.Collectors;
 
-public class PagingAndSortingHelper implements SpringBootUpgradeReportSection.Helper<List<String>> {
+public class PagingAndSortingHelper extends SpringBootUpgradeReportSectionHelper<List<String>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     private List<String> pagingAndSortingRepo;

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/SpringMVCAndWebFluxUrlMatchingChangesHelper.java

@@ -20,6 +20,7 @@
 import org.openrewrite.java.tree.J;
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.java.api.JavaSource;
 import org.springframework.sbm.java.impl.OpenRewriteJavaSource;
@@ -32,7 +33,7 @@
 /**
  * @author Fabian Krüger
  */
-public class SpringMVCAndWebFluxUrlMatchingChangesHelper implements SpringBootUpgradeReportSection.Helper<List<JavaSource>> {
+public class SpringMVCAndWebFluxUrlMatchingChangesHelper extends SpringBootUpgradeReportSectionHelper<List<JavaSource>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     private static final String SPRING_REST_CONTROLLER_FQN = "org.springframework.web.bind.annotation.RestController";

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/UpgradeDependenciesHelper.java

@@ -18,6 +18,7 @@
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportAction;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.java.api.JavaSource;
 
@@ -27,7 +28,7 @@
 /**
  * @author Fabian Krüger
  */
-public class UpgradeDependenciesHelper implements SpringBootUpgradeReportSection.Helper<List<String>> {
+public class UpgradeDependenciesHelper extends SpringBootUpgradeReportSectionHelper<List<String>> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     @Override

components/sbm-recipes-boot-upgrade/src/main/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/UpgradeSpringBootVersionHelper.java

@@ -17,6 +17,7 @@
 
 import org.springframework.sbm.boot.common.conditions.IsSpringBootProject;
 import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
+import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSectionHelper;
 import org.springframework.sbm.engine.context.ProjectContext;
 import org.springframework.sbm.java.api.JavaSource;
 
@@ -26,7 +27,7 @@
 /**
  * @author Fabian Krüger
  */
-public class UpgradeSpringBootVersionHelper implements SpringBootUpgradeReportSection.Helper<String> {
+public class UpgradeSpringBootVersionHelper extends SpringBootUpgradeReportSectionHelper<String> {
 
     public static final String VERSION_PATTERN = "(2\\.7\\..*)|(3\\.0\\..*)";
     @Override