From 3dc88c65aaad0a79608012a1d22a1e7a004f1b4e Mon Sep 17 00:00:00 2001 From: JustArchi Date: Mon, 25 Apr 2016 14:34:21 +0200 Subject: [PATCH] Revert recent trades When steam is going crazy we're also forgetting trades that simply failed to accept Also, trade that is not valid currently, might be valid in future, especially STM --- ArchiSteamFarm/Trading.cs | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/ArchiSteamFarm/Trading.cs b/ArchiSteamFarm/Trading.cs index 75792e39a..4f48d869f 100644 --- a/ArchiSteamFarm/Trading.cs +++ b/ArchiSteamFarm/Trading.cs @@ -37,7 +37,6 @@ namespace ArchiSteamFarm { private readonly Bot Bot; private readonly SemaphoreSlim TradesSemaphore = new SemaphoreSlim(1); - private readonly HashSet RecentlyParsedTrades = new HashSet(); private byte ParsingTasks; @@ -80,36 +79,12 @@ namespace ArchiSteamFarm { TradesSemaphore.Release(); } - private async Task ForgetRecentTrade(ulong tradeID) { - await Utilities.SleepAsync(24 * 60 * 60 * 1000).ConfigureAwait(false); - lock (RecentlyParsedTrades) { - RecentlyParsedTrades.Remove(tradeID); - RecentlyParsedTrades.TrimExcess(); - } - } - private async Task ParseActiveTrades() { HashSet tradeOffers = Bot.ArchiWebHandler.GetTradeOffers(); if (tradeOffers == null || tradeOffers.Count == 0) { return; } - lock (RecentlyParsedTrades) { - tradeOffers.RemoveWhere(trade => RecentlyParsedTrades.Contains(trade.TradeOfferID)); - } - - if (tradeOffers.Count == 0) { - return; - } - - foreach (Steam.TradeOffer tradeOffer in tradeOffers) { - lock (RecentlyParsedTrades) { - RecentlyParsedTrades.Add(tradeOffer.TradeOfferID); - } - - ForgetRecentTrade(tradeOffer.TradeOfferID).Forget(); - } - await tradeOffers.ForEachAsync(ParseTrade).ConfigureAwait(false); await Bot.AcceptConfirmations(true, Confirmation.ConfirmationType.Trade).ConfigureAwait(false); }