From 23158ccd61eebf1d2c8256e0a477c329bcb41c2d Mon Sep 17 00:00:00 2001
From: Ethan Moffat <ethan@moffat.io>
Date: Fri, 16 Sep 2022 10:33:32 -0700
Subject: [PATCH] Ensure trade agree is setting agreement for the correct
 player

---
 EOLib/PacketHandlers/Trade/TradeAgreeHandler.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/EOLib/PacketHandlers/Trade/TradeAgreeHandler.cs b/EOLib/PacketHandlers/Trade/TradeAgreeHandler.cs
index 3ce81b33d..aebf1711d 100644
--- a/EOLib/PacketHandlers/Trade/TradeAgreeHandler.cs
+++ b/EOLib/PacketHandlers/Trade/TradeAgreeHandler.cs
@@ -28,10 +28,10 @@ public class TradeAgreeHandler : InGameOnlyPacketHandler
 
         public override bool HandlePacket(IPacket packet)
         {
-            var otherPlayerId = packet.ReadShort();
+            var mainPlayerId = packet.ReadShort();
             var otherPlayerAgrees = packet.ReadChar() != 0;
 
-            _tradeRepository.SomeWhen(x => x.PlayerOneOffer.PlayerID == otherPlayerId)
+            _tradeRepository.SomeWhen(x => x.PlayerOneOffer.PlayerID == mainPlayerId)
                 .Map(x => x.PlayerOneOffer = x.PlayerOneOffer.WithAgrees(otherPlayerAgrees))
                 .Or(() => _tradeRepository.PlayerTwoOffer = _tradeRepository.PlayerTwoOffer.WithAgrees(otherPlayerAgrees));