From 9f7639fd6d2d7fd37587223259303377ee15bab4 Mon Sep 17 00:00:00 2001 From: Jeff Ames Date: Tue, 18 Mar 2008 03:41:05 +0000 Subject: Formatting cleanup. --- OpenSim/Framework/Communications/LoginService.cs | 163 ++++++++++----------- .../Framework/Communications/UserManagerBase.cs | 12 +- OpenSim/Framework/Data.MySQL/MySQLManager.cs | 31 ++-- OpenSim/Framework/Data.MySQL/MySQLUserData.cs | 46 ++---- 4 files changed, 111 insertions(+), 141 deletions(-) (limited to 'OpenSim/Framework') diff --git a/OpenSim/Framework/Communications/LoginService.cs b/OpenSim/Framework/Communications/LoginService.cs index 5547c24..bb8cad5 100644 --- a/OpenSim/Framework/Communications/LoginService.cs +++ b/OpenSim/Framework/Communications/LoginService.cs @@ -114,7 +114,6 @@ namespace OpenSim.Framework.UserManagement string clientversion = (string)requestData["version"]; m_log.Info("[LOGIN]: Client Version " + clientversion + " for " + firstname + " " + lastname); } - userProfile = GetTheUser(firstname, lastname); if (userProfile == null) @@ -398,7 +397,6 @@ namespace OpenSim.Framework.UserManagement public Hashtable ProcessHTMLLogin(Hashtable keysvals) { - // Matches all unspecified characters // Currently specified,; lowercase letters, upper case letters, numbers, underline // period, space, parens, and dash. @@ -422,44 +420,42 @@ namespace OpenSim.Framework.UserManagement // the client requires the HTML form field be named 'username' // however, the data it sends when it loads the first time is 'firstname' // another one of those little nuances. - if (keysvals.Contains("firstname")) - firstname = wfcut.Replace((string)keysvals["firstname"],String.Empty,99999); + firstname = wfcut.Replace((string)keysvals["firstname"], String.Empty, 99999); + if (keysvals.Contains("username")) - firstname = wfcut.Replace((string)keysvals["username"],String.Empty,99999); + firstname = wfcut.Replace((string)keysvals["username"], String.Empty, 99999); if (keysvals.Contains("lastname")) - lastname = wfcut.Replace((string)keysvals["lastname"],String.Empty,99999); + lastname = wfcut.Replace((string)keysvals["lastname"], String.Empty, 99999); if (keysvals.Contains("location")) - location = wfcut.Replace((string)keysvals["location"],String.Empty,99999); + location = wfcut.Replace((string)keysvals["location"], String.Empty, 99999); if (keysvals.Contains("region")) - region = wfcut.Replace((string)keysvals["region"],String.Empty,99999); + region = wfcut.Replace((string)keysvals["region"], String.Empty, 99999); if (keysvals.Contains("grid")) - grid = wfcut.Replace((string)keysvals["grid"],String.Empty,99999); + grid = wfcut.Replace((string)keysvals["grid"], String.Empty, 99999); if (keysvals.Contains("channel")) - channel = wfcut.Replace((string)keysvals["channel"],String.Empty,99999); + channel = wfcut.Replace((string)keysvals["channel"], String.Empty, 99999); if (keysvals.Contains("version")) - version = wfcut.Replace((string)keysvals["version"],String.Empty,99999); + version = wfcut.Replace((string)keysvals["version"], String.Empty, 99999); if (keysvals.Contains("lang")) - lang = wfcut.Replace((string)keysvals["lang"],String.Empty,99999); + lang = wfcut.Replace((string)keysvals["lang"], String.Empty, 99999); if (keysvals.Contains("password")) - password = wfcut.Replace((string)keysvals["password"], String.Empty, 99999); + password = wfcut.Replace((string)keysvals["password"], String.Empty, 99999); - // load our login form. - string loginform = GetLoginForm(firstname,lastname,location,region,grid,channel,version,lang,password,errormessages); + string loginform = GetLoginForm(firstname, lastname, location, region, grid, channel, version, lang, password, errormessages); if (keysvals.ContainsKey("show_login_form")) { - UserProfileData user = GetTheUser(firstname, lastname); bool goodweblogin = false; @@ -472,9 +468,10 @@ namespace OpenSim.Framework.UserManagement m_userManager.StoreWebLoginKey(user.UUID, webloginkey); statuscode = 301; - string redirectURL = "about:blank?redirect-http-hack=" + System.Web.HttpUtility.UrlEncode("secondlife:///app/login?first_name=" + firstname + "&last_name=" + - lastname + - "&location=" + location + "&grid=Other&web_login_key=" + webloginkey.ToString()); + string redirectURL = "about:blank?redirect-http-hack=" + + System.Web.HttpUtility.UrlEncode("secondlife:///app/login?first_name=" + firstname + "&last_name=" + + lastname + + "&location=" + location + "&grid=Other&web_login_key=" + webloginkey.ToString()); //m_log.Info("[WEB]: R:" + redirectURL); returnactions["int_response_code"] = statuscode; returnactions["str_redirect_location"] = redirectURL; @@ -487,11 +484,7 @@ namespace OpenSim.Framework.UserManagement loginform = GetLoginForm(firstname, lastname, location, region, grid, channel, version, lang, password, errormessages); returnactions["int_response_code"] = statuscode; returnactions["str_response_string"] = loginform; - } - - - } else { @@ -499,7 +492,6 @@ namespace OpenSim.Framework.UserManagement returnactions["str_response_string"] = loginform; } return returnactions; - } public string GetLoginForm(string firstname, string lastname, string location, string region, @@ -537,67 +529,66 @@ namespace OpenSim.Framework.UserManagement public string GetDefaultLoginForm() { string responseString = - ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + - ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + "OpenSim Login"; - responseString = responseString + "
"; - responseString = responseString + "
"; + ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += "OpenSim Login"; + responseString += "
"; + responseString += "
"; - responseString = responseString + "
"; - - responseString = responseString + "
[$errors]
"; - responseString = responseString + "
"; - responseString = responseString + "First Name:"; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + "
"; - responseString = responseString + "Last Name:"; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + "
"; - responseString = responseString + "Password:"; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + "
"; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + "
Connecting...
"; - - responseString = responseString + "
"; - responseString = responseString + " | "; - responseString = responseString + ""; - responseString = responseString + "
"; - - responseString = responseString + "
[$channel] | [$version]=[$lang]
"; - responseString = responseString + "
"; - responseString = responseString + ""; - responseString = responseString + "
"; - responseString = responseString + "
"; - responseString = responseString + ""; - responseString = responseString + ""; + responseString += "
"; + + responseString += "
[$errors]
"; + responseString += "
"; + responseString += "First Name:"; + responseString += ""; + responseString += "
"; + responseString += "
"; + responseString += "Last Name:"; + responseString += ""; + responseString += "
"; + responseString += "
"; + responseString += "Password:"; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += "
"; + responseString += "
"; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += "
"; + responseString += ""; + responseString += "
"; + responseString += "
Connecting...
"; + + responseString += "
"; + responseString += " | "; + responseString += ""; + responseString += "
"; + + responseString += "
[$channel] | [$version]=[$lang]
"; + responseString += "
"; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; + responseString += ""; return responseString; } @@ -611,7 +602,6 @@ namespace OpenSim.Framework.UserManagement return m_userManager.CommitAgent(ref profile); } - /// /// Checks a user against it's password hash /// @@ -638,7 +628,7 @@ namespace OpenSim.Framework.UserManagement //m_log.Info("[LOGIN]: userprofile:" + profile.passwordHash + " SubCT:" + password); passwordSuccess = (profile.passwordHash.Equals(s.ToString(), StringComparison.InvariantCultureIgnoreCase) - || profile.passwordHash.Equals(password,StringComparison.InvariantCultureIgnoreCase)); + || profile.passwordHash.Equals(password, StringComparison.InvariantCultureIgnoreCase)); return passwordSuccess; } @@ -699,7 +689,6 @@ namespace OpenSim.Framework.UserManagement buddyitem.BuddyRightsHave = (int)fl.FriendListOwnerPerms; buddyitem.BuddyRightsGiven = (int) fl.FriendPerms; buddylistreturn.AddNewBuddy(buddyitem); - } return buddylistreturn; } diff --git a/OpenSim/Framework/Communications/UserManagerBase.cs b/OpenSim/Framework/Communications/UserManagerBase.cs index b8e3114..48807c2 100644 --- a/OpenSim/Framework/Communications/UserManagerBase.cs +++ b/OpenSim/Framework/Communications/UserManagerBase.cs @@ -194,7 +194,6 @@ namespace OpenSim.Framework.UserManagement /// A List of FriendListItems that contains info about the user's friends public List GetUserFriendList(LLUUID ownerID) { - foreach (KeyValuePair plugin in _plugins) { try @@ -208,12 +207,10 @@ namespace OpenSim.Framework.UserManagement } return null; - } public void StoreWebLoginKey(LLUUID agentID, LLUUID webLoginKey) { - foreach (KeyValuePair plugin in _plugins) { try @@ -240,10 +237,8 @@ namespace OpenSim.Framework.UserManagement m_log.Info("[USERSTORAGE]: Unable to AddNewUserFriend via " + plugin.Key + "(" + e.ToString() + ")"); } } - } - public void RemoveUserFriend(LLUUID friendlistowner, LLUUID friend) { foreach (KeyValuePair plugin in _plugins) @@ -273,6 +268,7 @@ namespace OpenSim.Framework.UserManagement } } } + /// /// Loads a user agent by name (not called directly) /// @@ -295,7 +291,10 @@ namespace OpenSim.Framework.UserManagement return null; } - // TODO: document + /// + /// Resets the currentAgent in the user profile + /// + /// The agent's ID public void clearUserAgent(LLUUID agentID) { UserProfileData profile = GetUserProfile(agentID); @@ -459,7 +458,6 @@ namespace OpenSim.Framework.UserManagement // If currentagent is null, we can't reference it here or the UserServer crashes! m_log.Info("[LOGOUT]: didn't save logout position: " + userid.ToString()); } - } else { diff --git a/OpenSim/Framework/Data.MySQL/MySQLManager.cs b/OpenSim/Framework/Data.MySQL/MySQLManager.cs index 46fd1bc..d422687 100644 --- a/OpenSim/Framework/Data.MySQL/MySQLManager.cs +++ b/OpenSim/Framework/Data.MySQL/MySQLManager.cs @@ -138,13 +138,11 @@ namespace OpenSim.Framework.Data.MySQL string dllName = module.Assembly.ManifestModule.Name; Version dllVersion = module.Assembly.GetName().Version; - return string.Format("{0}.{1}.{2}.{3}", dllVersion.Major, dllVersion.Minor, dllVersion.Build, dllVersion.Revision); } - /// /// Extract a named string resource from the embedded resources /// @@ -156,7 +154,9 @@ namespace OpenSim.Framework.Data.MySQL string[] names = assem.GetManifestResourceNames(); foreach (string s in names) + { if (s.EndsWith(name)) + { using (Stream resource = assem.GetManifestResourceStream(s)) { using (StreamReader resourceReader = new StreamReader(resource)) @@ -165,6 +165,8 @@ namespace OpenSim.Framework.Data.MySQL return resourceString; } } + } + } throw new Exception(string.Format("Resource '{0}' was not found", name)); } @@ -214,8 +216,7 @@ namespace OpenSim.Framework.Data.MySQL } } - - // at some time this code should be cleaned up + // TODO: at some time this code should be cleaned up /// /// Runs a query with protection against SQL Injection by using parameterised input. @@ -249,7 +250,7 @@ namespace OpenSim.Framework.Data.MySQL { } - // Try reopen it + // Try to reopen it try { dbcon = new MySqlConnection(connectionString); @@ -368,7 +369,7 @@ namespace OpenSim.Framework.Data.MySQL retval.reservationMinY = Convert.ToInt32(reader["resYMin"].ToString()); retval.reservationName = (string) reader["resName"]; retval.status = Convert.ToInt32(reader["status"].ToString()) == 1; - LLUUID.TryParse((string) reader["userUUID"], out retval.userUUID); + LLUUID.TryParse((string) reader["userUUID"], out retval.userUUID); } else { @@ -458,22 +459,22 @@ namespace OpenSim.Framework.Data.MySQL else retval.profileAboutText = (string) reader["profileAboutText"]; - if (reader.IsDBNull( reader.GetOrdinal( "profileFirstText" ) ) ) + if (reader.IsDBNull(reader.GetOrdinal("profileFirstText"))) retval.profileFirstText = ""; else retval.profileFirstText = (string)reader["profileFirstText"]; - if (reader.IsDBNull( reader.GetOrdinal( "profileImage" ) ) ) + if (reader.IsDBNull(reader.GetOrdinal("profileImage"))) retval.profileImage = LLUUID.Zero; else LLUUID.TryParse((string)reader["profileImage"], out retval.profileImage); - if (reader.IsDBNull( reader.GetOrdinal( "profileFirstImage" ) ) ) + if (reader.IsDBNull(reader.GetOrdinal("profileFirstImage"))) retval.profileFirstImage = LLUUID.Zero; else LLUUID.TryParse((string)reader["profileFirstImage"], out retval.profileFirstImage); - if( reader.IsDBNull( reader.GetOrdinal( "webLoginKey" ) ) ) + if(reader.IsDBNull(reader.GetOrdinal("webLoginKey"))) { retval.webLoginKey = LLUUID.Zero; } @@ -489,7 +490,6 @@ namespace OpenSim.Framework.Data.MySQL return retval; } - /// /// Inserts a new row into the log database /// @@ -534,7 +534,6 @@ namespace OpenSim.Framework.Data.MySQL return returnval; } - /// /// Creates a new user and inserts it into the database /// @@ -630,7 +629,7 @@ namespace OpenSim.Framework.Data.MySQL return returnval; } - /// + /// /// Creates a new user and inserts it into the database /// /// User ID @@ -823,6 +822,7 @@ namespace OpenSim.Framework.Data.MySQL return returnval; } + /// /// Delete a region from the database /// @@ -833,14 +833,13 @@ namespace OpenSim.Framework.Data.MySQL { bool returnval = false; - string sql = - "DELETE FROM regions WHERE uuid = ?uuid;"; + string sql = "DELETE FROM regions WHERE uuid = ?uuid;"; Dictionary parameters = new Dictionary(); try { - parameters["?uuid"] = uuid; + parameters["?uuid"] = uuid; IDbCommand result = Query(sql, parameters); diff --git a/OpenSim/Framework/Data.MySQL/MySQLUserData.cs b/OpenSim/Framework/Data.MySQL/MySQLUserData.cs index 4c76461..1370b1b 100644 --- a/OpenSim/Framework/Data.MySQL/MySQLUserData.cs +++ b/OpenSim/Framework/Data.MySQL/MySQLUserData.cs @@ -108,7 +108,6 @@ namespace OpenSim.Framework.Data.MySQL UpgradeAgentsTable(tableList[m_agentsTableName]); UpgradeUsersTable(tableList[m_usersTableName]); UpgradeFriendsTable(tableList[m_userFriendsTableName]); - } /// @@ -227,7 +226,6 @@ namespace OpenSim.Framework.Data.MySQL "(?friendID,?ownerID,?friendPerms,?datetimestamp)", param); adder.ExecuteNonQuery(); - } } catch (Exception e) @@ -244,23 +242,21 @@ namespace OpenSim.Framework.Data.MySQL param["?ownerID"] = friendlistowner.UUID.ToString(); param["?friendID"] = friend.UUID.ToString(); - try { lock (database) { IDbCommand updater = database.Query( - "delete from " + m_userFriendsTableName + " where ownerID = ?ownerID and friendID = ?friendID", + "delete from " + m_userFriendsTableName + " where ownerID = ?ownerID and friendID = ?friendID", param); updater.ExecuteNonQuery(); updater = database.Query( - "delete from " + m_userFriendsTableName + " where ownerID = ?friendID and friendID = ?ownerID", + "delete from " + m_userFriendsTableName + " where ownerID = ?friendID and friendID = ?ownerID", param); updater.ExecuteNonQuery(); - } } catch (Exception e) @@ -270,13 +266,13 @@ namespace OpenSim.Framework.Data.MySQL return; } } + public void UpdateUserFriendPerms(LLUUID friendlistowner, LLUUID friend, uint perms) { Dictionary param = new Dictionary(); param["?ownerID"] = friendlistowner.UUID.ToString(); param["?friendID"] = friend.UUID.ToString(); param["?friendPerms"] = perms.ToString(); - try { @@ -284,12 +280,11 @@ namespace OpenSim.Framework.Data.MySQL { IDbCommand updater = database.Query( - "update " + m_userFriendsTableName + - " SET friendPerms = ?friendPerms " + - "where ownerID = ?ownerID and friendID = ?friendID", + "update " + m_userFriendsTableName + + " SET friendPerms = ?friendPerms " + + "where ownerID = ?ownerID and friendID = ?friendID", param); updater.ExecuteNonQuery(); - } } catch (Exception e) @@ -300,7 +295,6 @@ namespace OpenSim.Framework.Data.MySQL } } - public List GetUserFriendList(LLUUID friendlistowner) { List Lfli = new List(); @@ -315,12 +309,11 @@ namespace OpenSim.Framework.Data.MySQL //Left Join userfriends to itself IDbCommand result = database.Query( - "select a.ownerID,a.friendID,a.friendPerms,b.friendPerms as ownerperms from " + m_userFriendsTableName + " as a, " + m_userFriendsTableName + " as b" + - " where a.ownerID = ?ownerID and b.ownerID = a.friendID and b.friendID = a.ownerID", + "select a.ownerID,a.friendID,a.friendPerms,b.friendPerms as ownerperms from " + m_userFriendsTableName + " as a, " + m_userFriendsTableName + " as b" + + " where a.ownerID = ?ownerID and b.ownerID = a.friendID and b.friendID = a.ownerID", param); IDataReader reader = result.ExecuteReader(); - while (reader.Read()) { FriendListItem fli = new FriendListItem(); @@ -354,7 +347,6 @@ namespace OpenSim.Framework.Data.MySQL m_log.Info("[USER]: Stub UpdateUserCUrrentRegion called"); } - public List GeneratePickerResults(LLUUID queryID, string query) { List returnlist = new List(); @@ -378,7 +370,6 @@ namespace OpenSim.Framework.Data.MySQL param); IDataReader reader = result.ExecuteReader(); - while (reader.Read()) { Framework.AvatarPickerAvatar user = new Framework.AvatarPickerAvatar(); @@ -413,7 +404,6 @@ namespace OpenSim.Framework.Data.MySQL param); IDataReader reader = result.ExecuteReader(); - while (reader.Read()) { Framework.AvatarPickerAvatar user = new Framework.AvatarPickerAvatar(); @@ -489,7 +479,6 @@ namespace OpenSim.Framework.Data.MySQL public void StoreWebLoginKey(LLUUID AgentID, LLUUID WebLoginKey) { - Dictionary param = new Dictionary(); param["?UUID"] = AgentID.UUID.ToString(); param["?webLoginKey"] = WebLoginKey.UUID.ToString(); @@ -500,11 +489,10 @@ namespace OpenSim.Framework.Data.MySQL { IDbCommand updater = database.Query( - "update " + m_usersTableName + " SET webLoginKey = ?webLoginKey " + - "where UUID = ?UUID", + "update " + m_usersTableName + " SET webLoginKey = ?webLoginKey " + + "where UUID = ?UUID", param); updater.ExecuteNonQuery(); - } } catch (Exception e) @@ -513,10 +501,6 @@ namespace OpenSim.Framework.Data.MySQL m_log.Error(e.ToString()); return; } - - - - } /// @@ -605,11 +589,11 @@ namespace OpenSim.Framework.Data.MySQL /// The profile data to use to update the DB public bool UpdateUserProfile(UserProfileData user) { - database.updateUserRow(user.UUID, user.username, user.surname, user.passwordHash, user.passwordSalt - , user.homeRegion, user.homeLocation.X, user.homeLocation.Y, user.homeLocation.Z, user.homeLookAt.X - , user.homeLookAt.Y, user.homeLookAt.Z, user.created, user.lastLogin, user.userInventoryURI - , user.userAssetURI, user.profileCanDoMask, user.profileWantDoMask, user.profileAboutText - , user.profileFirstText, user.profileImage, user.profileFirstImage, user.webLoginKey); + database.updateUserRow(user.UUID, user.username, user.surname, user.passwordHash, user.passwordSalt, + user.homeRegion, user.homeLocation.X, user.homeLocation.Y, user.homeLocation.Z, user.homeLookAt.X, + user.homeLookAt.Y, user.homeLookAt.Z, user.created, user.lastLogin, user.userInventoryURI, + user.userAssetURI, user.profileCanDoMask, user.profileWantDoMask, user.profileAboutText, + user.profileFirstText, user.profileImage, user.profileFirstImage, user.webLoginKey); return true; } -- cgit v1.1