From 55b569069dfd6eb7c87d4fbd66d68083878f6c65 Mon Sep 17 00:00:00 2001
From: Sean Dague
Date: Tue, 31 Jul 2007 14:42:50 +0000
Subject: clear userAgent state on client shutdown, which fixes the issue where
 you could only login once with a given id in standalone mode.

---
 OpenSim/Framework/Communications/IUserServices.cs  |  1 +
 OpenSim/Framework/Data.DB4o/DB4oManager.cs         | 23 +++++++++-------
 OpenSim/Framework/Data.DB4o/DB4oUserData.cs        | 19 ++++++++++++-
 OpenSim/Framework/Data.MySQL/MySQLUserData.cs      |  7 +++++
 OpenSim/Framework/Data/UserData.cs                 |  6 +++++
 OpenSim/Framework/UserManager/UserManagerBase.cs   | 31 ++++++++++++++++++++++
 .../Region/Communications/OGS1/OGS1UserServices.cs |  5 ++++
 OpenSim/Region/Environment/Scenes/Scene.cs         |  4 ++-
 8 files changed, 84 insertions(+), 12 deletions(-)

(limited to 'OpenSim')

diff --git a/OpenSim/Framework/Communications/IUserServices.cs b/OpenSim/Framework/Communications/IUserServices.cs
index cd97b52..40a3182 100644
--- a/OpenSim/Framework/Communications/IUserServices.cs
+++ b/OpenSim/Framework/Communications/IUserServices.cs
@@ -35,6 +35,7 @@ namespace OpenSim.Framework.Communications
         UserProfileData GetUserProfile(string firstName, string lastName);
         UserProfileData GetUserProfile(string name);
         UserProfileData GetUserProfile(LLUUID avatarID);
+        void clearUserAgent(LLUUID avatarID);
 
         UserProfileData SetupMasterUser(string firstName, string lastName);
         UserProfileData SetupMasterUser(string firstName, string lastName, string password);
diff --git a/OpenSim/Framework/Data.DB4o/DB4oManager.cs b/OpenSim/Framework/Data.DB4o/DB4oManager.cs
index 43f9095..0e32938 100644
--- a/OpenSim/Framework/Data.DB4o/DB4oManager.cs
+++ b/OpenSim/Framework/Data.DB4o/DB4oManager.cs
@@ -129,26 +129,31 @@ namespace OpenSim.Framework.Data.DB4o
         }
 
         /// <summary>
-        /// Adds a new profile to the database (Warning: Probably slow.)
+        /// Adds or updates a record to the user database.  Do this when changes are needed
+        /// in the user profile that need to be persistant.
+        /// 
+        /// TODO: the logic here is not ACID, the local cache will be
+        /// updated even if the persistant data is not.  This may lead
+        /// to unexpected results.
         /// </summary>
-        /// <param name="row">The profile to add</param>
-        /// <returns>Successful?</returns>
-        public bool AddRow(UserProfileData row)
+        /// <param name="record">The profile to update</param>
+        /// <returns>true on success, false on fail to persist to db</returns>
+        public bool UpdateRecord(UserProfileData record)
         { 
-            if (userProfiles.ContainsKey(row.UUID))
+            if (userProfiles.ContainsKey(record.UUID))
             {
-                userProfiles[row.UUID] = row;
+                userProfiles[record.UUID] = record;
             }
             else
             {
-                userProfiles.Add(row.UUID, row);
+                userProfiles.Add(record.UUID, record);
             }
 
             try
             {
                 IObjectContainer database;
                 database = Db4oFactory.OpenFile(dbfl);
-                database.Set(row);
+                database.Set(record);
                 database.Close();
                 return true;
             }
@@ -157,7 +162,5 @@ namespace OpenSim.Framework.Data.DB4o
                 return false;
             }
         }
-
-
     }
 }
diff --git a/OpenSim/Framework/Data.DB4o/DB4oUserData.cs b/OpenSim/Framework/Data.DB4o/DB4oUserData.cs
index 831b198..88caeb8 100644
--- a/OpenSim/Framework/Data.DB4o/DB4oUserData.cs
+++ b/OpenSim/Framework/Data.DB4o/DB4oUserData.cs
@@ -139,13 +139,30 @@ namespace OpenSim.Framework.Data.DB4o
         {
             try
             {
-                manager.AddRow(user);
+                manager.UpdateRecord(user);
             }
             catch (Exception e)
             {
                 Console.WriteLine(e.ToString());
             }
         }
