Merge pull request #404 from radzki/fix/392

fix: Adds conditions to avoid equipping two-handed item when all slots are occupied
MUnique · Apr 10, 2024 · 4b5ffb9 · 4b5ffb9
2 parents 29229f6 + c23b6b8
commit 4b5ffb9
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/GameLogic/PlayerActions/Items/MoveItemAction.cs b/src/GameLogic/PlayerActions/Items/MoveItemAction.cs
@@ -238,9 +238,12 @@ private async ValueTask<Movement> CanMoveAsync(Player player, Item item, byte to
                 static bool IsOneHandedOrShield(ItemDefinition definition) =>
                     (definition.ItemSlot!.ItemSlots.Contains(RightHandSlot) && definition.ItemSlot.ItemSlots.Contains(LeftHandSlot)) || definition.Group == 6;
 
+                var rightHandItemDefinition = storage.GetItem(RightHandSlot)?.Definition!;
+
                 if ((toSlot == LeftHandSlot
                     && itemDefinition.Width >= 2
-                    && storage.GetItem(RightHandSlot)?.Definition!.Group == 6)
+                    && rightHandItemDefinition != null
+                    && !rightHandItemDefinition.IsAmmunition)
                     || (toSlot == RightHandSlot
                     && IsOneHandedOrShield(itemDefinition)
                     && storage.GetItem(LeftHandSlot)?.Definition!.Width >= 2))