* Add the ability to update profiles via the cache, so that cached profiles don't become stale

* Add corresponding unit test

2 files changed, 24 insertions, 1 deletions

diff --git a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
index 32c235a..98ec287 100644
--- a/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
+++ b/OpenSim/Framework/Communications/Cache/CachedUserInfo.cs
@@ -65,7 +65,7 @@ namespace OpenSim.Framework.Communications.Cache
         private readonly CommunicationsManager m_commsManager;
 
         public UserProfileData UserProfile { get { return m_userProfile; } }
-        private readonly UserProfileData m_userProfile;
+        protected internal UserProfileData m_userProfile;
 
         /// <summary>
         /// Have we received the user's inventory from the inventory service?
diff --git a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
index 714e279..36fe6e3 100644
--- a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
+++ b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
@@ -169,6 +169,29 @@ namespace OpenSim.Framework.Communications.Cache
         }
         
         /// <summary>
+        /// Update an existing profile
+        /// </summary>
+        /// <param name="userProfile"></param>
+        /// <returns>true if a user profile was found to update, false otherwise</returns>
+        public bool UpdateProfile(UserProfileData userProfile)
+        {            
+            lock (m_userProfilesById)
+            {            
+                CachedUserInfo userInfo = GetUserDetails(userProfile.ID);
+                
+                if (userInfo != null)
+                {
+                    userInfo.m_userProfile = userProfile;                   
+                    m_commsManager.UserService.UpdateUserProfile(userProfile);
+                    
+                    return true;
+                }
+            }
+            
+            return false;
+        }
+        
+        /// <summary>
         /// Populate caches with the given user profile
         /// </summary>
         /// <param name="userProfile"></param>