feature: unequip item by slot (AscensionGameDev#2256)

* feature: unequip by slot
- Adds ability to unequip items by slots

* fix: unequip options enabled
- as long as mMyCommand.Unequip is true, both unequip options should be enabled, they shouldn't disable each other

---------

Co-authored-by: Arufonsu <[email protected]>

Author: WeylonSantana

Intersect (Core)/GameObjects/Events/Commands/EventCommands.cs

@@ -482,8 +482,12 @@ public partial class EquipItemCommand : EventCommand
 
         public Guid ItemId { get; set; }
 
+        public int Slot { get; set; }
+
         public bool Unequip { get; set; }
 
+        public bool IsItem { get; set; } = true;
+
         public bool TriggerCooldown { get; set; }
 
     }

Intersect.Editor/Forms/Editors/Events/CommandPrinter.cs

@@ -864,7 +864,25 @@ private static string GetCommandText(ChangeItemsCommand command, MapInstance map
 
         private static string GetCommandText(EquipItemCommand command, MapInstance map)
         {
-            return command.Unequip ? Strings.EventCommandList.unequipitem.ToString(ItemBase.GetName(command.ItemId)) : Strings.EventCommandList.equipitem.ToString(ItemBase.GetName(command.ItemId));
+            var commandText = string.Empty;
+
+            if(!command.Unequip)
+            {
+                commandText = Strings.EventCommandList.equipitem.ToString(ItemBase.GetName(command.ItemId));
+            }
+            else
+            {
+                if (command.IsItem)
+                {
+                     commandText = Strings.EventCommandList.unequipitem.ToString(ItemBase.GetName(command.ItemId));
+                }
+                else
+                {
+                    commandText = Strings.EventCommandList.unequipslot.ToString(Options.EquipmentSlots[command.Slot]);
+                }
+            }
+
+            return commandText;
         }
 
         private static string GetCommandText(ChangeSpriteCommand command, MapInstance map)

Intersect.Editor/Forms/Editors/Events/Event Commands/EventCommand_EquipItem.Designer.cs

@@ -26,22 +26,30 @@ public EventCommandEquipItems(EquipItemCommand refCommand, FrmEvent editor)
             cmbItem.Items.AddRange(ItemBase.Names);
             cmbItem.SelectedIndex = ItemBase.ListIndex(mMyCommand.ItemId);
             chkUnequip.Checked = mMyCommand.Unequip;
+            optUnequipItem.Enabled = mMyCommand.Unequip;
+            optUnequipItem.Checked = mMyCommand.Unequip && mMyCommand.IsItem;
+            optUnequipSlot.Enabled = mMyCommand.Unequip;
+            optUnequipSlot.Checked = mMyCommand.Unequip && !mMyCommand.IsItem;
             chkTriggerCooldown.Checked = mMyCommand.TriggerCooldown;
         }
 
         private void InitLocalization()
         {
             grpEquipItem.Text = Strings.EventEquipItems.title;
-            chkUnequip.Text = Strings.EventEquipItems.unequip;
             chkTriggerCooldown.Text = Strings.EventEquipItems.TriggerCooldown;
+            chkUnequip.Text = Strings.EventEquipItems.unequip;
+            optUnequipItem.Text = Strings.EventEquipItems.item;
+            optUnequipSlot.Text = Strings.EventEquipItems.slot;
             btnSave.Text = Strings.EventEquipItems.okay;
             btnCancel.Text = Strings.EventEquipItems.cancel;
         }
 
         private void btnSave_Click(object sender, EventArgs e)
         {
-            mMyCommand.ItemId = ItemBase.IdFromList(cmbItem.SelectedIndex);
+            mMyCommand.ItemId = optUnequipItem.Checked || !chkUnequip.Checked ? ItemBase.IdFromList(cmbItem.SelectedIndex) : Guid.Empty;
+            mMyCommand.Slot = optUnequipSlot.Checked ? cmbItem.SelectedIndex : -1;
             mMyCommand.Unequip = chkUnequip.Checked;
+            mMyCommand.IsItem = optUnequipItem.Checked;
             mMyCommand.TriggerCooldown = chkTriggerCooldown.Checked;
             mEventEditor.FinishCommandEdit();
         }
@@ -57,12 +65,34 @@ private void chkUnequip_CheckedChanged(object sender, EventArgs e)
             {
                 chkTriggerCooldown.Checked = false;
                 chkTriggerCooldown.Enabled = false;
+                optUnequipItem.Checked = mMyCommand.IsItem;
+                optUnequipItem.Enabled = true;
+                optUnequipSlot.Checked = !mMyCommand.IsItem;
+                optUnequipSlot.Enabled = true;
             }
             else
             {
                 chkTriggerCooldown.Enabled = true;
+                optUnequipItem.Checked = false;
+                optUnequipItem.Enabled = false;
+                optUnequipSlot.Checked = false;
+                optUnequipSlot.Enabled = false;
+                UpdateUnequipOptions();
             }
         }
+
+        private void optUnequipSlot_CheckedChanged(object sender, EventArgs e)
+        {
+            UpdateUnequipOptions();
+        }
+
+        private void UpdateUnequipOptions()
+        {
+            cmbItem.Items.Clear();
+            cmbItem.Items.AddRange(optUnequipItem.Checked || !chkUnequip.Checked ? ItemBase.Names : Options.EquipmentSlots.ToArray());
+            cmbItem.SelectedIndex = cmbItem.Items.Count > 0 ? 0 : -1;
+            lblItem.Text = optUnequipItem.Checked || !chkUnequip.Checked ? Strings.EventEquipItems.item : Strings.EventEquipItems.slot;
+        }
     }
 
 }