From b3c42e952f24a5c280a3691e1f75912e21c77323 Mon Sep 17 00:00:00 2001
From: Sean McNamara
Date: Mon, 28 Feb 2011 11:54:07 -0500
Subject: Fix small bug with remove region; update settings docs.

Our impl of IRegionModuleBase.RemoveRegion didn't remove the scene from the states map.
---
 .../OptionalModules/World/AutoBackup/AutoBackupModule.cs   | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

(limited to 'OpenSim/Region')

diff --git a/OpenSim/Region/OptionalModules/World/AutoBackup/AutoBackupModule.cs b/OpenSim/Region/OptionalModules/World/AutoBackup/AutoBackupModule.cs
index 98127b7..7660342 100644
--- a/OpenSim/Region/OptionalModules/World/AutoBackup/AutoBackupModule.cs
+++ b/OpenSim/Region/OptionalModules/World/AutoBackup/AutoBackupModule.cs
@@ -71,6 +71,10 @@ using OpenSim.Region.Framework.Scenes;
  *  "Overwrite": Always save to file named "${AutoBackupDir}/RegionName.oar", even if we have to overwrite an existing file.
  * AutoBackupDir: String. Default: "." (the current directory).
  * 	A directory (absolute or relative) where backups should be saved.
+ * AutoBackupDilationThreshold: float. Default: 0.5. Lower bound on time dilation required for BusyCheck heuristics to pass.
+ *  If the time dilation is below this value, don't take a backup right now.
+ * AutoBackupAgentThreshold: int. Default: 10. Upper bound on # of agents in region required for BusyCheck heuristics to pass.
+ *  If the number of agents is greater than this value, don't take a backup right now.
  * */
 
 namespace OpenSim.Region.OptionalModules.World.AutoBackup
@@ -91,7 +95,6 @@ namespace OpenSim.Region.OptionalModules.World.AutoBackup
 		//AutoBackupModuleState: Auto-Backup state for one region (scene).
 		public class AutoBackupModuleState
 		{
-			private readonly IScene m_scene;
 			private bool m_enabled = false;
 			private NamingType m_naming = NamingType.TIME;
 			private Timer m_timer = null;
@@ -99,11 +102,9 @@ namespace OpenSim.Region.OptionalModules.World.AutoBackup
 			private string m_script = null;
 			private string m_dir = ".";
 
-			public AutoBackupModuleState (IScene scene)
+			public AutoBackupModuleState ()
 			{
-				m_scene = scene;
-				if (scene == null)
-					throw new NullReferenceException ("Required parameter missing for AutoBackupModuleState constructor");
+
 			}
 
 			public void SetEnabled (bool b)
@@ -226,6 +227,7 @@ namespace OpenSim.Region.OptionalModules.World.AutoBackup
 				timers.Remove (timer.Interval);
 				timer.Close ();
 			}
+			states.Remove(scene);
 		}
 
 		void IRegionModuleBase.RegionLoaded (Framework.Scenes.Scene scene)
@@ -238,7 +240,7 @@ namespace OpenSim.Region.OptionalModules.World.AutoBackup
 				return;
 			
 			string sRegionName = scene.RegionInfo.RegionName;
-			AutoBackupModuleState st = new AutoBackupModuleState (scene);
+			AutoBackupModuleState st = new AutoBackupModuleState ();
 			states.Add (scene, st);
 			
 			//Read the config settings and set variables.
-- 
cgit v1.1