* Put an Enabled switch on the async object deleter for future testing purposes

3 files changed, 21 insertions, 3 deletions

diff --git a/OpenSim/Region/Environment/Scenes/AsyncSceneObjectGroupDeleter.cs b/OpenSim/Region/Environment/Scenes/AsyncSceneObjectGroupDeleter.cs
index ca28bbd..706ba6a 100644
--- a/OpenSim/Region/Environment/Scenes/AsyncSceneObjectGroupDeleter.cs
+++ b/OpenSim/Region/Environment/Scenes/AsyncSceneObjectGroupDeleter.cs
@@ -54,9 +54,14 @@ namespace OpenSim.Region.Environment.Scenes
         private static readonly ILog m_log
             = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
         
+        /// <value>
+        /// Is the deleter currently enabled?
+        /// </value>
+        public bool Enabled;
+        
         private Timer m_inventoryTicker = new Timer(2000);       
         private readonly Queue<DeleteToInventoryHolder> m_inventoryDeletes = new Queue<DeleteToInventoryHolder>();        
-        private Scene m_scene;
+        private Scene m_scene;        
         
         public AsyncSceneObjectGroupDeleter(Scene scene)
         {
@@ -73,7 +78,8 @@ namespace OpenSim.Region.Environment.Scenes
                 SceneObjectGroup objectGroup, IClientAPI remoteClient, 
                 bool permissionToDelete)
         {
-            m_inventoryTicker.Stop();
+            if (Enabled)
+                m_inventoryTicker.Stop();
 
             lock (m_inventoryDeletes)
             {
@@ -87,7 +93,8 @@ namespace OpenSim.Region.Environment.Scenes
                 m_inventoryDeletes.Enqueue(dtis);
             }
 
-            m_inventoryTicker.Start();
+            if (Enabled)
+                m_inventoryTicker.Start();
         
             // Visually remove it, even if it isnt really gone yet.
             if (permissionToDelete)
@@ -132,6 +139,7 @@ namespace OpenSim.Region.Environment.Scenes
                                 m_log.DebugFormat("Exception background deleting object: "+e.ToString());
                             }
                         }
+                        
                         return true;
                     }
                 }
diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index fd4eb58..326380b 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -281,7 +281,9 @@ namespace OpenSim.Region.Environment.Scenes
 
             m_eventManager = new EventManager();
             m_externalChecks = new SceneExternalChecks(this);
+            
             m_asyncSceneObjectDeleter = new AsyncSceneObjectGroupDeleter(this);
+            m_asyncSceneObjectDeleter.Enabled = true;
 
             // Load region settings
             m_regInfo.RegionSettings = m_storageManager.DataStore.LoadRegionSettings(m_regInfo.RegionID);
diff --git a/OpenSim/Region/Environment/Scenes/Tests/SceneTests.cs b/OpenSim/Region/Environment/Scenes/Tests/SceneTests.cs
index 35b633e..b603e30 100644
--- a/OpenSim/Region/Environment/Scenes/Tests/SceneTests.cs
+++ b/OpenSim/Region/Environment/Scenes/Tests/SceneTests.cs
@@ -106,5 +106,13 @@ namespace OpenSim.Region.Environment.Scenes.Tests
             // If the parts have the same UUID then we will consider them as one and the same
             Assert.That(retrievedPart.UUID, Is.EqualTo(part.UUID));         
         }
+        
+        /*
+        public void TestRemoveSceneObject()
+        {
+            Scene scene = SetupScene();
+            SceneObjectPart part = AddSceneObject(scene);            
+        }
+        */
     }
 }
\ No newline at end of file