From 906fa4d8f84aedc4dbb58e14b60631867c13b9b9 Mon Sep 17 00:00:00 2001
From: Rutherther <fandabohacek@gmail.com>
Date: Sat, 31 Dec 2022 15:15:44 +0100
Subject: [PATCH] feat(client): do not use removed IPacketHandler

---
 .../NostaleLocalClient.cs                     | 30 ++++++++-----------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/src/Core/NosSmooth.LocalClient/NostaleLocalClient.cs b/src/Core/NosSmooth.LocalClient/NostaleLocalClient.cs
index 959ab81..2339ee7 100644
--- a/src/Core/NosSmooth.LocalClient/NostaleLocalClient.cs
+++ b/src/Core/NosSmooth.LocalClient/NostaleLocalClient.cs
@@ -36,7 +36,7 @@ public class NostaleLocalClient : BaseNostaleClient
     private readonly PetManagerBinding _petManagerBinding;
     private readonly ControlCommands _controlCommands;
     private readonly IPacketSerializer _packetSerializer;
-    private readonly IPacketHandler _packetHandler;
+    private readonly PacketHandler _packetHandler;
     private readonly UserActionDetector _userActionDetector;
     private readonly ILogger _logger;
     private readonly IServiceProvider _provider;
@@ -66,7 +66,7 @@ public class NostaleLocalClient : BaseNostaleClient
         ControlCommands controlCommands,
         CommandProcessor commandProcessor,
         IPacketSerializer packetSerializer,
-        IPacketHandler packetHandler,
+        PacketHandler packetHandler,
         UserActionDetector userActionDetector,
         ILogger<NostaleLocalClient> logger,
         IOptions<LocalClientOptions> options,
@@ -204,17 +204,8 @@ public class NostaleLocalClient : BaseNostaleClient
                 packet = packetResult.Entity;
             }
 
-            Result result;
-            if (type == PacketSource.Server)
-            {
-                result = await _packetHandler.HandleReceivedPacketAsync
-                    (this, packet, packetString, _stopRequested ?? default);
-            }
-            else
-            {
-                result = await _packetHandler.HandleSentPacketAsync
-                    (this, packet, packetString, _stopRequested ?? default);
-            }
+            Result result = await _packetHandler.HandlePacketAsync
+                (this, type, packet, packetString, _stopRequested ?? default);
 
             if (!result.IsSuccess)
             {
@@ -230,11 +221,14 @@ public class NostaleLocalClient : BaseNostaleClient
 
     private bool FollowEntity(MapBaseObj? obj)
     {
-        Task.Run
-        (
-            async () => await _controlCommands.CancelAsync
-                (ControlCommandsFilter.UserCancellable, false, (CancellationToken)_stopRequested!)
-        );
+        if (obj is not null)
+        {
+            Task.Run
+            (
+                async () => await _controlCommands.CancelAsync
+                    (ControlCommandsFilter.UserCancellable, false, (CancellationToken)_stopRequested!)
+            );
+        }
         return true;
     }
 
-- 
2.49.0