From 5725fb5984249cb42beed50b17fae0eae359bbd3 Mon Sep 17 00:00:00 2001 From: Charles Krinke Date: Sat, 23 Aug 2008 20:31:32 +0000 Subject: [PATCH] Mantis#2036. Thank you kindly, HomerHorwitz for a patch that: This patch adds some more informative logging if the region isn't found. --- OpenSim/Grid/GridServer/GridManager.cs | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/OpenSim/Grid/GridServer/GridManager.cs b/OpenSim/Grid/GridServer/GridManager.cs index 98abd53388..7c9e77621d 100644 --- a/OpenSim/Grid/GridServer/GridManager.cs +++ b/OpenSim/Grid/GridServer/GridManager.cs @@ -716,23 +716,29 @@ namespace OpenSim.Grid.GridServer RegionProfileData simData = null; if (requestData.ContainsKey("region_UUID")) { - simData = GetRegion(new LLUUID((string)requestData["region_UUID"])); + LLUUID regionID = new LLUUID((string)requestData["region_UUID"]); + simData = GetRegion(regionID); + if (simData == null) m_log.WarnFormat("[DATA] didn't find region for regionID {0}", regionID); } else if (requestData.ContainsKey("region_handle")) { //CFK: The if/else below this makes this message redundant. //CFK: Console.WriteLine("requesting data for region " + (string) requestData["region_handle"]); - simData = GetRegion(Convert.ToUInt64((string)requestData["region_handle"])); + ulong regionHandle = Convert.ToUInt64((string)requestData["region_handle"]); + simData = GetRegion(regionHandle); + if (simData == null) m_log.WarnFormat("[DATA] didn't find region for regionHandle {0}", regionHandle); } else if (requestData.ContainsKey("region_name_search")) { - simData = GetRegion((string)requestData["region_name_search"]); + string regionName = (string)requestData["region_name_search"]; + simData = GetRegion(regionName); + if (simData == null) m_log.WarnFormat("[DATA] didn't find region for regionName {0}", regionName); } + else m_log.Warn("[DATA] regionlookup without regionID, regionHandle or regionHame"); if (simData == null) { //Sim does not exist - Console.WriteLine("region not found"); responseData["error"] = "Sim does not exist"; } else