Misc

JustArchiNET · Sep 19, 2024 · 440e439 · 440e439
1 parent 1dff9a4
commit 440e439
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/ArchiSteamFarm/Steam/Exchange/Trading.cs b/ArchiSteamFarm/Steam/Exchange/Trading.cs
@@ -550,6 +550,9 @@ private async Task<ParseTradeResult> ParseTrade(TradeOffer tradeOffer, ISet<(uin
 
 		bool accept = IsTradeNeutralOrBetter(inventory, tradeOffer.ItemsToGive, tradeOffer.ItemsToReceive);
 
+		// We're now sure whether the trade is neutral+ for us or not
+		ParseTradeResult.EResult acceptResult;
+
 		if (accept) {
 			// Ensure that accepting this trade offer does not create conflicts with other
 			lock (handledSets) {
@@ -561,10 +564,11 @@ private async Task<ParseTradeResult> ParseTrade(TradeOffer tradeOffer, ISet<(uin
 
 				handledSets.UnionWith(wantedSets);
 			}
-		}
 
-		// We're now sure whether the trade is neutral+ for us or not
-		ParseTradeResult.EResult acceptResult = accept ? ParseTradeResult.EResult.Accepted : ParseTradeResult.EResult.Rejected;
+			acceptResult = ParseTradeResult.EResult.Accepted;
+		} else {
+			acceptResult = ParseTradeResult.EResult.Rejected;
+		}
 
 		Bot.ArchiLogger.LogGenericDebug(Strings.FormatBotTradeOfferResult(tradeOffer.TradeOfferID, acceptResult, nameof(IsTradeNeutralOrBetter)));