* refactor: Move RequestInventoryForUser() from service to CachedUserInfo

* This simplifies callers in most cases - CachedUserInfo is already handling the rest of the fetch inventory work anyway

2 files changed, 19 insertions, 30 deletions

diff --git a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
index 15066cd..c5bbd6a 100644
--- a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
+++ b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
@@ -202,6 +202,25 @@ namespace OpenSim.Framework.Communications.Cache
                 m_rootFolder = null;
             }
         }
+        
+        /// <summary>
+        /// Fetch inventory for this user.
+        /// </summary>
+        /// This has to be executed as a separate step once user information is retreived.  
+        /// This will occur synchronously if the inventory service is in the same process as this class, and
+        /// asynchronously otherwise.
+        public void FetchInventory()
+        {
+            if (m_commsManager.SecureInventoryService != null)
+            {
+                m_commsManager.SecureInventoryService.RequestInventoryForUser(
+                    UserProfile.ID, SessionID, InventoryReceive);
+            }
+            else
+            {
+                m_commsManager.InventoryService.RequestInventoryForUser(UserProfile.ID, InventoryReceive);
+            }           
+        }
 
         /// <summary>
         /// Callback invoked when the inventory is received from an async request to the inventory service
diff --git a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
index ef190c8..b1ce3e7 100644
--- a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
+++ b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
@@ -105,36 +105,6 @@ namespace OpenSim.Framework.Communications.Cache
         }
 
         /// <summary>
-        /// Request the inventory data for the given user.  This will occur asynchronously if running on a grid
-        /// </summary>
-        /// <param name="userID"></param>
-        /// <param name="userInfo"></param>
-        public void RequestInventoryForUser(UUID userID)
-        {
-            CachedUserInfo userInfo = GetUserDetails(userID);
-            if (userInfo != null)
-            {
-                if (m_commsManager.SecureInventoryService != null)
-                {
-                    m_commsManager.SecureInventoryService.RequestInventoryForUser(userID, userInfo.SessionID, userInfo.InventoryReceive);
-                }
-                else
-                {
-                    m_commsManager.InventoryService.RequestInventoryForUser(userID, userInfo.InventoryReceive);
-                }
-                //IInventoryServices invService = userInfo.GetInventoryService();
-                //if (invService != null)
-                //{
-                //    invService.RequestInventoryForUser(userID, userInfo.InventoryReceive);
-                //}
-            }
-            else
-            {
-                m_log.ErrorFormat("[USER CACHE]: RequestInventoryForUser() - user profile for user {0} not found", userID);
-            }
-        }
-
-        /// <summary>
         /// Get cached details of the given user.  If the user isn't in cache then the user is requested from the 
         /// profile service.  
         /// </summary>