diff options
42 files changed, 85 insertions, 572 deletions
diff --git a/OpenSim/ApplicationPlugins/LoadRegions/LoadRegionsPlugin.cs b/OpenSim/ApplicationPlugins/LoadRegions/LoadRegionsPlugin.cs index a8a3237..fcb6991 100644 --- a/OpenSim/ApplicationPlugins/LoadRegions/LoadRegionsPlugin.cs +++ b/OpenSim/ApplicationPlugins/LoadRegions/LoadRegionsPlugin.cs | |||
@@ -136,9 +136,6 @@ namespace OpenSim.ApplicationPlugins.LoadRegions | |||
136 | } | 136 | } |
137 | } | 137 | } |
138 | } | 138 | } |
139 | |||
140 | m_openSim.ModuleLoader.PostInitialise(); | ||
141 | m_openSim.ModuleLoader.ClearCache(); | ||
142 | } | 139 | } |
143 | 140 | ||
144 | public void Dispose() | 141 | public void Dispose() |
diff --git a/OpenSim/ApplicationPlugins/RegionModulesController/RegionModulesControllerPlugin.cs b/OpenSim/ApplicationPlugins/RegionModulesController/RegionModulesControllerPlugin.cs index 49bd911..633d005 100644 --- a/OpenSim/ApplicationPlugins/RegionModulesController/RegionModulesControllerPlugin.cs +++ b/OpenSim/ApplicationPlugins/RegionModulesController/RegionModulesControllerPlugin.cs | |||
@@ -215,7 +215,7 @@ namespace OpenSim.ApplicationPlugins.RegionModulesController | |||
215 | } | 215 | } |
216 | } | 216 | } |
217 | 217 | ||
218 | #region IRegionModulesController implementation | 218 | #region Region Module interfacesController implementation |
219 | 219 | ||
220 | /// <summary> | 220 | /// <summary> |
221 | /// Check that the given module is no disabled in the [Modules] section of the config files. | 221 | /// Check that the given module is no disabled in the [Modules] section of the config files. |
diff --git a/OpenSim/Region/Application/OpenSim.cs b/OpenSim/Region/Application/OpenSim.cs index 5f07272..4c4abb6 100644 --- a/OpenSim/Region/Application/OpenSim.cs +++ b/OpenSim/Region/Application/OpenSim.cs | |||
@@ -757,33 +757,13 @@ namespace OpenSim | |||
757 | switch (cmdparams[0].ToLower()) | 757 | switch (cmdparams[0].ToLower()) |
758 | { | 758 | { |
759 | case "list": | 759 | case "list": |
760 | foreach (IRegionModule irm in m_moduleLoader.GetLoadedSharedModules) | 760 | //TODO: Convert to new region modules |
761 | { | ||
762 | MainConsole.Instance.Output(String.Format("Shared region module: {0}", irm.Name)); | ||
763 | } | ||
764 | break; | 761 | break; |
765 | case "unload": | 762 | case "unload": |
766 | if (cmdparams.Length > 1) | 763 | //TODO: Convert to new region modules |
767 | { | ||
768 | foreach (IRegionModule rm in new ArrayList(m_moduleLoader.GetLoadedSharedModules)) | ||
769 | { | ||
770 | if (rm.Name.ToLower() == cmdparams[1].ToLower()) | ||
771 | { | ||
772 | MainConsole.Instance.Output(String.Format("Unloading module: {0}", rm.Name)); | ||
773 | m_moduleLoader.UnloadModule(rm); | ||
774 | } | ||
775 | } | ||
776 | } | ||
777 | break; | 764 | break; |
778 | case "load": | 765 | case "load": |
779 | if (cmdparams.Length > 1) | 766 | //TODO: Convert to new region modules |
780 | { | ||
781 | foreach (Scene s in new ArrayList(SceneManager.Scenes)) | ||
782 | { | ||
783 | MainConsole.Instance.Output(String.Format("Loading module: {0}", cmdparams[1])); | ||
784 | m_moduleLoader.LoadRegionModules(cmdparams[1], s); | ||
785 | } | ||
786 | } | ||
787 | break; | 767 | break; |
788 | } | 768 | } |
789 | } | 769 | } |
@@ -1018,28 +998,9 @@ namespace OpenSim | |||
1018 | break; | 998 | break; |
1019 | 999 | ||
1020 | case "modules": | 1000 | case "modules": |
1021 | MainConsole.Instance.Output("The currently loaded shared modules are:"); | ||
1022 | foreach (IRegionModule module in m_moduleLoader.GetLoadedSharedModules) | ||
1023 | { | ||
1024 | MainConsole.Instance.Output("Shared Module: " + module.Name); | ||
1025 | } | ||
1026 | |||
1027 | SceneManager.ForEachScene( | ||
1028 | delegate(Scene scene) { | ||
1029 | m_log.Error("The currently loaded modules in " + scene.RegionInfo.RegionName + " are:"); | ||
1030 | foreach (IRegionModule module in scene.Modules.Values) | ||
1031 | { | ||
1032 | if (!module.IsSharedModule) | ||
1033 | { | ||
1034 | m_log.Error("Region Module: " + module.Name); | ||
1035 | } | ||
1036 | } | ||
1037 | } | ||
1038 | ); | ||
1039 | |||
1040 | SceneManager.ForEachScene( | 1001 | SceneManager.ForEachScene( |
1041 | delegate(Scene scene) { | 1002 | delegate(Scene scene) { |
1042 | MainConsole.Instance.Output("Loaded new region modules in" + scene.RegionInfo.RegionName + " are:"); | 1003 | MainConsole.Instance.Output("Loaded region modules in" + scene.RegionInfo.RegionName + " are:"); |
1043 | foreach (IRegionModuleBase module in scene.RegionModules.Values) | 1004 | foreach (IRegionModuleBase module in scene.RegionModules.Values) |
1044 | { | 1005 | { |
1045 | Type type = module.GetType().GetInterface("ISharedRegionModule"); | 1006 | Type type = module.GetType().GetInterface("ISharedRegionModule"); |
diff --git a/OpenSim/Region/Application/OpenSimBase.cs b/OpenSim/Region/Application/OpenSimBase.cs index 7232383..1c28219 100644 --- a/OpenSim/Region/Application/OpenSimBase.cs +++ b/OpenSim/Region/Application/OpenSimBase.cs | |||
@@ -131,14 +131,6 @@ namespace OpenSim | |||
131 | get { return m_httpServerPort; } | 131 | get { return m_httpServerPort; } |
132 | } | 132 | } |
133 | 133 | ||
134 | public ModuleLoader ModuleLoader | ||
135 | { | ||
136 | get { return m_moduleLoader; } | ||
137 | set { m_moduleLoader = value; } | ||
138 | } | ||
139 | |||
140 | protected ModuleLoader m_moduleLoader; | ||
141 | |||
142 | protected IRegistryCore m_applicationRegistry = new RegistryCore(); | 134 | protected IRegistryCore m_applicationRegistry = new RegistryCore(); |
143 | 135 | ||
144 | public IRegistryCore ApplicationRegistry | 136 | public IRegistryCore ApplicationRegistry |
@@ -232,9 +224,6 @@ namespace OpenSim | |||
232 | 224 | ||
233 | base.StartupSpecific(); | 225 | base.StartupSpecific(); |
234 | 226 | ||
235 | // Create a ModuleLoader instance | ||
236 | m_moduleLoader = new ModuleLoader(m_config.Source); | ||
237 | |||
238 | LoadPlugins(); | 227 | LoadPlugins(); |
239 | 228 | ||
240 | if (m_plugins.Count == 0) // We failed to load any modules. Mono Addins glitch! | 229 | if (m_plugins.Count == 0) // We failed to load any modules. Mono Addins glitch! |
@@ -385,12 +374,6 @@ namespace OpenSim | |||
385 | 374 | ||
386 | m_log.Info("[MODULES]: Loading Region's modules (old style)"); | 375 | m_log.Info("[MODULES]: Loading Region's modules (old style)"); |
387 | 376 | ||
388 | List<IRegionModule> modules = m_moduleLoader.PickupModules(scene, "."); | ||
389 | |||
390 | // This needs to be ahead of the script engine load, so the | ||
391 | // script module can pick up events exposed by a module | ||
392 | m_moduleLoader.InitialiseSharedModules(scene); | ||
393 | |||
394 | // Use this in the future, the line above will be deprecated soon | 377 | // Use this in the future, the line above will be deprecated soon |
395 | m_log.Info("[REGIONMODULES]: Loading Region's modules (new style)"); | 378 | m_log.Info("[REGIONMODULES]: Loading Region's modules (new style)"); |
396 | IRegionModulesController controller; | 379 | IRegionModulesController controller; |
@@ -400,28 +383,29 @@ namespace OpenSim | |||
400 | } | 383 | } |
401 | else m_log.Error("[REGIONMODULES]: The new RegionModulesController is missing..."); | 384 | else m_log.Error("[REGIONMODULES]: The new RegionModulesController is missing..."); |
402 | 385 | ||
403 | if (m_securePermissionsLoading) | 386 | // XPTO: Fix this |
404 | { | 387 | // if (m_securePermissionsLoading) |
405 | foreach (string s in m_permsModules) | 388 | // { |
406 | { | 389 | // foreach (string s in m_permsModules) |
407 | if (!scene.RegionModules.ContainsKey(s)) | 390 | // { |
408 | { | 391 | // if (!scene.RegionModules.ContainsKey(s)) |
409 | bool found = false; | 392 | // { |
410 | foreach (IRegionModule m in modules) | 393 | // bool found = false; |
411 | { | 394 | // foreach (IRegionModule m in modules) |
412 | if (m.Name == s) | 395 | // { |
413 | { | 396 | // if (m.Name == s) |
414 | found = true; | 397 | // { |
415 | } | 398 | // found = true; |
416 | } | 399 | // } |
417 | if (!found) | 400 | // } |
418 | { | 401 | // if (!found) |
419 | m_log.Fatal("[MODULES]: Required module " + s + " not found."); | 402 | // { |
420 | Environment.Exit(0); | 403 | // m_log.Fatal("[MODULES]: Required module " + s + " not found."); |
421 | } | 404 | // Environment.Exit(0); |
422 | } | 405 | // } |
423 | } | 406 | // } |
424 | } | 407 | // } |
408 | // } | ||
425 | 409 | ||
426 | scene.SetModuleInterfaces(); | 410 | scene.SetModuleInterfaces(); |
427 | // First Step of bootreport sequence | 411 | // First Step of bootreport sequence |
@@ -498,13 +482,6 @@ namespace OpenSim | |||
498 | { | 482 | { |
499 | scene.SnmpService.BootInfo("Initializing region modules", scene); | 483 | scene.SnmpService.BootInfo("Initializing region modules", scene); |
500 | } | 484 | } |
501 | if (do_post_init) | ||
502 | { | ||
503 | foreach (IRegionModule module in modules) | ||
504 | { | ||
505 | module.PostInitialise(); | ||
506 | } | ||
507 | } | ||
508 | scene.EventManager.OnShutdown += delegate() { ShutdownRegion(scene); }; | 485 | scene.EventManager.OnShutdown += delegate() { ShutdownRegion(scene); }; |
509 | 486 | ||
510 | mscene = scene; | 487 | mscene = scene; |
@@ -805,7 +782,7 @@ namespace OpenSim | |||
805 | 782 | ||
806 | return new Scene( | 783 | return new Scene( |
807 | regionInfo, circuitManager, sceneGridService, | 784 | regionInfo, circuitManager, sceneGridService, |
808 | simDataService, estateDataService, m_moduleLoader, false, | 785 | simDataService, estateDataService, false, |
809 | m_config.Source, m_version); | 786 | m_config.Source, m_version); |
810 | } | 787 | } |
811 | 788 | ||
diff --git a/OpenSim/Region/ClientStack/Linden/Caps/GetMeshModule.cs b/OpenSim/Region/ClientStack/Linden/Caps/GetMeshModule.cs index 0d7b1fc..9bb0f5e 100644 --- a/OpenSim/Region/ClientStack/Linden/Caps/GetMeshModule.cs +++ b/OpenSim/Region/ClientStack/Linden/Caps/GetMeshModule.cs | |||
@@ -58,7 +58,7 @@ namespace OpenSim.Region.ClientStack.Linden | |||
58 | private bool m_Enabled = true; | 58 | private bool m_Enabled = true; |
59 | private string m_URL; | 59 | private string m_URL; |
60 | 60 | ||
61 | #region IRegionModuleBase Members | 61 | #region Region Module interfaceBase Members |
62 | 62 | ||
63 | public Type ReplaceableInterface | 63 | public Type ReplaceableInterface |
64 | { | 64 | { |
diff --git a/OpenSim/Region/ClientStack/Linden/Caps/ObjectCaps/UploadObjectAssetModule.cs b/OpenSim/Region/ClientStack/Linden/Caps/ObjectCaps/UploadObjectAssetModule.cs index ba902b2..725b4f1 100644 --- a/OpenSim/Region/ClientStack/Linden/Caps/ObjectCaps/UploadObjectAssetModule.cs +++ b/OpenSim/Region/ClientStack/Linden/Caps/ObjectCaps/UploadObjectAssetModule.cs | |||
@@ -58,7 +58,7 @@ namespace OpenSim.Region.ClientStack.Linden | |||
58 | LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | 58 | LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); |
59 | private Scene m_scene; | 59 | private Scene m_scene; |
60 | 60 | ||
61 | #region IRegionModuleBase Members | 61 | #region Region Module interfaceBase Members |
62 | 62 | ||
63 | 63 | ||
64 | public Type ReplaceableInterface | 64 | public Type ReplaceableInterface |
@@ -92,7 +92,7 @@ namespace OpenSim.Region.ClientStack.Linden | |||
92 | #endregion | 92 | #endregion |
93 | 93 | ||
94 | 94 | ||
95 | #region IRegionModule Members | 95 | #region Region Module interface |
96 | 96 | ||
97 | 97 | ||
98 | 98 | ||
diff --git a/OpenSim/Region/CoreModules/Agent/AssetTransaction/AssetTransactionModule.cs b/OpenSim/Region/CoreModules/Agent/AssetTransaction/AssetTransactionModule.cs index 4bb8986..7332415 100644 --- a/OpenSim/Region/CoreModules/Agent/AssetTransaction/AssetTransactionModule.cs +++ b/OpenSim/Region/CoreModules/Agent/AssetTransaction/AssetTransactionModule.cs | |||
@@ -54,7 +54,7 @@ namespace OpenSim.Region.CoreModules.Agent.AssetTransaction | |||
54 | private Dictionary<UUID, AgentAssetTransactions> AgentTransactions = | 54 | private Dictionary<UUID, AgentAssetTransactions> AgentTransactions = |
55 | new Dictionary<UUID, AgentAssetTransactions>(); | 55 | new Dictionary<UUID, AgentAssetTransactions>(); |
56 | 56 | ||
57 | #region IRegionModule Members | 57 | #region Region Module interface |
58 | 58 | ||
59 | public void Initialise(IConfigSource source) | 59 | public void Initialise(IConfigSource source) |
60 | { | 60 | { |
diff --git a/OpenSim/Region/CoreModules/Agent/TextureSender/J2KDecoderModule.cs b/OpenSim/Region/CoreModules/Agent/TextureSender/J2KDecoderModule.cs index a54fec7..47c4b59 100644 --- a/OpenSim/Region/CoreModules/Agent/TextureSender/J2KDecoderModule.cs +++ b/OpenSim/Region/CoreModules/Agent/TextureSender/J2KDecoderModule.cs | |||
@@ -123,7 +123,7 @@ namespace OpenSim.Region.CoreModules.Agent.TextureSender | |||
123 | get { return null; } | 123 | get { return null; } |
124 | } | 124 | } |
125 | 125 | ||
126 | #endregion IRegionModule | 126 | #endregion Region Module interface |
127 | 127 | ||
128 | #region IJ2KDecoder | 128 | #region IJ2KDecoder |
129 | 129 | ||
diff --git a/OpenSim/Region/CoreModules/Avatar/AvatarFactory/AvatarFactoryModule.cs b/OpenSim/Region/CoreModules/Avatar/AvatarFactory/AvatarFactoryModule.cs index 5b04a39..7f083a9 100644 --- a/OpenSim/Region/CoreModules/Avatar/AvatarFactory/AvatarFactoryModule.cs +++ b/OpenSim/Region/CoreModules/Avatar/AvatarFactory/AvatarFactoryModule.cs | |||
@@ -62,7 +62,7 @@ namespace OpenSim.Region.CoreModules.Avatar.AvatarFactory | |||
62 | 62 | ||
63 | private object m_setAppearanceLock = new object(); | 63 | private object m_setAppearanceLock = new object(); |
64 | 64 | ||
65 | #region IRegionModule | 65 | #region Region Module interface |
66 | 66 | ||
67 | public void Initialise(IConfigSource config) | 67 | public void Initialise(IConfigSource config) |
68 | { | 68 | { |
diff --git a/OpenSim/Region/CoreModules/Avatar/InstantMessage/InstantMessageModule.cs b/OpenSim/Region/CoreModules/Avatar/InstantMessage/InstantMessageModule.cs index 727f1c9..a48b700 100644 --- a/OpenSim/Region/CoreModules/Avatar/InstantMessage/InstantMessageModule.cs +++ b/OpenSim/Region/CoreModules/Avatar/InstantMessage/InstantMessageModule.cs | |||
@@ -54,7 +54,7 @@ namespace OpenSim.Region.CoreModules.Avatar.InstantMessage | |||
54 | 54 | ||
55 | private readonly List<Scene> m_scenes = new List<Scene>(); | 55 | private readonly List<Scene> m_scenes = new List<Scene>(); |
56 | 56 | ||
57 | #region IRegionModule Members | 57 | #region Region Module interface |
58 | 58 | ||
59 | private IMessageTransferModule m_TransferModule = null; | 59 | private IMessageTransferModule m_TransferModule = null; |
60 | 60 | ||
diff --git a/OpenSim/Region/CoreModules/Avatar/Inventory/Transfer/InventoryTransferModule.cs b/OpenSim/Region/CoreModules/Avatar/Inventory/Transfer/InventoryTransferModule.cs index e26beec..5d20524 100644 --- a/OpenSim/Region/CoreModules/Avatar/Inventory/Transfer/InventoryTransferModule.cs +++ b/OpenSim/Region/CoreModules/Avatar/Inventory/Transfer/InventoryTransferModule.cs | |||
@@ -51,7 +51,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Inventory.Transfer | |||
51 | private IMessageTransferModule m_TransferModule = null; | 51 | private IMessageTransferModule m_TransferModule = null; |
52 | private bool m_Enabled = true; | 52 | private bool m_Enabled = true; |
53 | 53 | ||
54 | #region IRegionModule Members | 54 | #region Region Module interface |
55 | 55 | ||
56 | public void Initialise(IConfigSource config) | 56 | public void Initialise(IConfigSource config) |
57 | { | 57 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Asset/AssetServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Asset/AssetServiceInConnectorModule.cs index 5541684..563b1a1 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Asset/AssetServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Asset/AssetServiceInConnectorModule.cs | |||
@@ -47,7 +47,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Asset | |||
47 | private IConfigSource m_Config; | 47 | private IConfigSource m_Config; |
48 | bool m_Registered = false; | 48 | bool m_Registered = false; |
49 | 49 | ||
50 | #region IRegionModule interface | 50 | #region Region Module interface |
51 | 51 | ||
52 | public void Initialise(IConfigSource config) | 52 | public void Initialise(IConfigSource config) |
53 | { | 53 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Authentication/AuthenticationServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Authentication/AuthenticationServiceInConnectorModule.cs index ccf2275..2c6a0b8 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Authentication/AuthenticationServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Authentication/AuthenticationServiceInConnectorModule.cs | |||
@@ -49,7 +49,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Authentication | |||
49 | private IConfigSource m_Config; | 49 | private IConfigSource m_Config; |
50 | bool m_Registered = false; | 50 | bool m_Registered = false; |
51 | 51 | ||
52 | #region IRegionModule interface | 52 | #region Region Module interface |
53 | 53 | ||
54 | public void Initialise(IConfigSource config) | 54 | public void Initialise(IConfigSource config) |
55 | { | 55 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Grid/GridInfoServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Grid/GridInfoServiceInConnectorModule.cs index 0e16e5a..f37b75c 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Grid/GridInfoServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Grid/GridInfoServiceInConnectorModule.cs | |||
@@ -49,7 +49,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Grid | |||
49 | private IConfigSource m_Config; | 49 | private IConfigSource m_Config; |
50 | bool m_Registered = false; | 50 | bool m_Registered = false; |
51 | 51 | ||
52 | #region IRegionModule interface | 52 | #region Region Module interface |
53 | 53 | ||
54 | public void Initialise(IConfigSource config) | 54 | public void Initialise(IConfigSource config) |
55 | { | 55 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Hypergrid/HypergridServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Hypergrid/HypergridServiceInConnectorModule.cs index a7dd0dd..f681ef2 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Hypergrid/HypergridServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Hypergrid/HypergridServiceInConnectorModule.cs | |||
@@ -53,7 +53,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Hypergrid | |||
53 | private GatekeeperServiceInConnector m_HypergridHandler; | 53 | private GatekeeperServiceInConnector m_HypergridHandler; |
54 | private UserAgentServerConnector m_UASHandler; | 54 | private UserAgentServerConnector m_UASHandler; |
55 | 55 | ||
56 | #region IRegionModule interface | 56 | #region Region Module interface |
57 | 57 | ||
58 | public void Initialise(IConfigSource config) | 58 | public void Initialise(IConfigSource config) |
59 | { | 59 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Inventory/InventoryServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Inventory/InventoryServiceInConnectorModule.cs index 831ea27..87b4119 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Inventory/InventoryServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Inventory/InventoryServiceInConnectorModule.cs | |||
@@ -47,7 +47,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Inventory | |||
47 | private IConfigSource m_Config; | 47 | private IConfigSource m_Config; |
48 | bool m_Registered = false; | 48 | bool m_Registered = false; |
49 | 49 | ||
50 | #region IRegionModule interface | 50 | #region Region Module interface |
51 | 51 | ||
52 | public void Initialise(IConfigSource config) | 52 | public void Initialise(IConfigSource config) |
53 | { | 53 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Land/LandServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Land/LandServiceInConnectorModule.cs index 2f3c350..dab89ea 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Land/LandServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Land/LandServiceInConnectorModule.cs | |||
@@ -51,7 +51,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Land | |||
51 | private IConfigSource m_Config; | 51 | private IConfigSource m_Config; |
52 | private List<Scene> m_Scenes = new List<Scene>(); | 52 | private List<Scene> m_Scenes = new List<Scene>(); |
53 | 53 | ||
54 | #region IRegionModule interface | 54 | #region Region Module interface |
55 | 55 | ||
56 | public void Initialise(IConfigSource config) | 56 | public void Initialise(IConfigSource config) |
57 | { | 57 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Login/LLLoginServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Login/LLLoginServiceInConnectorModule.cs index ecdb380..0a86fb7 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Login/LLLoginServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Login/LLLoginServiceInConnectorModule.cs | |||
@@ -51,7 +51,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Login | |||
51 | private IConfigSource m_Config; | 51 | private IConfigSource m_Config; |
52 | private List<Scene> m_Scenes = new List<Scene>(); | 52 | private List<Scene> m_Scenes = new List<Scene>(); |
53 | 53 | ||
54 | #region IRegionModule interface | 54 | #region Region Module interface |
55 | 55 | ||
56 | public void Initialise(IConfigSource config) | 56 | public void Initialise(IConfigSource config) |
57 | { | 57 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/MapImage/MapImageServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/MapImage/MapImageServiceInConnectorModule.cs index d38af23..ec14a98 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/MapImage/MapImageServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/MapImage/MapImageServiceInConnectorModule.cs | |||
@@ -50,7 +50,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.MapImage | |||
50 | 50 | ||
51 | private IConfigSource m_Config; | 51 | private IConfigSource m_Config; |
52 | 52 | ||
53 | #region IRegionModule interface | 53 | #region Region Module interface |
54 | 54 | ||
55 | public void Initialise(IConfigSource config) | 55 | public void Initialise(IConfigSource config) |
56 | { | 56 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Neighbour/NeighbourServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Neighbour/NeighbourServiceInConnectorModule.cs index b544ab3..1f26ed9 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Neighbour/NeighbourServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Neighbour/NeighbourServiceInConnectorModule.cs | |||
@@ -50,7 +50,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Neighbour | |||
50 | private IConfigSource m_Config; | 50 | private IConfigSource m_Config; |
51 | private List<Scene> m_Scenes = new List<Scene>(); | 51 | private List<Scene> m_Scenes = new List<Scene>(); |
52 | 52 | ||
53 | #region IRegionModule interface | 53 | #region Region Module interface |
54 | 54 | ||
55 | public void Initialise(IConfigSource config) | 55 | public void Initialise(IConfigSource config) |
56 | { | 56 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Simulation/SimulationServiceInConnectorModule.cs b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Simulation/SimulationServiceInConnectorModule.cs index 0a5275d..4748e19 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsIn/Simulation/SimulationServiceInConnectorModule.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsIn/Simulation/SimulationServiceInConnectorModule.cs | |||
@@ -49,7 +49,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Simulation | |||
49 | private IConfigSource m_Config; | 49 | private IConfigSource m_Config; |
50 | bool m_Registered = false; | 50 | bool m_Registered = false; |
51 | 51 | ||
52 | #region IRegionModule interface | 52 | #region Region Module interface |
53 | 53 | ||
54 | public void Initialise(IConfigSource config) | 54 | public void Initialise(IConfigSource config) |
55 | { | 55 | { |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs index 8ed1833..c2dc7a1 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs | |||
@@ -58,7 +58,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
58 | /// </summary> | 58 | /// </summary> |
59 | private bool m_ModuleEnabled = false; | 59 | private bool m_ModuleEnabled = false; |
60 | 60 | ||
61 | #region IRegionModule | 61 | #region Region Module interface |
62 | 62 | ||
63 | public void Initialise(IConfigSource config) | 63 | public void Initialise(IConfigSource config) |
64 | { | 64 | { |
@@ -156,7 +156,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
156 | } | 156 | } |
157 | } | 157 | } |
158 | 158 | ||
159 | #endregion /* IRegionModule */ | 159 | #endregion |
160 | 160 | ||
161 | #region ISimulation | 161 | #region ISimulation |
162 | 162 | ||
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/RemoteSimulationConnector.cs b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/RemoteSimulationConnector.cs index 68be552..75bdabf 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/RemoteSimulationConnector.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/RemoteSimulationConnector.cs | |||
@@ -60,7 +60,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
60 | protected bool m_safemode; | 60 | protected bool m_safemode; |
61 | protected IPAddress m_thisIP; | 61 | protected IPAddress m_thisIP; |
62 | 62 | ||
63 | #region IRegionModule | 63 | #region Region Module interface |
64 | 64 | ||
65 | public virtual void Initialise(IConfigSource config) | 65 | public virtual void Initialise(IConfigSource config) |
66 | { | 66 | { |
@@ -147,7 +147,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
147 | m_thisIP = Util.GetHostFromDNS(scene.RegionInfo.ExternalHostName); | 147 | m_thisIP = Util.GetHostFromDNS(scene.RegionInfo.ExternalHostName); |
148 | } | 148 | } |
149 | 149 | ||
150 | #endregion /* IRegionModule */ | 150 | #endregion |
151 | 151 | ||
152 | #region IInterregionComms | 152 | #region IInterregionComms |
153 | 153 | ||
@@ -317,4 +317,4 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
317 | 317 | ||
318 | #endregion /* IInterregionComms */ | 318 | #endregion /* IInterregionComms */ |
319 | } | 319 | } |
320 | } \ No newline at end of file | 320 | } |
diff --git a/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs b/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs index 5fd1bce..e1c5418 100644 --- a/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs +++ b/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs | |||
@@ -1109,7 +1109,7 @@ namespace OpenSim.Region.CoreModules.World.Estate | |||
1109 | 1109 | ||
1110 | #endregion | 1110 | #endregion |
1111 | 1111 | ||
1112 | #region IRegionModule Members | 1112 | #region Region Module interface |
1113 | 1113 | ||
1114 | public string Name { get { return "EstateManagementModule"; } } | 1114 | public string Name { get { return "EstateManagementModule"; } } |
1115 | 1115 | ||
diff --git a/OpenSim/Region/CoreModules/World/LegacyMap/MapImageModule.cs b/OpenSim/Region/CoreModules/World/LegacyMap/MapImageModule.cs index aa306c7..6a443d0 100644 --- a/OpenSim/Region/CoreModules/World/LegacyMap/MapImageModule.cs +++ b/OpenSim/Region/CoreModules/World/LegacyMap/MapImageModule.cs | |||
@@ -131,7 +131,7 @@ namespace OpenSim.Region.CoreModules.World.LegacyMap | |||
131 | 131 | ||
132 | #endregion | 132 | #endregion |
133 | 133 | ||
134 | #region IRegionModule Members | 134 | #region Region Module interface |
135 | 135 | ||
136 | public void Initialise(IConfigSource source) | 136 | public void Initialise(IConfigSource source) |
137 | { | 137 | { |
diff --git a/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs b/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs index 33f6c3f..31f8aec 100644 --- a/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs +++ b/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs | |||
@@ -66,7 +66,7 @@ namespace OpenSim.Region.CoreModules.World.Warp3DMap | |||
66 | private Bitmap lastImage = null; | 66 | private Bitmap lastImage = null; |
67 | private DateTime lastImageTime = DateTime.MinValue; | 67 | private DateTime lastImageTime = DateTime.MinValue; |
68 | 68 | ||
69 | #region IRegionModule Members | 69 | #region Region Module interface |
70 | 70 | ||
71 | public void Initialise(IConfigSource source) | 71 | public void Initialise(IConfigSource source) |
72 | { | 72 | { |
diff --git a/OpenSim/Region/DataSnapshot/DataSnapshotManager.cs b/OpenSim/Region/DataSnapshot/DataSnapshotManager.cs index 18aa609..a7b9099 100644 --- a/OpenSim/Region/DataSnapshot/DataSnapshotManager.cs +++ b/OpenSim/Region/DataSnapshot/DataSnapshotManager.cs | |||
@@ -99,7 +99,7 @@ namespace OpenSim.Region.DataSnapshot | |||
99 | 99 | ||
100 | #endregion | 100 | #endregion |
101 | 101 | ||
102 | #region IRegionModule | 102 | #region Region Module interface |
103 | 103 | ||
104 | public void Initialise(IConfigSource config) | 104 | public void Initialise(IConfigSource config) |
105 | { | 105 | { |
diff --git a/OpenSim/Region/Framework/Interfaces/IRegionModule.cs b/OpenSim/Region/Framework/Interfaces/IRegionModule.cs deleted file mode 100644 index aacc26b..0000000 --- a/OpenSim/Region/Framework/Interfaces/IRegionModule.cs +++ /dev/null | |||
@@ -1,55 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (c) Contributors, http://opensimulator.org/ | ||
3 | * See CONTRIBUTORS.TXT for a full list of copyright holders. | ||
4 | * | ||
5 | * Redistribution and use in source and binary forms, with or without | ||
6 | * modification, are permitted provided that the following conditions are met: | ||
7 | * * Redistributions of source code must retain the above copyright | ||
8 | * notice, this list of conditions and the following disclaimer. | ||
9 | * * Redistributions in binary form must reproduce the above copyright | ||
10 | * notice, this list of conditions and the following disclaimer in the | ||
11 | * documentation and/or other materials provided with the distribution. | ||
12 | * * Neither the name of the OpenSimulator Project nor the | ||
13 | * names of its contributors may be used to endorse or promote products | ||
14 | * derived from this software without specific prior written permission. | ||
15 | * | ||
16 | * THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY | ||
17 | * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | ||
18 | * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
19 | * DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY | ||
20 | * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES | ||
21 | * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; | ||
22 | * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND | ||
23 | * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | ||
24 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | ||
25 | * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
26 | */ | ||
27 | using System; | ||
28 | |||
29 | using Nini.Config; | ||
30 | using OpenSim.Region.Framework.Scenes; | ||
31 | |||
32 | namespace OpenSim.Region.Framework.Interfaces | ||
33 | { | ||
34 | /// <summary> | ||
35 | /// DEPRECATED! Use INonSharedRegionModule or ISharedRegionModule instead | ||
36 | /// </summary> | ||
37 | [Obsolete("Use INonSharedRegionModule or ISharedRegionModule instead", false)] | ||
38 | public interface IRegionModule | ||
39 | { | ||
40 | /// <summary> | ||
41 | /// Initialize the module. | ||
42 | /// </summary> | ||
43 | /// <remarks> | ||
44 | /// For a shared module this can be called multiple times - once per scene. | ||
45 | /// </remarks> | ||
46 | /// <param name="scene"></param> | ||
47 | /// <param name="source">Configuration information. For a shared module this will be identical on every scene call</param> | ||
48 | void Initialise(Scene scene, IConfigSource source); | ||
49 | |||
50 | void PostInitialise(); | ||
51 | void Close(); | ||
52 | string Name { get; } | ||
53 | bool IsSharedModule { get; } | ||
54 | } | ||
55 | } | ||
diff --git a/OpenSim/Region/Framework/ModuleLoader.cs b/OpenSim/Region/Framework/ModuleLoader.cs deleted file mode 100644 index 32ee674..0000000 --- a/OpenSim/Region/Framework/ModuleLoader.cs +++ /dev/null | |||
@@ -1,263 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (c) Contributors, http://opensimulator.org/ | ||
3 | * See CONTRIBUTORS.TXT for a full list of copyright holders. | ||
4 | * | ||
5 | * Redistribution and use in source and binary forms, with or without | ||
6 | * modification, are permitted provided that the following conditions are met: | ||
7 | * * Redistributions of source code must retain the above copyright | ||
8 | * notice, this list of conditions and the following disclaimer. | ||
9 | * * Redistributions in binary form must reproduce the above copyright | ||
10 | * notice, this list of conditions and the following disclaimer in the | ||
11 | * documentation and/or other materials provided with the distribution. | ||
12 | * * Neither the name of the OpenSimulator Project nor the | ||
13 | * names of its contributors may be used to endorse or promote products | ||
14 | * derived from this software without specific prior written permission. | ||
15 | * | ||
16 | * THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY | ||
17 | * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | ||
18 | * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
19 | * DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY | ||
20 | * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES | ||
21 | * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; | ||
22 | * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND | ||
23 | * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | ||
24 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | ||
25 | * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
26 | */ | ||
27 | |||
28 | using System; | ||
29 | using System.Collections.Generic; | ||
30 | using System.IO; | ||
31 | using System.Reflection; | ||
32 | using log4net; | ||
33 | using Nini.Config; | ||
34 | using OpenSim.Region.Framework.Interfaces; | ||
35 | using OpenSim.Region.Framework.Scenes; | ||
36 | |||
37 | namespace OpenSim.Region.Framework | ||
38 | { | ||
39 | public class ModuleLoader | ||
40 | { | ||
41 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | ||
42 | |||
43 | public Dictionary<string, Assembly> LoadedAssemblys = new Dictionary<string, Assembly>(); | ||
44 | |||
45 | private readonly List<IRegionModule> m_loadedModules = new List<IRegionModule>(); | ||
46 | private readonly Dictionary<string, IRegionModule> m_loadedSharedModules = new Dictionary<string, IRegionModule>(); | ||
47 | private readonly IConfigSource m_config; | ||
48 | |||
49 | public ModuleLoader(IConfigSource config) | ||
50 | { | ||
51 | m_config = config; | ||
52 | } | ||
53 | |||
54 | public IRegionModule[] GetLoadedSharedModules | ||
55 | { | ||
56 | get | ||
57 | { | ||
58 | IRegionModule[] regionModules = new IRegionModule[m_loadedSharedModules.Count]; | ||
59 | m_loadedSharedModules.Values.CopyTo(regionModules, 0); | ||
60 | return regionModules; | ||
61 | } | ||
62 | } | ||
63 | |||
64 | public List<IRegionModule> PickupModules(Scene scene, string moduleDir) | ||
65 | { | ||
66 | DirectoryInfo dir = new DirectoryInfo(moduleDir); | ||
67 | List<IRegionModule> modules = new List<IRegionModule>(); | ||
68 | |||
69 | foreach (FileInfo fileInfo in dir.GetFiles("*.dll")) | ||
70 | { | ||
71 | modules.AddRange(LoadRegionModules(fileInfo.FullName, scene)); | ||
72 | } | ||
73 | return modules; | ||
74 | } | ||
75 | |||
76 | public void LoadDefaultSharedModule(IRegionModule module) | ||
77 | { | ||
78 | if (m_loadedSharedModules.ContainsKey(module.Name)) | ||
79 | { | ||
80 | m_log.ErrorFormat("[MODULES]: Module name \"{0}\" already exists in module list. Module not added!", module.Name); | ||
81 | } | ||
82 | else | ||
83 | { | ||
84 | m_loadedSharedModules.Add(module.Name, module); | ||
85 | } | ||
86 | } | ||
87 | |||
88 | |||
89 | public void InitialiseSharedModules(Scene scene) | ||
90 | { | ||
91 | foreach (IRegionModule module in m_loadedSharedModules.Values) | ||
92 | { | ||
93 | module.Initialise(scene, m_config); | ||
94 | scene.AddModule(module.Name, module); //should be doing this? | ||
95 | } | ||
96 | } | ||
97 | |||
98 | public void InitializeModule(IRegionModule module, Scene scene) | ||
99 | { | ||
100 | module.Initialise(scene, m_config); | ||
101 | scene.AddModule(module.Name, module); | ||
102 | m_loadedModules.Add(module); | ||
103 | } | ||
104 | |||
105 | /// <summary> | ||
106 | /// Loads/initialises a Module instance that can be used by multiple Regions | ||
107 | /// </summary> | ||
108 | /// <param name="dllName"></param> | ||
109 | /// <param name="moduleName"></param> | ||
110 | public void LoadSharedModule(string dllName, string moduleName) | ||
111 | { | ||
112 | IRegionModule module = LoadModule(dllName, moduleName); | ||
113 | |||
114 | if (module != null) | ||
115 | LoadSharedModule(module); | ||
116 | } | ||
117 | |||
118 | /// <summary> | ||
119 | /// Loads/initialises a Module instance that can be used by multiple Regions | ||
120 | /// </summary> | ||
121 | /// <param name="module"></param> | ||
122 | public void LoadSharedModule(IRegionModule module) | ||
123 | { | ||
124 | if (!m_loadedSharedModules.ContainsKey(module.Name)) | ||
125 | { | ||
126 | m_loadedSharedModules.Add(module.Name, module); | ||
127 | } | ||
128 | } | ||
129 | |||
130 | public List<IRegionModule> LoadRegionModules(string dllName, Scene scene) | ||
131 | { | ||
132 | IRegionModule[] modules = LoadModules(dllName); | ||
133 | List<IRegionModule> initializedModules = new List<IRegionModule>(); | ||
134 | |||
135 | if (modules.Length > 0) | ||
136 | { | ||
137 | m_log.InfoFormat("[MODULES]: Found Module Library [{0}]", dllName); | ||
138 | foreach (IRegionModule module in modules) | ||
139 | { | ||
140 | if (!module.IsSharedModule) | ||
141 | { | ||
142 | m_log.InfoFormat("[MODULES]: [{0}]: Initializing.", module.Name); | ||
143 | InitializeModule(module, scene); | ||
144 | initializedModules.Add(module); | ||
145 | } | ||
146 | else | ||
147 | { | ||
148 | m_log.InfoFormat("[MODULES]: [{0}]: Loading Shared Module.", module.Name); | ||
149 | LoadSharedModule(module); | ||
150 | } | ||
151 | } | ||
152 | } | ||
153 | return initializedModules; | ||
154 | } | ||
155 | |||
156 | public void LoadRegionModule(string dllName, string moduleName, Scene scene) | ||
157 | { | ||
158 | IRegionModule module = LoadModule(dllName, moduleName); | ||
159 | if (module != null) | ||
160 | { | ||
161 | InitializeModule(module, scene); | ||
162 | } | ||
163 | } | ||
164 | |||
165 | /// <summary> | ||
166 | /// Loads a external Module (if not already loaded) and creates a new instance of it. | ||
167 | /// </summary> | ||
168 | /// <param name="dllName"></param> | ||
169 | /// <param name="moduleName"></param> | ||
170 | public IRegionModule LoadModule(string dllName, string moduleName) | ||
171 | { | ||
172 | IRegionModule[] modules = LoadModules(dllName); | ||
173 | |||
174 | foreach (IRegionModule module in modules) | ||
175 | { | ||
176 | if ((module != null) && (module.Name == moduleName)) | ||
177 | { | ||
178 | return module; | ||
179 | } | ||
180 | } | ||
181 | |||
182 | return null; | ||
183 | } | ||
184 | |||
185 | public IRegionModule[] LoadModules(string dllName) | ||
186 | { | ||
187 | //m_log.DebugFormat("[MODULES]: Looking for modules in {0}", dllName); | ||
188 | |||
189 | List<IRegionModule> modules = new List<IRegionModule>(); | ||
190 | |||
191 | Assembly pluginAssembly; | ||
192 | if (!LoadedAssemblys.TryGetValue(dllName, out pluginAssembly)) | ||
193 | { | ||
194 | try | ||
195 | { | ||
196 | pluginAssembly = Assembly.LoadFrom(dllName); | ||
197 | LoadedAssemblys.Add(dllName, pluginAssembly); | ||
198 | } | ||
199 | catch (BadImageFormatException) | ||
200 | { | ||
201 | //m_log.InfoFormat("[MODULES]: The file [{0}] is not a module assembly.", e.FileName); | ||
202 | } | ||
203 | } | ||
204 | |||
205 | if (pluginAssembly != null) | ||
206 | { | ||
207 | try | ||
208 | { | ||
209 | foreach (Type pluginType in pluginAssembly.GetTypes()) | ||
210 | { | ||
211 | if (pluginType.IsPublic) | ||
212 | { | ||
213 | if (!pluginType.IsAbstract) | ||
214 | { | ||
215 | if (pluginType.GetInterface("IRegionModule") != null) | ||
216 | { | ||
217 | modules.Add((IRegionModule)Activator.CreateInstance(pluginType)); | ||
218 | } | ||
219 | } | ||
220 | } | ||
221 | } | ||
222 | } | ||
223 | catch (Exception e) | ||
224 | { | ||
225 | m_log.ErrorFormat( | ||
226 | "[MODULES]: Could not load types for plugin DLL {0}. Exception {1} {2}", | ||
227 | pluginAssembly.FullName, e.Message, e.StackTrace); | ||
228 | |||
229 | // justincc: Right now this is fatal to really get the user's attention | ||
230 | // TomMeta: WTF? No, how about we /don't/ throw a fatal exception when there's no need to? | ||
231 | //throw e; | ||
232 | } | ||
233 | } | ||
234 | |||
235 | return modules.ToArray(); | ||
236 | } | ||
237 | |||
238 | public void PostInitialise() | ||
239 | { | ||
240 | foreach (IRegionModule module in m_loadedSharedModules.Values) | ||
241 | { | ||
242 | module.PostInitialise(); | ||
243 | } | ||
244 | |||
245 | foreach (IRegionModule module in m_loadedModules) | ||
246 | { | ||
247 | module.PostInitialise(); | ||
248 | } | ||
249 | } | ||
250 | |||
251 | public void ClearCache() | ||
252 | { | ||
253 | LoadedAssemblys.Clear(); | ||
254 | } | ||
255 | |||
256 | public void UnloadModule(IRegionModule rm) | ||
257 | { | ||
258 | rm.Close(); | ||
259 | |||
260 | m_loadedModules.Remove(rm); | ||
261 | } | ||
262 | } | ||
263 | } | ||
diff --git a/OpenSim/Region/Framework/Scenes/Scene.cs b/OpenSim/Region/Framework/Scenes/Scene.cs index 2543333..80d9f6e 100644 --- a/OpenSim/Region/Framework/Scenes/Scene.cs +++ b/OpenSim/Region/Framework/Scenes/Scene.cs | |||
@@ -177,7 +177,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
177 | protected List<RegionInfo> m_regionRestartNotifyList = new List<RegionInfo>(); | 177 | protected List<RegionInfo> m_regionRestartNotifyList = new List<RegionInfo>(); |
178 | protected List<RegionInfo> m_neighbours = new List<RegionInfo>(); | 178 | protected List<RegionInfo> m_neighbours = new List<RegionInfo>(); |
179 | protected string m_simulatorVersion = "OpenSimulator Server"; | 179 | protected string m_simulatorVersion = "OpenSimulator Server"; |
180 | protected ModuleLoader m_moduleLoader; | ||
181 | protected AgentCircuitManager m_authenticateHandler; | 180 | protected AgentCircuitManager m_authenticateHandler; |
182 | protected SceneCommunicationService m_sceneGridService; | 181 | protected SceneCommunicationService m_sceneGridService; |
183 | protected ISnmpModule m_snmpService = null; | 182 | protected ISnmpModule m_snmpService = null; |
@@ -679,7 +678,7 @@ namespace OpenSim.Region.Framework.Scenes | |||
679 | public Scene(RegionInfo regInfo, AgentCircuitManager authen, | 678 | public Scene(RegionInfo regInfo, AgentCircuitManager authen, |
680 | SceneCommunicationService sceneGridService, | 679 | SceneCommunicationService sceneGridService, |
681 | ISimulationDataService simDataService, IEstateDataService estateDataService, | 680 | ISimulationDataService simDataService, IEstateDataService estateDataService, |
682 | ModuleLoader moduleLoader, bool dumpAssetsToFile, | 681 | bool dumpAssetsToFile, |
683 | IConfigSource config, string simulatorVersion) | 682 | IConfigSource config, string simulatorVersion) |
684 | : this(regInfo) | 683 | : this(regInfo) |
685 | { | 684 | { |
@@ -690,7 +689,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
690 | Random random = new Random(); | 689 | Random random = new Random(); |
691 | 690 | ||
692 | m_lastAllocatedLocalId = (uint)(random.NextDouble() * (double)(uint.MaxValue / 2)) + (uint)(uint.MaxValue / 4); | 691 | m_lastAllocatedLocalId = (uint)(random.NextDouble() * (double)(uint.MaxValue / 2)) + (uint)(uint.MaxValue / 4); |
693 | m_moduleLoader = moduleLoader; | ||
694 | m_authenticateHandler = authen; | 692 | m_authenticateHandler = authen; |
695 | m_sceneGridService = sceneGridService; | 693 | m_sceneGridService = sceneGridService; |
696 | m_SimulationDataService = simDataService; | 694 | m_SimulationDataService = simDataService; |
diff --git a/OpenSim/Region/Framework/Scenes/SceneBase.cs b/OpenSim/Region/Framework/Scenes/SceneBase.cs index 7c8bd88..74c9582 100644 --- a/OpenSim/Region/Framework/Scenes/SceneBase.cs +++ b/OpenSim/Region/Framework/Scenes/SceneBase.cs | |||
@@ -67,12 +67,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
67 | /// <value> | 67 | /// <value> |
68 | /// All the region modules attached to this scene. | 68 | /// All the region modules attached to this scene. |
69 | /// </value> | 69 | /// </value> |
70 | public Dictionary<string, IRegionModule> Modules | ||
71 | { | ||
72 | get { return m_modules; } | ||
73 | } | ||
74 | protected Dictionary<string, IRegionModule> m_modules = new Dictionary<string, IRegionModule>(); | ||
75 | |||
76 | public Dictionary<string, IRegionModuleBase> RegionModules | 70 | public Dictionary<string, IRegionModuleBase> RegionModules |
77 | { | 71 | { |
78 | get { return m_regionModules; } | 72 | get { return m_regionModules; } |
@@ -273,16 +267,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
273 | /// </summary> | 267 | /// </summary> |
274 | public virtual void Close() | 268 | public virtual void Close() |
275 | { | 269 | { |
276 | // Shut down all non shared modules. | ||
277 | foreach (IRegionModule module in Modules.Values) | ||
278 | { | ||
279 | if (!module.IsSharedModule) | ||
280 | { | ||
281 | module.Close(); | ||
282 | } | ||
283 | } | ||
284 | Modules.Clear(); | ||
285 | |||
286 | try | 270 | try |
287 | { | 271 | { |
288 | EventManager.TriggerShutdown(); | 272 | EventManager.TriggerShutdown(); |
@@ -313,19 +297,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
313 | #region Module Methods | 297 | #region Module Methods |
314 | 298 | ||
315 | /// <summary> | 299 | /// <summary> |
316 | /// Add a module to this scene. | ||
317 | /// </summary> | ||
318 | /// <param name="name"></param> | ||
319 | /// <param name="module"></param> | ||
320 | public void AddModule(string name, IRegionModule module) | ||
321 | { | ||
322 | if (!Modules.ContainsKey(name)) | ||
323 | { | ||
324 | Modules.Add(name, module); | ||
325 | } | ||
326 | } | ||
327 | |||
328 | /// <summary> | ||
329 | /// Add a region-module to this scene. TODO: This will replace AddModule in the future. | 300 | /// Add a region-module to this scene. TODO: This will replace AddModule in the future. |
330 | /// </summary> | 301 | /// </summary> |
331 | /// <param name="name"></param> | 302 | /// <param name="name"></param> |
@@ -509,9 +480,9 @@ namespace OpenSim.Region.Framework.Scenes | |||
509 | /// <param name="shorthelp"></param> | 480 | /// <param name="shorthelp"></param> |
510 | /// <param name="longhelp"></param> | 481 | /// <param name="longhelp"></param> |
511 | /// <param name="callback"></param> | 482 | /// <param name="callback"></param> |
512 | public void AddCommand(object mod, string command, string shorthelp, string longhelp, CommandDelegate callback) | 483 | public void AddCommand(IRegionModuleBase module, string command, string shorthelp, string longhelp, CommandDelegate callback) |
513 | { | 484 | { |
514 | AddCommand(mod, command, shorthelp, longhelp, string.Empty, callback); | 485 | AddCommand(module, command, shorthelp, longhelp, string.Empty, callback); |
515 | } | 486 | } |
516 | 487 | ||
517 | /// <summary> | 488 | /// <summary> |
@@ -529,9 +500,9 @@ namespace OpenSim.Region.Framework.Scenes | |||
529 | /// <param name="longhelp"></param> | 500 | /// <param name="longhelp"></param> |
530 | /// <param name="callback"></param> | 501 | /// <param name="callback"></param> |
531 | public void AddCommand( | 502 | public void AddCommand( |
532 | string category, object mod, string command, string shorthelp, string longhelp, CommandDelegate callback) | 503 | string category, IRegionModuleBase module, string command, string shorthelp, string longhelp, CommandDelegate callback) |
533 | { | 504 | { |
534 | AddCommand(category, mod, command, shorthelp, longhelp, string.Empty, callback); | 505 | AddCommand(category, module, command, shorthelp, longhelp, string.Empty, callback); |
535 | } | 506 | } |
536 | 507 | ||
537 | /// <summary> | 508 | /// <summary> |
@@ -543,29 +514,14 @@ namespace OpenSim.Region.Framework.Scenes | |||
543 | /// <param name="longhelp"></param> | 514 | /// <param name="longhelp"></param> |
544 | /// <param name="descriptivehelp"></param> | 515 | /// <param name="descriptivehelp"></param> |
545 | /// <param name="callback"></param> | 516 | /// <param name="callback"></param> |
546 | public void AddCommand(object mod, string command, string shorthelp, string longhelp, string descriptivehelp, CommandDelegate callback) | 517 | public void AddCommand(IRegionModuleBase module, string command, string shorthelp, string longhelp, string descriptivehelp, CommandDelegate callback) |
547 | { | 518 | { |
548 | string moduleName = ""; | 519 | string moduleName = ""; |
549 | 520 | ||
550 | if (mod != null) | 521 | if (module != null) |
551 | { | 522 | moduleName = module.Name; |
552 | if (mod is IRegionModule) | ||
553 | { | ||
554 | IRegionModule module = (IRegionModule)mod; | ||
555 | moduleName = module.Name; | ||
556 | } | ||
557 | else if (mod is IRegionModuleBase) | ||
558 | { | ||
559 | IRegionModuleBase module = (IRegionModuleBase)mod; | ||
560 | moduleName = module.Name; | ||
561 | } | ||
562 | else | ||
563 | { | ||
564 | throw new Exception("AddCommand module parameter must be IRegionModule or IRegionModuleBase"); | ||
565 | } | ||
566 | } | ||
567 | 523 | ||
568 | AddCommand(moduleName, mod, command, shorthelp, longhelp, descriptivehelp, callback); | 524 | AddCommand(moduleName, module, command, shorthelp, longhelp, descriptivehelp, callback); |
569 | } | 525 | } |
570 | 526 | ||
571 | /// <summary> | 527 | /// <summary> |
@@ -581,7 +537,7 @@ namespace OpenSim.Region.Framework.Scenes | |||
581 | /// <param name="descriptivehelp"></param> | 537 | /// <param name="descriptivehelp"></param> |
582 | /// <param name="callback"></param> | 538 | /// <param name="callback"></param> |
583 | public void AddCommand( | 539 | public void AddCommand( |
584 | string category, object mod, string command, | 540 | string category, IRegionModuleBase module, string command, |
585 | string shorthelp, string longhelp, string descriptivehelp, CommandDelegate callback) | 541 | string shorthelp, string longhelp, string descriptivehelp, CommandDelegate callback) |
586 | { | 542 | { |
587 | if (MainConsole.Instance == null) | 543 | if (MainConsole.Instance == null) |
@@ -589,22 +545,8 @@ namespace OpenSim.Region.Framework.Scenes | |||
589 | 545 | ||
590 | bool shared = false; | 546 | bool shared = false; |
591 | 547 | ||
592 | if (mod != null) | 548 | if (module != null) |
593 | { | 549 | shared = module is ISharedRegionModule; |
594 | if (mod is IRegionModule) | ||
595 | { | ||
596 | IRegionModule module = (IRegionModule)mod; | ||
597 | shared = module.IsSharedModule; | ||
598 | } | ||
599 | else if (mod is IRegionModuleBase) | ||
600 | { | ||
601 | shared = mod is ISharedRegionModule; | ||
602 | } | ||
603 | else | ||
604 | { | ||
605 | throw new Exception("AddCommand module parameter must be IRegionModule or IRegionModuleBase"); | ||
606 | } | ||
607 | } | ||
608 | 550 | ||
609 | MainConsole.Instance.Commands.AddCommand( | 551 | MainConsole.Instance.Commands.AddCommand( |
610 | category, shared, command, shorthelp, longhelp, descriptivehelp, callback); | 552 | category, shared, command, shorthelp, longhelp, descriptivehelp, callback); |
diff --git a/OpenSim/Region/Framework/Scenes/SceneManager.cs b/OpenSim/Region/Framework/Scenes/SceneManager.cs index dba3a61..89e3ac5 100644 --- a/OpenSim/Region/Framework/Scenes/SceneManager.cs +++ b/OpenSim/Region/Framework/Scenes/SceneManager.cs | |||
@@ -138,31 +138,6 @@ namespace OpenSim.Region.Framework.Scenes | |||
138 | 138 | ||
139 | public void Close() | 139 | public void Close() |
140 | { | 140 | { |
141 | // collect known shared modules in sharedModules | ||
142 | Dictionary<string, IRegionModule> sharedModules = new Dictionary<string, IRegionModule>(); | ||
143 | |||
144 | List<Scene> sceneList = Scenes; | ||
145 | for (int i = 0; i < sceneList.Count; i++) | ||
146 | { | ||
147 | // extract known shared modules from scene | ||
148 | foreach (string k in sceneList[i].Modules.Keys) | ||
149 | { | ||
150 | if (sceneList[i].Modules[k].IsSharedModule && | ||
151 | !sharedModules.ContainsKey(k)) | ||
152 | sharedModules[k] = sceneList[i].Modules[k]; | ||
153 | } | ||
154 | // close scene/region | ||
155 | sceneList[i].Close(); | ||
156 | } | ||
157 | |||
158 | // all regions/scenes are now closed, we can now safely | ||
159 | // close all shared modules | ||
160 | foreach (IRegionModule mod in sharedModules.Values) | ||
161 | { | ||
162 | mod.Close(); | ||
163 | } | ||
164 | |||
165 | m_localScenes.Clear(); | ||
166 | } | 141 | } |
167 | 142 | ||
168 | public void Close(Scene cscene) | 143 | public void Close(Scene cscene) |
diff --git a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsMessagingModule.cs b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsMessagingModule.cs index 1528330..e03a27c 100644 --- a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsMessagingModule.cs +++ b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsMessagingModule.cs | |||
@@ -79,7 +79,7 @@ namespace OpenSim.Region.OptionalModules.Avatar.XmlRpcGroups | |||
79 | 79 | ||
80 | private int m_usersOnlineCacheExpirySeconds = 20; | 80 | private int m_usersOnlineCacheExpirySeconds = 20; |
81 | 81 | ||
82 | #region IRegionModuleBase Members | 82 | #region Region Module interfaceBase Members |
83 | 83 | ||
84 | public void Initialise(IConfigSource config) | 84 | public void Initialise(IConfigSource config) |
85 | { | 85 | { |
diff --git a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs index 79e9994..f26092a 100644 --- a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs +++ b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs | |||
@@ -86,7 +86,7 @@ namespace OpenSim.Region.OptionalModules.Avatar.XmlRpcGroups | |||
86 | private bool m_debugEnabled = false; | 86 | private bool m_debugEnabled = false; |
87 | private int m_levelGroupCreate = 0; | 87 | private int m_levelGroupCreate = 0; |
88 | 88 | ||
89 | #region IRegionModuleBase Members | 89 | #region Region Module interfaceBase Members |
90 | 90 | ||
91 | public void Initialise(IConfigSource config) | 91 | public void Initialise(IConfigSource config) |
92 | { | 92 | { |
diff --git a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/SimianGroupsServicesConnectorModule.cs b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/SimianGroupsServicesConnectorModule.cs index 5d57f70..5c7a3ac 100644 --- a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/SimianGroupsServicesConnectorModule.cs +++ b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/SimianGroupsServicesConnectorModule.cs | |||
@@ -176,7 +176,7 @@ namespace OpenSim.Region.OptionalModules.Avatar.XmlRpcGroups | |||
176 | // private IUserAccountService m_accountService = null; | 176 | // private IUserAccountService m_accountService = null; |
177 | 177 | ||
178 | 178 | ||
179 | #region IRegionModuleBase Members | 179 | #region Region Module interfaceBase Members |
180 | 180 | ||
181 | public string Name | 181 | public string Name |
182 | { | 182 | { |
diff --git a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/XmlRpcGroupsServicesConnectorModule.cs b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/XmlRpcGroupsServicesConnectorModule.cs index d412cd1..cfc26a0 100644 --- a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/XmlRpcGroupsServicesConnectorModule.cs +++ b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/XmlRpcGroupsServicesConnectorModule.cs | |||
@@ -83,7 +83,7 @@ namespace OpenSim.Region.OptionalModules.Avatar.XmlRpcGroups | |||
83 | private Dictionary<UUID, List<UUID>> m_groupsAgentsDroppedFromChatSession = new Dictionary<UUID, List<UUID>>(); | 83 | private Dictionary<UUID, List<UUID>> m_groupsAgentsDroppedFromChatSession = new Dictionary<UUID, List<UUID>>(); |
84 | private Dictionary<UUID, List<UUID>> m_groupsAgentsInvitedToChatSession = new Dictionary<UUID, List<UUID>>(); | 84 | private Dictionary<UUID, List<UUID>> m_groupsAgentsInvitedToChatSession = new Dictionary<UUID, List<UUID>>(); |
85 | 85 | ||
86 | #region IRegionModuleBase Members | 86 | #region Region Module interfaceBase Members |
87 | 87 | ||
88 | public string Name | 88 | public string Name |
89 | { | 89 | { |
diff --git a/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreModule.cs b/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreModule.cs index 732c28f..e68764a 100644 --- a/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreModule.cs +++ b/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreModule.cs | |||
@@ -59,7 +59,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore | |||
59 | private Dictionary<UUID,JsonStore> m_JsonValueStore; | 59 | private Dictionary<UUID,JsonStore> m_JsonValueStore; |
60 | private UUID m_sharedStore; | 60 | private UUID m_sharedStore; |
61 | 61 | ||
62 | #region IRegionModule Members | 62 | #region Region Module interface |
63 | 63 | ||
64 | // ----------------------------------------------------------------- | 64 | // ----------------------------------------------------------------- |
65 | /// <summary> | 65 | /// <summary> |
diff --git a/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreScriptModule.cs b/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreScriptModule.cs index 6910d14..0c175ca 100644 --- a/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreScriptModule.cs +++ b/OpenSim/Region/OptionalModules/Scripting/JsonStore/JsonStoreScriptModule.cs | |||
@@ -58,7 +58,7 @@ namespace OpenSim.Region.OptionalModules.Scripting.JsonStore | |||
58 | private IScriptModuleComms m_comms; | 58 | private IScriptModuleComms m_comms; |
59 | private IJsonStoreModule m_store; | 59 | private IJsonStoreModule m_store; |
60 | 60 | ||
61 | #region IRegionModule Members | 61 | #region Region Module interface |
62 | 62 | ||
63 | // ----------------------------------------------------------------- | 63 | // ----------------------------------------------------------------- |
64 | /// <summary> | 64 | /// <summary> |
diff --git a/OpenSim/Region/OptionalModules/ServiceConnectorsIn/Freeswitch/FreeswitchServiceInConnectorModule.cs b/OpenSim/Region/OptionalModules/ServiceConnectorsIn/Freeswitch/FreeswitchServiceInConnectorModule.cs index 74c5139..78c870a 100644 --- a/OpenSim/Region/OptionalModules/ServiceConnectorsIn/Freeswitch/FreeswitchServiceInConnectorModule.cs +++ b/OpenSim/Region/OptionalModules/ServiceConnectorsIn/Freeswitch/FreeswitchServiceInConnectorModule.cs | |||
@@ -48,7 +48,7 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsIn.Freeswitch | |||
48 | private IConfigSource m_Config; | 48 | private IConfigSource m_Config; |
49 | bool m_Registered = false; | 49 | bool m_Registered = false; |
50 | 50 | ||
51 | #region IRegionModule interface | 51 | #region Region Module interface |
52 | 52 | ||
53 | public void Initialise(IConfigSource config) | 53 | public void Initialise(IConfigSource config) |
54 | { | 54 | { |
diff --git a/OpenSim/Region/OptionalModules/World/TreePopulator/TreePopulatorModule.cs b/OpenSim/Region/OptionalModules/World/TreePopulator/TreePopulatorModule.cs index 6ebdf4d..3d0b9fb 100644 --- a/OpenSim/Region/OptionalModules/World/TreePopulator/TreePopulatorModule.cs +++ b/OpenSim/Region/OptionalModules/World/TreePopulator/TreePopulatorModule.cs | |||
@@ -168,7 +168,7 @@ namespace OpenSim.Region.OptionalModules.World.TreePopulator | |||
168 | 168 | ||
169 | #endregion | 169 | #endregion |
170 | 170 | ||
171 | #region IRegionModule Members | 171 | #region Region Module interface |
172 | 172 | ||
173 | public void Initialise(IConfigSource config) | 173 | public void Initialise(IConfigSource config) |
174 | { | 174 | { |
@@ -200,7 +200,6 @@ namespace OpenSim.Region.OptionalModules.World.TreePopulator | |||
200 | public void AddRegion(Scene scene) | 200 | public void AddRegion(Scene scene) |
201 | { | 201 | { |
202 | m_scene = scene; | 202 | m_scene = scene; |
203 | //m_scene.RegisterModuleInterface<IRegionModule>(this); | ||
204 | m_scene.RegisterModuleCommander(m_commander); | 203 | m_scene.RegisterModuleCommander(m_commander); |
205 | m_scene.EventManager.OnPluginConsole += EventManager_OnPluginConsole; | 204 | m_scene.EventManager.OnPluginConsole += EventManager_OnPluginConsole; |
206 | 205 | ||
diff --git a/OpenSim/Tests/Common/Helpers/SceneHelpers.cs b/OpenSim/Tests/Common/Helpers/SceneHelpers.cs index db0a7fc..939fc33 100644 --- a/OpenSim/Tests/Common/Helpers/SceneHelpers.cs +++ b/OpenSim/Tests/Common/Helpers/SceneHelpers.cs | |||
@@ -139,7 +139,7 @@ namespace OpenSim.Tests.Common | |||
139 | SceneCommunicationService scs = new SceneCommunicationService(); | 139 | SceneCommunicationService scs = new SceneCommunicationService(); |
140 | 140 | ||
141 | TestScene testScene = new TestScene( | 141 | TestScene testScene = new TestScene( |
142 | regInfo, m_acm, scs, m_simDataService, m_estateDataService, null, false, configSource, null); | 142 | regInfo, m_acm, scs, m_simDataService, m_estateDataService, false, configSource, null); |
143 | 143 | ||
144 | INonSharedRegionModule godsModule = new GodsModule(); | 144 | INonSharedRegionModule godsModule = new GodsModule(); |
145 | godsModule.Initialise(new IniConfigSource()); | 145 | godsModule.Initialise(new IniConfigSource()); |
@@ -359,28 +359,10 @@ namespace OpenSim.Tests.Common | |||
359 | List<IRegionModuleBase> newModules = new List<IRegionModuleBase>(); | 359 | List<IRegionModuleBase> newModules = new List<IRegionModuleBase>(); |
360 | foreach (object module in modules) | 360 | foreach (object module in modules) |
361 | { | 361 | { |
362 | // Console.WriteLine("MODULE RAW {0}", module); | 362 | IRegionModuleBase m = (IRegionModuleBase)module; |
363 | if (module is IRegionModule) | 363 | // Console.WriteLine("MODULE {0}", m.Name); |
364 | { | 364 | m.Initialise(config); |
365 | IRegionModule m = (IRegionModule)module; | 365 | newModules.Add(m); |
366 | |||
367 | foreach (Scene scene in scenes) | ||
368 | { | ||
369 | m.Initialise(scene, config); | ||
370 | scene.AddModule(m.Name, m); | ||
371 | } | ||
372 | |||
373 | m.PostInitialise(); | ||
374 | } | ||
375 | else if (module is IRegionModuleBase) | ||
376 | { | ||
377 | // for the new system, everything has to be initialised first, | ||
378 | // shared modules have to be post-initialised, then all get an AddRegion with the scene | ||
379 | IRegionModuleBase m = (IRegionModuleBase)module; | ||
380 | // Console.WriteLine("MODULE {0}", m.Name); | ||
381 | m.Initialise(config); | ||
382 | newModules.Add(m); | ||
383 | } | ||
384 | } | 366 | } |
385 | 367 | ||
386 | foreach (IRegionModuleBase module in newModules) | 368 | foreach (IRegionModuleBase module in newModules) |
@@ -709,4 +691,4 @@ namespace OpenSim.Tests.Common | |||
709 | return sog; | 691 | return sog; |
710 | } | 692 | } |
711 | } | 693 | } |
712 | } \ No newline at end of file | 694 | } |
diff --git a/OpenSim/Tests/Common/Mock/TestScene.cs b/OpenSim/Tests/Common/Mock/TestScene.cs index eea68c3..d4b5648 100644 --- a/OpenSim/Tests/Common/Mock/TestScene.cs +++ b/OpenSim/Tests/Common/Mock/TestScene.cs | |||
@@ -41,9 +41,9 @@ namespace OpenSim.Tests.Common.Mock | |||
41 | public TestScene( | 41 | public TestScene( |
42 | RegionInfo regInfo, AgentCircuitManager authen, | 42 | RegionInfo regInfo, AgentCircuitManager authen, |
43 | SceneCommunicationService sceneGridService, ISimulationDataService simDataService, IEstateDataService estateDataService, | 43 | SceneCommunicationService sceneGridService, ISimulationDataService simDataService, IEstateDataService estateDataService, |
44 | ModuleLoader moduleLoader, bool dumpAssetsToFile, | 44 | bool dumpAssetsToFile, |
45 | IConfigSource config, string simulatorVersion) | 45 | IConfigSource config, string simulatorVersion) |
46 | : base(regInfo, authen, sceneGridService, simDataService, estateDataService, moduleLoader, | 46 | : base(regInfo, authen, sceneGridService, simDataService, estateDataService, |
47 | dumpAssetsToFile, config, simulatorVersion) | 47 | dumpAssetsToFile, config, simulatorVersion) |
48 | { | 48 | { |
49 | } | 49 | } |
@@ -73,4 +73,4 @@ namespace OpenSim.Tests.Common.Mock | |||
73 | get { return m_asyncSceneObjectDeleter; } | 73 | get { return m_asyncSceneObjectDeleter; } |
74 | } | 74 | } |
75 | } | 75 | } |
76 | } \ No newline at end of file | 76 | } |