MigrateJaxRsRecipe adds required=false to all @RequestParam with unset required

Author: fabapp2

components/sbm-core/src/main/java/org/springframework/sbm/engine/recipe/OpenRewriteDeclarativeRecipeAdapter.java

@@ -18,12 +18,12 @@
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import lombok.*;
+import lombok.experimental.SuperBuilder;
 import org.openrewrite.Recipe;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.sbm.engine.context.ProjectContext;
 
-@Builder
-@NoArgsConstructor
+@SuperBuilder
 @AllArgsConstructor
 public class OpenRewriteDeclarativeRecipeAdapter extends AbstractAction {
     @Setter
@@ -32,11 +32,17 @@ public class OpenRewriteDeclarativeRecipeAdapter extends AbstractAction {
 
     @Autowired
     @JsonIgnore
+    @Setter
     private RewriteRecipeLoader rewriteRecipeLoader;
     @JsonIgnore
     @Autowired
+    @Setter
     private RewriteRecipeRunner rewriteRecipeRunner;
 
+    public OpenRewriteDeclarativeRecipeAdapter() {
+        super(builder());
+    }
+
     @Override
     public boolean isApplicable(ProjectContext context) {
         return super.isApplicable(context);

components/sbm-recipes-jee-to-boot/src/main/java/org/springframework/sbm/jee/jaxrs/MigrateJaxRsRecipe.java

@@ -23,7 +23,10 @@
 import org.springframework.sbm.build.api.Dependency;
 import org.springframework.sbm.build.migration.actions.AddDependencies;
 import org.springframework.sbm.build.migration.conditions.NoExactDependencyExist;
+import org.springframework.sbm.engine.recipe.OpenRewriteDeclarativeRecipeAdapter;
 import org.springframework.sbm.engine.recipe.Recipe;
+import org.springframework.sbm.engine.recipe.RewriteRecipeLoader;
+import org.springframework.sbm.engine.recipe.RewriteRecipeRunner;
 import org.springframework.sbm.java.JavaRecipeAction;
 import org.springframework.sbm.java.migration.actions.ReplaceTypeAction;
 import org.springframework.sbm.java.migration.conditions.HasAnnotation;
@@ -34,6 +37,7 @@
 import org.springframework.sbm.jee.jaxrs.recipes.SwapCacheControl;
 import org.springframework.sbm.jee.jaxrs.recipes.SwapHttHeaders;
 import org.springframework.sbm.jee.jaxrs.recipes.SwapResponseWithResponseEntity;
+import org.springframework.sbm.support.openrewrite.java.AddOrReplaceAnnotationAttribute;
 
 import java.util.List;
 import java.util.function.Supplier;
@@ -46,7 +50,7 @@ public class MigrateJaxRsRecipe {
     private final JavaParser javaParserSupplier;
 
     @Bean
-    public Recipe jaxRs() {
+    public Recipe jaxRs(RewriteRecipeLoader rewriteRecipeLoader, RewriteRecipeRunner rewriteRecipeRunner) {
         return Recipe.builder()
                 .name("migrate-jax-rs")
                 .order(60)
@@ -121,6 +125,26 @@ public Recipe jaxRs() {
                                         .condition(HasImportStartingWith.builder().value("javax.ws.rs.core.Response").build())
                                         .description("Replace JaxRs Response and ResponseBuilder with it's Spring equivalent.")
                                         .recipe(new SwapResponseWithResponseEntity(() -> javaParserSupplier))
+                                        .build(),
+
+                                OpenRewriteDeclarativeRecipeAdapter.builder()
+                                        .condition(HasAnnotation.builder().annotation("org.springframework.web.bind.annotation.RequestParam").build())
+                                        .description("Adds required=false to all @RequestParam annotations")
+                                        .rewriteRecipeLoader(rewriteRecipeLoader)
+                                        .rewriteRecipeRunner(rewriteRecipeRunner)
+                                        .openRewriteRecipe(
+                                                """
+                                                type: specs.openrewrite.org/v1beta/recipe
+                                                name: org.openrewrite.java.spring.boot3.data.UpgradeSpringData30
+                                                displayName: Upgrade to SpringBoot 3.0
+                                                description: 'Upgrade to SpringBoot to 3.0 from any prior version.'
+                                                recipeList:
+                                                  - org.openrewrite.java.AddOrUpdateAnnotationAttribute:
+                                                      annotationType: "org.springframework.web.bind.annotation.RequestParam"
+                                                      attributeName: "required"
+                                                      attributeValue: "false"
+                                                      addOnly: true
+                                                """)
                                         .build()
                         )
                 )

components/sbm-recipes-jee-to-boot/src/test/java/org/springframework/sbm/jee/jaxrs/recipes/BootifyJaxRsAnnotationsRecipeTest.java

@@ -33,7 +33,7 @@ public class BootifyJaxRsAnnotationsRecipeTest {
     @Test
     void test() {
 
-        Recipe jaxRsRecipe = new MigrateJaxRsRecipe(new RewriteJavaParser(new SbmApplicationProperties())).jaxRs();
+        Recipe jaxRsRecipe = new MigrateJaxRsRecipe(new RewriteJavaParser(new SbmApplicationProperties())).jaxRs(null, null);
         Optional<Recipe> recipe = Optional.of(jaxRsRecipe);
         RecipeTestSupport.assertThatRecipeExists(recipe);
         RecipeTestSupport.assertThatRecipeHasActions(recipe,

components/sbm-support-boot/src/main/java/org/springframework/sbm/boot/common/conditions/IsSpringBootProject.java

@@ -1,3 +1,18 @@
+/*
+ * 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.common.conditions;
 
 import org.springframework.sbm.engine.context.ProjectContext;

components/sbm-support-boot/src/test/java/org/springframework/sbm/boot/common/conditions/HasSpringBootDependencyImportTest.java

@@ -1,3 +1,18 @@
+/*
+ * 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.common.conditions;
 
 import org.junit.jupiter.api.Test;

components/sbm-support-boot/src/test/java/org/springframework/sbm/boot/common/conditions/HasSpringBootDependencyManuallyManagedTest.java

@@ -13,7 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.springframework.sbm.boot.common.conditions;
 
 import org.junit.jupiter.api.Test;

components/sbm-support-boot/src/test/java/org/springframework/sbm/boot/common/conditions/HasSpringBootProjectTest.java

@@ -1,3 +1,18 @@
+/*
+ * 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.common.conditions;
 
 import org.junit.jupiter.api.Test;

components/sbm-support-boot/src/test/java/org/springframework/sbm/boot/common/conditions/HasSpringBootStarterParentTest.java

@@ -1,3 +1,18 @@
+/*
+ * 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.common.conditions;
 
 import org.junit.jupiter.api.Test;