From 9324c3f110d70d44ef91c18e570cffc59d067683 Mon Sep 17 00:00:00 2001
From: Justin Clarke Casey
Date: Wed, 15 Oct 2008 16:35:27 +0000
Subject: * refactor: Move error logging from GetUserDetails up to callers,
since there are some circumstances in which not finding a user is not an
error
---
.../Communications/Cache/UserProfileCacheService.cs | 6 +++---
OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs | 8 +++++++-
.../Modules/Agent/AssetTransaction/AssetXferUploader.cs | 12 ++++++++++++
.../Avatar/Currency/SampleMoney/SampleMoneyModule.cs | 7 +++++++
.../Modules/World/Permissions/PermissionsModule.cs | 17 +++++++++++++++++
OpenSim/Region/Environment/Scenes/Scene.Inventory.cs | 13 +++++++++++++
6 files changed, 59 insertions(+), 4 deletions(-)
diff --git a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
index cf6a74d..45102d8 100644
--- a/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
+++ b/OpenSim/Framework/Communications/Cache/UserProfileCacheService.cs
@@ -75,6 +75,7 @@ namespace OpenSim.Framework.Communications.Cache
{
if (userID == UUID.Zero)
return;
+
m_log.DebugFormat("[USER CACHE]: Adding user profile for {0}", userID);
GetUserDetails(userID);
}
@@ -133,8 +134,8 @@ namespace OpenSim.Framework.Communications.Cache
}
///
- /// Get the details of the given user. A caller should try this method first if it isn't sure that
- /// a user profile exists for the given user.
+ /// Get cached details of the given user. If the user isn't in cache then the user is requested from the
+ /// profile service.
///
///
/// null if no user details are found
@@ -160,7 +161,6 @@ namespace OpenSim.Framework.Communications.Cache
}
else
{
- m_log.ErrorFormat("[USER CACHE]: User profile for user {0} not found.", userID);
return null;
}
}
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index cb162ab..b826a12 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -5044,8 +5044,14 @@ namespace OpenSim.Region.ClientStack.LindenUDP
{
CachedUserInfo userInfo = ((Scene)m_scene).CommsManager.UserProfileCacheService.GetUserDetails(AgentId);
if (userInfo == null)
- break;
+ {
+ m_log.ErrorFormat(
+ "[CLIENT]: Could not resolve user {0} for caps inventory update",
+ AgentId);
+ break;
+ }
+
if (userInfo.RootFolder == null)
break;
diff --git a/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AssetXferUploader.cs b/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AssetXferUploader.cs
index 5484ad5..2763831 100644
--- a/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AssetXferUploader.cs
+++ b/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AssetXferUploader.cs
@@ -244,6 +244,12 @@ namespace OpenSim.Region.Environment.Modules.Agent.AssetTransaction
userInfo.UpdateItem(item);
}
+ else
+ {
+ m_log.ErrorFormat(
+ "[ASSET TRANSACTIONS]: Could not find user {0} for inventory item update",
+ remoteClient.AgentId);
+ }
}
}
@@ -294,6 +300,12 @@ namespace OpenSim.Region.Environment.Modules.Agent.AssetTransaction
userInfo.AddItem(item);
ourClient.SendInventoryItemCreateUpdate(item);
}
+ else
+ {
+ m_log.ErrorFormat(
+ "[ASSET TRANSACTIONS]: Could not find user {0} for inventory item creation",
+ ourClient.AgentId);
+ }
}
///
diff --git a/OpenSim/Region/Environment/Modules/Avatar/Currency/SampleMoney/SampleMoneyModule.cs b/OpenSim/Region/Environment/Modules/Avatar/Currency/SampleMoney/SampleMoneyModule.cs
index a015c63..1f57fb5 100644
--- a/OpenSim/Region/Environment/Modules/Avatar/Currency/SampleMoney/SampleMoneyModule.cs
+++ b/OpenSim/Region/Environment/Modules/Avatar/Currency/SampleMoney/SampleMoneyModule.cs
@@ -604,6 +604,13 @@ namespace OpenSim.Region.Environment.Modules.Avatar.Currency.SampleMoney
string avatarname = profile.UserProfile.FirstName + " " + profile.UserProfile.SurName;
return avatarname;
}
+ else
+ {
+ m_log.ErrorFormat(
+ "[MONEY]: Could not resolve user {0}",
+ agentID);
+ }
+
return String.Empty;
}
diff --git a/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs b/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
index abbf40e..b6a0c23 100644
--- a/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
+++ b/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
@@ -259,6 +259,10 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
if (profile.UserProfile.GodLevel >= 200)
return true;
}
+ else
+ {
+ m_log.ErrorFormat("[PERMISSIONS]: Could not find user {0} for administrator check", user);
+ }
}
return false;
@@ -646,8 +650,13 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
{
CachedUserInfo userInfo =
scene.CommsManager.UserProfileCacheService.GetUserDetails(user);
+
if (userInfo == null)
+ {
+ m_log.ErrorFormat("[PERMISSIONS]: Could not find user {0} for edit notecard check", user);
return false;
+ }
+
if (userInfo.RootFolder == null)
return false;
@@ -1024,8 +1033,12 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
{
CachedUserInfo userInfo =
scene.CommsManager.UserProfileCacheService.GetUserDetails(user);
+
if (userInfo == null)
+ {
+ m_log.ErrorFormat("[PERMISSIONS]: Could not find user {0} for administrator check", user);
return false;
+ }
if (userInfo.RootFolder == null)
return false;
@@ -1102,8 +1115,12 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
{
CachedUserInfo userInfo =
scene.CommsManager.UserProfileCacheService.GetUserDetails(user);
+
if (userInfo == null)
+ {
+ m_log.ErrorFormat("[PERMISSIONS]: Could not find user {0} for view notecard check", user);
return false;
+ }
if (userInfo.RootFolder == null)
return false;
diff --git a/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs b/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
index 55cc60a..226f39a 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
@@ -143,6 +143,12 @@ namespace OpenSim.Region.Environment.Scenes
AddInventoryItem(remoteClient.AgentId, item);
remoteClient.SendInventoryItemCreateUpdate(item);
}
+ else
+ {
+ m_log.ErrorFormat(
+ "[AGENT INVENTORY]: Could not resolve user {0} for adding an inventory item",
+ remoteClient.AgentId);
+ }
}
///
@@ -194,6 +200,13 @@ namespace OpenSim.Region.Environment.Scenes
}
}
}
+ else
+ {
+ m_log.ErrorFormat(
+ "[AGENT INVENTORY]: Could not resolve user {0} for caps inventory update",
+ remoteClient.AgentId);
+ }
+
return UUID.Zero;
}
--
cgit v1.1