Fixed ConcurrentModificationException in scenario-cutout util (#3624)

* Fixed ConcurrentModificationException when loading populations with unselected plans * Changed to more stable solution (instead of directly manipulating the list-object, we now use the removePlan-method) --------- Co-authored-by: rakow <[email protected]>
matsim-org · Dec 11, 2024 · 77bf4a4 · 77bf4a4
1 parent 5f58a97
commit 77bf4a4
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/...plication/src/main/java/org/matsim/application/prepare/scenario/CreateScenarioCutOut.java b/...plication/src/main/java/org/matsim/application/prepare/scenario/CreateScenarioCutOut.java
@@ -590,9 +590,12 @@ public void run(Person person) {
 		}
 
 		// Remove all unselected plans because these are not handled
-		person.getPlans().stream()
-			.filter(p -> p != person.getSelectedPlan())
-			.forEach(person::removePlan);
+		List<Plan> plans = new ArrayList<>(person.getPlans());
+		for(Plan p : plans){
+			if (p != person.getSelectedPlan()){
+				person.removePlan(p);
+			}
+		}
 	}