Cut SendOnFarmingFinished spam

Allow only one trade to be sent if we didn't farm anything

Author: JustArchi

ArchiSteamFarm/Bot.cs

@@ -65,7 +65,7 @@ internal sealed class Bot {
 
 		internal bool KeepRunning { get; private set; }
 
-		private bool InvalidPassword, LoggedInElsewhere;
+		private bool InvalidPassword, LoggedInElsewhere, FirstTradeSent;
 		private string AuthCode, TwoFactorCode;
 
 		internal static async Task RefreshCMs(uint cellID) {
@@ -319,8 +319,9 @@ internal async Task<bool> RefreshSession() {
 			return true;
 		}
 
-		internal async Task OnFarmingFinished() {
-			if (BotConfig.SendOnFarmingFinished) {
+		internal async Task OnFarmingFinished(bool farmedSomething) {
+			if ((farmedSomething || !FirstTradeSent) && BotConfig.SendOnFarmingFinished) {
+				FirstTradeSent = true;
 				await ResponseSendTrade(BotConfig.SteamMasterID).ConfigureAwait(false);
 			}
 
@@ -1422,6 +1423,8 @@ private async void OnDisconnected(SteamClient.DisconnectedCallback callback) {
 			}
 
 			Logging.LogGenericInfo("Disconnected from Steam!", BotName);
+
+			FirstTradeSent = false;
 			CardsFarmer.StopFarming().Forget();
 
 			// If we initiated disconnect, do not attempt to reconnect

ArchiSteamFarm/CardsFarmer.cs

@@ -94,7 +94,7 @@ internal async Task StartFarming() {
 			if (!await IsAnythingToFarm().ConfigureAwait(false)) {
 				FarmingSemaphore.Release(); // We have nothing to do, don't forget to release semaphore
 				Logging.LogGenericInfo("We don't have anything to farm on this account!", Bot.BotName);
-				await Bot.OnFarmingFinished().ConfigureAwait(false);
+				await Bot.OnFarmingFinished(false).ConfigureAwait(false);
 				return;
 			}
 
@@ -145,7 +145,7 @@ internal async Task StartFarming() {
 			NowFarming = false;
 
 			Logging.LogGenericInfo("Farming finished!", Bot.BotName);
-			await Bot.OnFarmingFinished().ConfigureAwait(false);
+			await Bot.OnFarmingFinished(true).ConfigureAwait(false);
 		}
 
 		internal async Task StopFarming() {