Ignore finished plan for upgrades

ElectronicObserverEN · Dec 7, 2024 · e48a485 · e48a485
1 parent 7322c0a
commit e48a485
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/ElectronicObserver/Window/Tools/EquipmentUpgradePlanner/EquipmentUpgradePlanManager.cs b/ElectronicObserver/Window/Tools/EquipmentUpgradePlanner/EquipmentUpgradePlanManager.cs
@@ -193,15 +193,16 @@ private void UpdatePlanAfterEquipmentImprovement(EquipmentUpgradePlanItemViewMod
 	/// <returns>Null if not found</returns>
 	private EquipmentUpgradePlanItemViewModel? FindEquipmentPlanFromEquipmentData(IEquipmentData equipmentData)
 	{
-		EquipmentUpgradePlanItemViewModel? foundPlan = PlannedUpgrades.FirstOrDefault(plan => plan.Equipment?.MasterID == equipmentData.MasterID);
+		List<EquipmentUpgradePlanItemViewModel> plans = PlannedUpgrades.Where(plan => !plan.Finished).ToList();
+
+		EquipmentUpgradePlanItemViewModel? foundPlan = plans.FirstOrDefault(plan => plan.Equipment?.MasterID == equipmentData.MasterID);
 
 		// Plan found, return it
 		if (foundPlan != null) return foundPlan;
 
 		// Not found => look for a matching plan
-		foundPlan = PlannedUpgrades
+		foundPlan = plans
 			.Where(plan => plan.Equipment?.MasterID == 0)
-			.Where(plan => !plan.Finished)
 			.Where(plan => plan.DesiredUpgradeLevel is UpgradeLevel.Conversion || plan.DesiredUpgradeLevel > equipmentData.UpgradeLevel)
 			.FirstOrDefault(plan => plan.Equipment?.EquipmentID == equipmentData.EquipmentID);