feat: Pr133 Usable change (#403)

* PR-133 UsableItem

* No breaking change

* Remove breaking change

EXILED/Exiled.API/Features/Items/Item.cs

@@ -357,7 +357,7 @@ public static T Get<T>(ushort serial)
         /// <param name="owner">The <see cref="Player"/> who owns the item by default.</param>
         /// <typeparam name="T">The specified <see cref="Item"/> type.</typeparam>
         /// <returns>The <see cref="Item"/> created. This can be cast as a subclass.</returns>
-        public static Item Create<T>(ItemType type, Player owner = null)
+        public static Item Create<T>(ItemType type, Player owner = null) // TODO modify return type to "T"
             where T : Item => Create(type, owner) as T;
 
         /// <summary>

EXILED/Exiled.API/Features/Items/Usable.cs

@@ -7,6 +7,7 @@
 
 namespace Exiled.API.Features.Items
 {
+    using Exiled.API.Extensions;
     using Exiled.API.Features.Pickups;
     using Exiled.API.Interfaces;
 
@@ -126,13 +127,23 @@ public override Pickup CreatePickup(Vector3 position, Quaternion? rotation = nul
         /// <summary>
         /// Uses the item.
         /// </summary>
-        /// <exception cref="System.InvalidOperationException">The <see cref="Item.Owner"/> of the item cannot be <see langword="null"/>.</exception>
-        public virtual void Use()
+        public virtual void Use() => Use(Owner);
+
+        /// <summary>
+        /// Uses the item.
+        /// </summary>
+        /// <param name="owner">Target <see cref="Player"/> to use an <see cref="Usable"/>.</param>
+        public virtual void Use(Player owner = null)
         {
-            if (Owner is null)
-                throw new System.InvalidOperationException("The Owner of the item cannot be null.");
+            Player oldOwner = Owner;
+            owner ??= Owner;
+
+            Base.Owner = owner.ReferenceHub;
+            Base.ServerOnUsingCompleted();
+
+            typeof(UsableItemsController).InvokeStaticEvent(nameof(UsableItemsController.ServerOnUsingCompleted), new object[] { owner.ReferenceHub, Base });
 
-            Owner.UseItem(this);
+            Base.Owner = oldOwner.ReferenceHub;
         }
 
         /// <inheritdoc/>

EXILED/Exiled.API/Features/Pickups/Pickup.cs

@@ -532,7 +532,7 @@ public static IEnumerable<T> Get<T>(IEnumerable<GameObject> gameObjects)
         /// <typeparam name="T">The specified <see cref="Pickup"/> type.</typeparam>
         /// <returns>The created <see cref="Pickup"/>.</returns>
         /// <seealso cref="Projectile.Create(Enums.ProjectileType)"/>
-        public static Pickup Create<T>(ItemType type)
+        public static Pickup Create<T>(ItemType type) // TODO modify return type to "T"
             where T : Pickup => Create(type) as T;
 
         /// <summary>

EXILED/Exiled.API/Features/Pickups/Projectiles/Projectile.cs

@@ -111,7 +111,7 @@ internal Projectile(ItemType type)
         /// <param name="projectiletype">The <see cref="Enums.ProjectileType"/> of the projectile.</param>
         /// <typeparam name="T">The specified <see cref="Projectile"/> type.</typeparam>
         /// <returns>The created <see cref="Projectile"/>.</returns>
-        public static Projectile Create<T>(ProjectileType projectiletype)
+        public static Projectile Create<T>(ProjectileType projectiletype) // TODO modify return type to "T"
             where T : Projectile => Create(projectiletype) as T;
 
         /// <summary>

EXILED/Exiled.API/Features/Player.cs

@@ -2206,6 +2206,12 @@ public void Heal(float amount, bool overrideMaxHealth = false)
         /// <returns><see langword="true"/> if item was used successfully. Otherwise, <see langword="false"/>.</returns>
         public bool UseItem(ItemType usableItem) => UseItem(Item.Create(usableItem));
 
+        /// <summary>
+        /// Forces the player to use an item.
+        /// </summary>
+        /// <param name="usable">The item to be used.</param>
+        public void UseItem(Usable usable) => usable?.Use(this);
+
         /// <summary>
         /// Forces the player to use an item.
         /// </summary>
@@ -2216,14 +2222,7 @@ public bool UseItem(Item item)
             if (item is not Usable usableItem)
                 return false;
 
-            usableItem.Base.Owner = referenceHub;
-            usableItem.Base.ServerOnUsingCompleted();
-
-            typeof(UsableItemsController).InvokeStaticEvent(nameof(UsableItemsController.ServerOnUsingCompleted), new object[] { referenceHub, usableItem.Base });
-
-            if (usableItem.Base is not null)
-                usableItem.Destroy();
-
+            UseItem(usableItem);
             return true;
         }