* Refactor: Eliminate RequestUsersRoot() redundant method

2 files changed, 9 insertions, 13 deletions

diff --git a/OpenSim/Framework/Communications/InventoryServiceBase.cs b/OpenSim/Framework/Communications/InventoryServiceBase.cs
index 78de4a5..595fa65 100644
--- a/OpenSim/Framework/Communications/InventoryServiceBase.cs
+++ b/OpenSim/Framework/Communications/InventoryServiceBase.cs
@@ -110,18 +110,9 @@ namespace OpenSim.Framework.Communications
         }
 
         // See IInventoryServices
-        public InventoryFolderBase RequestUsersRoot(LLUUID userID)
-        {
-            foreach (KeyValuePair<string, IInventoryData> plugin in m_plugins)
-            {
-                return plugin.Value.getUserRootFolder(userID);
-            }
-            return null;
-        }
-
-        // See IInventoryServices
         public void MoveInventoryFolder(LLUUID userID, InventoryFolderBase folder)
         {
+            // FIXME: Probably doesn't do what was originally intended - only ever queries the first plugin
             foreach (KeyValuePair<string, IInventoryData> plugin in m_plugins)
             {
                 plugin.Value.moveInventoryFolder(folder);
@@ -137,7 +128,12 @@ namespace OpenSim.Framework.Communications
         // See IInventoryServices
         public InventoryFolderBase RequestRootFolder(LLUUID userID)
         {
-            return RequestUsersRoot(userID);
+            // FIXME: Probably doesn't do what was originally intended - only ever queries the first plugin
+            foreach (KeyValuePair<string, IInventoryData> plugin in m_plugins)
+            {
+                return plugin.Value.getUserRootFolder(userID);
+            }
+            return null;
         }
 
         // See IInventoryServices
@@ -149,7 +145,7 @@ namespace OpenSim.Framework.Communications
         // See IInventoryServices
         public void CreateNewUserInventory(LLUUID user)
         {
-            InventoryFolderBase existingRootFolder = RequestUsersRoot(user);
+            InventoryFolderBase existingRootFolder = RequestRootFolder(user);
             
             if (null != existingRootFolder)
             {
diff --git a/OpenSim/Region/Communications/Local/LocalInventoryService.cs b/OpenSim/Region/Communications/Local/LocalInventoryService.cs
index 3396d5c..f154cd8 100644
--- a/OpenSim/Region/Communications/Local/LocalInventoryService.cs
+++ b/OpenSim/Region/Communications/Local/LocalInventoryService.cs
@@ -88,7 +88,7 @@ namespace OpenSim.Region.Communications.Local
 
         public override bool HasInventoryForUser(LLUUID userID)
         {
-            InventoryFolderBase root = RequestUsersRoot(userID);
+            InventoryFolderBase root = RequestRootFolder(userID);
             if (root == null)
             {
                 return false;