* Refactor: Renaming item and folder tree search methods to have Find*() prefixes

3 files changed, 21 insertions, 22 deletions

diff --git a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
index ceca1a4..6a7b6b7 100644
--- a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
+++ b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
@@ -35,7 +35,8 @@ using log4net;
 
 namespace OpenSim.Framework.Communications.Cache
 {
-    internal delegate void CreateFolderDelegate(string folderName, LLUUID folderID, ushort folderType, LLUUID parentID); 
+    //internal delegate void DeleteItemDelegate(
+    internal delegate void CreateFolderDelegate(string folderName, LLUUID folderID, ushort folderType, LLUUID parentID);
     internal delegate void MoveFolderDelegate(LLUUID folderID, LLUUID parentID);         
     internal delegate void PurgeFolderDelegate(LLUUID folderID);
     internal delegate void UpdateFolderDelegate(string name, LLUUID folderID, ushort type, LLUUID parentID);    
@@ -234,7 +235,7 @@ namespace OpenSim.Framework.Communications.Cache
             }
             else
             {
-                InventoryFolderImpl folder = RootFolder.GetDescendentFolder(folderInfo.ParentID);
+                InventoryFolderImpl folder = RootFolder.FindFolder(folderInfo.ParentID);
                 lock (folder.SubFolders)
                 {
                     if (folder != null)
@@ -275,7 +276,6 @@ namespace OpenSim.Framework.Communications.Cache
                     {
                         if (!RootFolder.Items.ContainsKey(itemInfo.ID))
                         {
-
                             RootFolder.Items.Add(itemInfo.ID, itemInfo);
                         }
                         else 
@@ -286,7 +286,7 @@ namespace OpenSim.Framework.Communications.Cache
                 }
                 else
                 {
-                    InventoryFolderImpl folder = RootFolder.GetDescendentFolder(itemInfo.Folder);
+                    InventoryFolderImpl folder = RootFolder.FindFolder(itemInfo.Folder);
                     if (folder != null)
                     {
                         lock (folder.Items)
@@ -346,7 +346,7 @@ namespace OpenSim.Framework.Communications.Cache
                 }
                 else
                 {
-                    InventoryFolderImpl folder = RootFolder.GetDescendentFolder(parentID);
+                    InventoryFolderImpl folder = RootFolder.FindFolder(parentID);
                     
                     if (folder != null)
                     {
@@ -479,7 +479,7 @@ namespace OpenSim.Framework.Communications.Cache
             
             if (HasInventory)
             {
-                InventoryFolderImpl purgedFolder = RootFolder.GetDescendentFolder(folderID);
+                InventoryFolderImpl purgedFolder = RootFolder.FindFolder(folderID);
                 
                 if (purgedFolder != null)
                 {                        
diff --git a/OpenSim/Framework/Communications/Cache/InventoryFolderImpl.cs b/OpenSim/Framework/Communications/Cache/InventoryFolderImpl.cs
index f6accf6..39296eb 100644
--- a/OpenSim/Framework/Communications/Cache/InventoryFolderImpl.cs
+++ b/OpenSim/Framework/Communications/Cache/InventoryFolderImpl.cs
@@ -104,14 +104,12 @@ namespace OpenSim.Framework.Communications.Cache
         }
 
         /// <summary>
-        /// Does this folder or any of its subfolders contain the given item?
+        /// Returns the item if it exists in this folder or any of this folder's subfolders?
         /// </summary>
         /// <param name="itemID"></param>
-        /// <returns></returns>
-        public InventoryItemBase HasItem(LLUUID itemID)
+        /// <returns>null if the item is not found</returns>
+        public InventoryItemBase FindItem(LLUUID itemID)
         {
-            InventoryItemBase base2 = null;
-            
             lock (Items)
             {
                 if (Items.ContainsKey(itemID))
@@ -124,15 +122,16 @@ namespace OpenSim.Framework.Communications.Cache
             {
                 foreach (InventoryFolderImpl folder in SubFolders.Values)
                 {
-                    base2 = folder.HasItem(itemID);
-                    if (base2 != null)
+                    InventoryItemBase item = folder.FindItem(itemID);
+                    
+                    if (item != null)
                     {
-                        break;
+                        return item;
                     }
                 }
             }
             
-            return base2;
+            return null;
         }
 
         /// <summary>
@@ -171,7 +170,7 @@ namespace OpenSim.Framework.Communications.Cache
         /// Returns the folder requested if it exists as a descendent of this folder
         /// </summary>
         /// <returns>The requested folder if it exists, null if it does not.</returns>
-        public InventoryFolderImpl GetDescendentFolder(LLUUID folderID)
+        public InventoryFolderImpl FindFolder(LLUUID folderID)
         {            
             InventoryFolderImpl returnFolder = null;
             
@@ -185,7 +184,7 @@ namespace OpenSim.Framework.Communications.Cache
                 {
                     foreach (InventoryFolderImpl folder in SubFolders.Values)
                     {
-                        returnFolder = folder.GetDescendentFolder(folderID);
+                        returnFolder = folder.FindFolder(folderID);
                         if (returnFolder != null)
                         {
                             break;
diff --git a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
index 0404477..92b08e5 100644
--- a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
+++ b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
@@ -259,7 +259,7 @@ namespace OpenSim.Framework.Communications.Cache
                 return;
             }
 
-            if ((fold = libraryRoot.GetDescendentFolder(folderID)) != null)
+            if ((fold = libraryRoot.FindFolder(folderID)) != null)
             {
                 remoteClient.SendInventoryFolderDetails(
                     libraryRoot.Owner, folderID, fold.RequestListOfItems(),
@@ -308,7 +308,7 @@ namespace OpenSim.Framework.Communications.Cache
                     }
                     else
                     {
-                        if ((fold = userProfile.RootFolder.GetDescendentFolder(folderID)) != null)
+                        if ((fold = userProfile.RootFolder.FindFolder(folderID)) != null)
                         {
 //                            m_log.DebugFormat(
 //                                "[AGENT INVENTORY]: Found folder {0} for client {1}", 
@@ -369,7 +369,7 @@ namespace OpenSim.Framework.Communications.Cache
                 return libraryRoot.RequestListOfItems();
             }
 
-            if ((fold = libraryRoot.GetDescendentFolder(folderID)) != null)
+            if ((fold = libraryRoot.FindFolder(folderID)) != null)
             {
                 return fold.RequestListOfItems();
             }         
@@ -410,7 +410,7 @@ namespace OpenSim.Framework.Communications.Cache
                     }
                     else
                     {
-                        if ((fold = userProfile.RootFolder.GetDescendentFolder(folderID)) != null)
+                        if ((fold = userProfile.RootFolder.FindFolder(folderID)) != null)
                         {
                             return fold.RequestListOfItems();
                         }
@@ -479,7 +479,7 @@ namespace OpenSim.Framework.Communications.Cache
             {
                 if (userProfile.HasInventory)
                 {
-                    InventoryItemBase item = userProfile.RootFolder.HasItem(itemID);
+                    InventoryItemBase item = userProfile.RootFolder.FindItem(itemID);
                     if (item != null)
                     {
                         remoteClient.SendInventoryItemDetails(ownerID, item);