panda's review

Signed-off-by: Arufonsu <17498701+Arufonsu@users.noreply.github.com>

Author: Arufonsu

Intersect.Client.Core/Interface/Game/Bag/BagItem.cs

@@ -56,20 +56,28 @@ public BagItem(BagWindow bagWindow, Base parent, int index, ContextMenu contextM
 
     protected override void OnContextMenuOpening(ContextMenu contextMenu)
     {
-        // Clear out the old options since we might not show all of them
         contextMenu.ClearChildren();
 
-        if (Globals.BagSlots[SlotIndex] is not { } bagSlot)
+        if (Globals.BagSlots is not { Length: > 0 } bagSlots)
+        {
+            return;
+        }
+
+        if (bagSlots[SlotIndex] is null)
+        {
+            return;
+        }
+
+        if (SlotIndex >= bagSlots.Length)
         {
             return;
         }
 
-        if (!ItemDescriptor.TryGet(bagSlot.ItemId, out var item))
+        if (!ItemDescriptor.TryGet(bagSlots[SlotIndex].ItemId, out var item))
         {
             return;
         }
 
-        // update context menu
         _withdrawContextItem.SetText(Strings.BagContextMenu.Withdraw.ToString(item.Name));
         contextMenu.AddChild(_withdrawContextItem);
         base.OnContextMenuOpening(contextMenu);

Intersect.Client.Core/Interface/Game/Bank/BankItem.cs

@@ -59,20 +59,29 @@ public BankItem(BankWindow bankWindow, Base parent, int index, ContextMenu conte
 
     protected override void OnContextMenuOpening(ContextMenu contextMenu)
     {
-        // Clear out the old options since we might not show all of them
-        contextMenu.ClearChildren();
+        contextMenu.ClearChildren(); // Clear context menu
+
+        if (Globals.BankSlots is not { Length: > 0 } bankSlots)
+        {
+            return;
+        }
+
+        if (bankSlots[SlotIndex] is null)
+        {
+            return;
+        }
 
-        if (Globals.BankSlots[SlotIndex] is not { } bankSlot)
+        if (SlotIndex >= bankSlots.Length)
         {
             return;
         }
 
-        if (!ItemDescriptor.TryGet(bankSlot.ItemId, out var item))
+        if (!ItemDescriptor.TryGet(bankSlots[SlotIndex].ItemId, out var item))
         {
             return;
         }
 
-        // update context menu
+        // Update context menu
         _withdrawContextItem.SetText(Strings.BankContextMenu.Withdraw.ToString(item.Name));
         contextMenu.AddChild(_withdrawContextItem);
         base.OnContextMenuOpening(contextMenu);