+      
+        /// <summary>
+        /// Creates a new user profile
+        /// </summary>
+        /// <param name="user">The profile to add to the database</param>
+        /// <returns>True on success, false on error</returns>
+        public bool updateUserProfile(UserProfileData user)
+        {
+            try {
+                return manager.UpdateRecord(user);
+            } catch (Exception e) {
+                Console.WriteLine(e.ToString());
+                return false;
+            }
+        }
+
+      
 
         /// <summary>
         /// Creates a new user agent
diff --git a/OpenSim/Framework/Data.MySQL/MySQLUserData.cs b/OpenSim/Framework/Data.MySQL/MySQLUserData.cs
index 66ea465..e301eb8 100644
--- a/OpenSim/Framework/Data.MySQL/MySQLUserData.cs
+++ b/OpenSim/Framework/Data.MySQL/MySQLUserData.cs
@@ -224,6 +224,13 @@ namespace OpenSim.Framework.Data.MySQL
         {
             // Do nothing.
         }
+        
+        
+        public bool updateUserProfile(UserProfileData user)
+        {
+            return true;
+            // TODO: implement
+        }
 
         /// <summary>
         /// Performs a money transfer request between two accounts
diff --git a/OpenSim/Framework/Data/UserData.cs b/OpenSim/Framework/Data/UserData.cs
index d849e12..cf8ec7b 100644
--- a/OpenSim/Framework/Data/UserData.cs
+++ b/OpenSim/Framework/Data/UserData.cs
@@ -85,6 +85,12 @@ namespace OpenSim.Framework.Data
         void addNewUserProfile(UserProfileData user);
 
         /// <summary>
+        ///  Updates an existing user profile
+        /// </summary>
+        /// <param name="user">UserProfile to update</param>
+        bool updateUserProfile(UserProfileData user);
+
+        /// <summary>
         /// Adds a new agent to the database
         /// </summary>
         /// <param name="agent">The agent to add</param>
diff --git a/OpenSim/Framework/UserManager/UserManagerBase.cs b/OpenSim/Framework/UserManager/UserManagerBase.cs
index bc923b6..61b5df4 100644
--- a/OpenSim/Framework/UserManager/UserManagerBase.cs
+++ b/OpenSim/Framework/UserManager/UserManagerBase.cs
@@ -155,6 +155,28 @@ namespace OpenSim.Framework.UserManagement
 
             return null;
         }
+
+        /// <summary>
+        /// Set's user profile from object
+        /// </summary>
+        /// <param name="fname">First name</param>
+        /// <param name="lname">Last name</param>
+        /// <returns>A user profile</returns>
+        public bool setUserProfile(UserProfileData data)
+        {
+            foreach (KeyValuePair<string, IUserData> plugin in _plugins)
+            {
+                try {
+                    plugin.Value.updateUserProfile(data);
+                    return true;
+                } catch (Exception e) {
+                    MainLog.Instance.Verbose( "Unable to set user via " + plugin.Key + "(" + e.ToString() + ")");
+                }
+            }
+            
+            return false;
+        }
+
         #endregion
 
         #region Get UserAgent
@@ -202,6 +224,15 @@ namespace OpenSim.Framework.UserManagement
             return null;
         }
 
+        // TODO: document
+        public void clearUserAgent(LLUUID agentID)
+        {
+            UserProfileData profile = getUserProfile(agentID);
+            profile.currentAgent = null;
+            setUserProfile(profile);
+        }
+
+
         /// <summary>
         /// Loads a user agent by name (not called directly)
         /// </summary>
diff --git a/OpenSim/Region/Communications/OGS1/OGS1UserServices.cs b/OpenSim/Region/Communications/OGS1/OGS1UserServices.cs
index cd9c4fe..0459e80 100644
--- a/OpenSim/Region/Communications/OGS1/OGS1UserServices.cs
+++ b/OpenSim/Region/Communications/OGS1/OGS1UserServices.cs
@@ -86,6 +86,11 @@ namespace OpenSim.Region.Communications.OGS1
             return null;
         }
 
+        public void clearUserAgent(LLUUID avatarID) 
+        {
+            // TODO: implement
+        }
+
         public UserProfileData SetupMasterUser(string firstName, string lastName)
         {
             return SetupMasterUser(firstName, lastName, "");
diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index 70aead3..dbc6f92 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -674,7 +674,9 @@ namespace OpenSim.Region.Environment.Scenes
                 }
             }
             // TODO: Add the removal from physics ?
-
+            
+            // Remove client agent from profile, so new logins will work
+            commsManager.UserServer.clearUserAgent(agentID);
 
             return;
         }
-- 
cgit v1.1