panda's review (II)

- tested and working as intended with both: bag and bank item slots

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

Author: Arufonsu

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

@@ -63,17 +63,17 @@ protected override void OnContextMenuOpening(ContextMenu contextMenu)
             return;
         }
 
-        if (bagSlots[SlotIndex] is null)
+        if (SlotIndex >= bagSlots.Length)
         {
             return;
         }
 
-        if (SlotIndex >= bagSlots.Length)
+        if (bagSlots[SlotIndex] is not { } bagSlot)
         {
             return;
         }
 
-        if (!ItemDescriptor.TryGet(bagSlots[SlotIndex].ItemId, out var item))
+        if (!ItemDescriptor.TryGet(bagSlot.ItemId, out var item))
         {
             return;
         }

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

@@ -66,17 +66,17 @@ protected override void OnContextMenuOpening(ContextMenu contextMenu)
             return;
         }
 
-        if (bankSlots[SlotIndex] is null)
+        if (SlotIndex >= bankSlots.Length)
         {
             return;
         }
 
-        if (SlotIndex >= bankSlots.Length)
+        if (bankSlots[SlotIndex] is not { } bankSlot)
         {
             return;
         }
 
-        if (!ItemDescriptor.TryGet(bankSlots[SlotIndex].ItemId, out var item))
+        if (!ItemDescriptor.TryGet(bankSlot.ItemId, out var item))
         {
             return;
         }