From 994a35407b479591ac84c14a8ce0e2521a0d4cac Mon Sep 17 00:00:00 2001
From: Cinder <cinder@cinderblocks.biz>
Date: Sun, 23 Jan 2022 14:26:13 -0600
Subject: [PATCH] Wire in AISv3 for Item and Folder removal in InventoryManager

---
 LibreMetaverse/InventoryAISClient.cs |  8 +++----
 LibreMetaverse/InventoryManager.cs   | 32 +++++++++++++++++++++++-----
 2 files changed, 31 insertions(+), 9 deletions(-)

diff --git a/LibreMetaverse/InventoryAISClient.cs b/LibreMetaverse/InventoryAISClient.cs
index 4e0e832a..a0ecdde4 100644
--- a/LibreMetaverse/InventoryAISClient.cs
+++ b/LibreMetaverse/InventoryAISClient.cs
@@ -166,7 +166,7 @@ public async Task SlamFolder(UUID folderUuid, OSD newInventory, Action<bool> cal
             }
         }
 
-        public async Task RemoveCategory(UUID categoryUuid, Action<bool> callback)
+        public async Task RemoveCategory(UUID categoryUuid, Action<bool, UUID> callback)
         {
             var cap = getInventoryCap();
             if (cap == null)
@@ -204,11 +204,11 @@ public async Task RemoveCategory(UUID categoryUuid, Action<bool> callback)
             }
             finally
             {
-                callback?.Invoke(success);
+                callback?.Invoke(success, categoryUuid);
             }
         }
 
-        public async Task RemoveItem(UUID itemUuid, Action<bool> callback)
+        public async Task RemoveItem(UUID itemUuid, Action<bool, UUID> callback)
         {
             var cap = getInventoryCap();
             if (cap == null)
@@ -247,7 +247,7 @@ public async Task RemoveItem(UUID itemUuid, Action<bool> callback)
             }
             finally
             {
-                callback?.Invoke(success);
+                callback?.Invoke(success, itemUuid);
             }
         }
 
diff --git a/LibreMetaverse/InventoryManager.cs b/LibreMetaverse/InventoryManager.cs
index 8a079d85..7ca7a962 100644
--- a/LibreMetaverse/InventoryManager.cs
+++ b/LibreMetaverse/InventoryManager.cs
@@ -1449,8 +1449,17 @@ public void RemoveDescendants(UUID folder)
         /// <param name="item">The <seealso cref="UUID"/> of the inventory item to remove</param>
         public void RemoveItem(UUID item)
         {
-            List<UUID> items = new List<UUID>(1) { item };
-            Remove(items, null);
+            if (Client.AisClient.IsAvailable)
+            {
+                Client.AisClient.RemoveItem(item, RemoveLocalUi).ConfigureAwait(false);
+            }
+            else
+            {
+                List<UUID> items = new List<UUID>(1) { item };
+#pragma warning disable CS0612 // Type or member is obsolete
+                Remove(items, null);
+#pragma warning restore CS0612 // Type or member is obsolete
+            }
         }
 
         /// <summary>
@@ -1459,15 +1468,26 @@ public void RemoveItem(UUID item)
         /// <param name="folder">The <seealso cref="UUID"/> of the folder to remove</param>
         public void RemoveFolder(UUID folder)
         {
-            List<UUID> folders = new List<UUID>(1) { folder };
-            Remove(null, folders);
+            if (Client.AisClient.IsAvailable)
+            {
+                Client.AisClient.RemoveCategory(folder, RemoveLocalUi).ConfigureAwait(false);
+            } 
+            else
+            {
+                List<UUID> folders = new List<UUID>(1) { folder };
+#pragma warning disable CS0612 // Type or member is obsolete
+                Remove(null, folders);
+#pragma warning restore CS0612 // Type or member is obsolete
+            }
         }
 
         /// <summary>
-        /// Remove multiple items or folders from inventory
+        /// Remove multiple items or folders from inventory. Note that this uses the LLUDP method
+        /// which Second Life has deprecated and removed.
         /// </summary>
         /// <param name="items">A List containing the <seealso cref="UUID"/>s of items to remove</param>
         /// <param name="folders">A List containing the <seealso cref="UUID"/>s of the folders to remove</param>
+        [Obsolete]
         public void Remove(List<UUID> items, List<UUID> folders)
         {
             if ((items == null || items.Count == 0) && (folders == null || folders.Count == 0))
@@ -1583,7 +1603,9 @@ private void EmptySystemFolder(FolderType folderType)
                     }
                 }
 
+#pragma warning disable CS0612 // Type or member is obsolete
                 Remove(remItems, remFolders);
+#pragma warning restore CS0612 // Type or member is obsolete
             }
         }
         #endregion Remove