From 947c9e9f238f7d3db4c8a4d52523f08e4ae748e6 Mon Sep 17 00:00:00 2001
From: JustArchi <JustArchi@JustArchi.net>
Date: Wed, 3 Apr 2019 18:57:25 +0200
Subject: [PATCH] Bot-related improvements

---
 ArchiSteamFarm/Actions.cs |  3 ---
 ArchiSteamFarm/Bot.cs     | 18 +++---------------
 2 files changed, 3 insertions(+), 18 deletions(-)

diff --git a/ArchiSteamFarm/Actions.cs b/ArchiSteamFarm/Actions.cs
index 40ebd22b4..928079211 100644
--- a/ArchiSteamFarm/Actions.cs
+++ b/ArchiSteamFarm/Actions.cs
@@ -41,8 +41,6 @@ namespace ArchiSteamFarm {
 		private readonly ConcurrentHashSet<ulong> HandledGifts = new ConcurrentHashSet<ulong>();
 		private readonly SemaphoreSlim TradingSemaphore = new SemaphoreSlim(1, 1);
 
-		internal bool SkipFirstShutdown { get; set; }
-
 		private Timer CardsFarmerResumeTimer;
 		private bool ProcessingGiftsScheduled;
 		private bool TradingScheduled;
@@ -294,7 +292,6 @@ namespace ArchiSteamFarm {
 				return (false, Strings.BotAlreadyRunning);
 			}
 
-			SkipFirstShutdown = true;
 			Utilities.InBackground(Bot.Start);
 
 			return (true, Strings.Done);
diff --git a/ArchiSteamFarm/Bot.cs b/ArchiSteamFarm/Bot.cs
index b16ad525c..8d841517c 100755
--- a/ArchiSteamFarm/Bot.cs
+++ b/ArchiSteamFarm/Bot.cs
@@ -982,26 +982,14 @@ namespace ArchiSteamFarm {
 		internal async Task OnFarmingFinished(bool farmedSomething) {
 			await OnFarmingStopped().ConfigureAwait(false);
 
-			if (farmedSomething || !FirstTradeSent) {
+			if (BotConfig.SendOnFarmingFinished && (farmedSomething || !FirstTradeSent)) {
 				FirstTradeSent = true;
 
-				if (BotConfig.SendOnFarmingFinished) {
-					await Actions.SendTradeOffer(wantedTypes: BotConfig.LootableTypes).ConfigureAwait(false);
-				}
+				await Actions.SendTradeOffer(wantedTypes: BotConfig.LootableTypes).ConfigureAwait(false);
 			}
 
 			if (BotConfig.ShutdownOnFarmingFinished) {
-				if (farmedSomething || (ASF.GlobalConfig.IdleFarmingPeriod == 0)) {
-					Stop();
-
-					return;
-				}
-
-				if (Actions.SkipFirstShutdown) {
-					Actions.SkipFirstShutdown = false;
-				} else {
-					Stop();
-				}
+				Stop();
 			}
 		}