refactor the core of system with correct data/type

Author: WeylonSantana

Intersect.Client.Core/Interface/Game/Draggable.cs

@@ -8,12 +8,18 @@ public partial class Draggable(Base parent, string name) : ImagePanel(parent, na
 {
     public bool IsDragging => DragAndDrop.CurrentPackage?.DrawControl == this;
 
+    // TODO: Fix drag and drop names
     public override bool DragAndDrop_Draggable()
     {
         return true;
     }
 
-    public override Package DragAndDrop_GetPackage(int x, int y)
+    public override bool DragAndDrop_CanAcceptPackage(Package package)
+    {
+        return true;
+    }
+
+    public override Package? DragAndDrop_GetPackage(int x, int y)
     {
         return new Package()
         {

Intersect.Client.Core/Interface/Game/Shop/ShopItem.cs

@@ -163,7 +163,7 @@ public void LoadItem()
         }
     }
 
-    public override bool DragAndDrop_HandleDrop(Package p, int x, int y)
+    public override bool DragAndDrop_HandleDrop(Package package, int x, int y)
     {
         var targetNode = Interface.FindComponentUnderCursor();
 

Intersect.Client.Core/Interface/Game/SlotItem.cs

@@ -1,5 +1,6 @@
 using Intersect.Client.Framework.Gwen;
 using Intersect.Client.Framework.Gwen.Control;
+using Intersect.Client.Framework.Gwen.DragDrop;
 
 namespace Intersect.Client.Interface.Game;
 
@@ -42,6 +43,11 @@ public void OpenContextMenu()
         OnContextMenuOpening(contextMenu);
     }
 
+    public override bool DragAndDrop_CanAcceptPackage(Package package)
+    {
+        return true;
+    }
+
     protected virtual void OnContextMenuOpening(ContextMenu contextMenu)
     {
         // Display our menu... If we have anything to display.

Intersect.Client.Core/Interface/Game/Spells/SpellItem.cs

@@ -161,6 +161,11 @@ private void Icon_DoubleClicked(Base sender, MouseButtonState arguments)
 
     public override bool DragAndDrop_HandleDrop(Package package, int x, int y)
     {
+        if (Globals.Me is not { } player)
+        {
+            return false;
+        }
+
         var targetNode = Interface.FindComponentUnderCursor();
 
         // Find the first parent acceptable in that tree that can accept the package
@@ -169,11 +174,11 @@ public override bool DragAndDrop_HandleDrop(Package package, int x, int y)
             switch (targetNode)
             {
                 case SpellItem spellItem:
-                    Globals.Me?.SwapSpells(SlotIndex, spellItem.SlotIndex);
+                    player.SwapSpells(SlotIndex, spellItem.SlotIndex);
                     return true;
 
                 case HotbarItem hotbarItem:
-                    Globals.Me?.AddToHotbar(hotbarItem.SlotIndex, 1, SlotIndex);
+                    player.AddToHotbar(hotbarItem.SlotIndex, 1, SlotIndex);
                     return true;
 
                 default:

Intersect.Client.Framework/Gwen/Control/Base.cs

@@ -4258,7 +4258,7 @@ public virtual void UpdateCursor()
     }
 
     // giver
-    public virtual Package DragAndDrop_GetPackage(int x, int y)
+    public virtual Package? DragAndDrop_GetPackage(int x, int y)
     {
         return _dragPayload;
     }
@@ -4330,7 +4330,7 @@ public virtual void DragAndDrop_Hover(Package p, int x, int y)
     // receiver
     public virtual bool DragAndDrop_CanAcceptPackage(Package p)
     {
-        return true;
+        return false;
     }
 
     public record struct SizeToChildrenArgs(