* minor: slightly pointlessly increase the verbosity of the message given when a queued deletion from scene to inventory fails because the root part of the group has been zapped

1 files changed, 9 insertions, 3 deletions

diff --git a/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs b/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
index d0beac3..23dd8b6 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
@@ -1563,7 +1563,8 @@ namespace OpenSim.Region.Environment.Scenes
 
         private bool InventoryDeQueueAndDelete()
         {
-            DeleteToInventoryHolder x;
+            DeleteToInventoryHolder x = null;
+            
             try
             {
                 lock (m_inventoryDeletes)
@@ -1577,9 +1578,14 @@ namespace OpenSim.Region.Environment.Scenes
                         return true;
                     }
                 }
-            } catch(Exception e)
+            } 
+            catch(Exception e)
             {
-                m_log.Error(e.ToString());
+                // We can't put the object group details in here since the root part may have disappeared (which is where these sit).
+                // FIXME: This needs to be fixed.
+                m_log.ErrorFormat(
+                    "[AGENT INVENTORY]: Queued deletion of scene object to agent {0} {1} failed: {2}", 
+                    (x != null ? x.remoteClient.Name : "unavailable"), (x != null ? x.remoteClient.AgentId : "unavailable"), e.ToString());
             }
 
             m_log.Info("No objects left in inventory delete queue.");