fix: Use a stable recipe order for NullableRecipe.getRecipeList()

Avoids shuffling the order in the docs.
TimefoldAI · Dec 4, 2024 · 07cc41d · 07cc41d
1 parent 9d25467
commit 07cc41d
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/migration/src/main/java/ai/timefold/solver/migration/v8/NullableRecipe.java b/migration/src/main/java/ai/timefold/solver/migration/v8/NullableRecipe.java
@@ -25,7 +25,7 @@ public String getDescription() {
     @Override
     public List<Recipe> getRecipeList() {
         var packageName = "ai.timefold.solver.core.api.score.stream";
-        var methodsToRename = Map.ofEntries(
+        var methodsToRename = List.of(
                 Map.entry("ConstraintFactory forEachIncludingNullVars(Class)",
                         "forEachIncludingUnassigned"),
                 Map.entry("uni.UniConstraintStream ifExistsIncludingNullVars(..)",
@@ -50,9 +50,9 @@ public List<Recipe> getRecipeList() {
                         "ifNotExistsIncludingUnassigned"));
 
         var result = new ArrayList<Recipe>();
-        methodsToRename.forEach((oldPattern, newMethodName) -> {
-            var pattern = packageName + "." + oldPattern;
-            result.add(new ChangeMethodName(pattern, newMethodName, null, null));
+        methodsToRename.forEach(entry -> {
+            var pattern = packageName + "." + entry.getKey();
+            result.add(new ChangeMethodName(pattern, entry.getValue(), null, null));
         });
         result.add(new ChangeAnnotationAttributeName("ai.timefold.solver.core.api.domain.variable.PlanningVariable", "nullable",
                 "allowsUnassigned"));