Fix item use handler to handle correct packet type. Fix inventory amo…

…unt update
ethanmoffat · May 9, 2021 · be69c44 · be69c44
1 parent 7ded0ec
commit be69c44
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/EOLib/PacketHandlers/Items/UseItemHandler.cs b/EOLib/PacketHandlers/Items/UseItemHandler.cs
@@ -23,7 +23,7 @@ public class UseItemHandler : InGameOnlyPacketHandler
 
         public override PacketFamily Family => PacketFamily.Item;
 
-        public override PacketAction Action => PacketAction.Use;
+        public override PacketAction Action => PacketAction.Reply;
 
         public UseItemHandler(IPlayerInfoProvider playerInfoProvider,
                               ICharacterInventoryRepository characterInventoryRepository,
@@ -48,7 +48,11 @@ public override bool HandlePacket(IPacket packet)
             var weight = packet.ReadChar();
             var maxWeight = packet.ReadChar();
 
-            _characterInventoryRepository.ItemInventory.RemoveAll(x => x.ItemID == itemId);
+            var oldItem = _characterInventoryRepository.ItemInventory.SingleOrDefault(x => x.ItemID == itemId);
+            if (oldItem == null)
+                return false;
+
+            _characterInventoryRepository.ItemInventory.Remove(oldItem);
             _characterInventoryRepository.ItemInventory.Add(new InventoryItem(itemId, amount));
 
             var character = _characterRepository.MainCharacter;