* Applying Mantis #1852 - Plugin Provider refactoring. Courtesy of Ryan/Sempuki.

2 files changed, 53 insertions, 20 deletions

diff --git a/OpenSim/Framework/PluginLoader.cs b/OpenSim/Framework/PluginLoader.cs
index c4602a8..616fa3e 100644
--- a/OpenSim/Framework/PluginLoader.cs
+++ b/OpenSim/Framework/PluginLoader.cs
@@ -27,6 +27,7 @@
 
 using System;
 using System.IO;
+using System.Xml;
 using System.Collections.Generic;
 using System.Reflection;
 using log4net;
@@ -61,7 +62,7 @@ namespace OpenSim.Framework
     /// </summary>
     public interface IPluginFilter
     {
-        bool Apply (ExtensionNode plugin);
+        bool Apply (PluginExtensionNode plugin);
     }
 
     /// <summary>
@@ -152,7 +153,7 @@ namespace OpenSim.Framework
                 if (filters.ContainsKey (ext))
                     filter = filters [ext];
 
-                foreach (TypeExtensionNode node in AddinManager.GetExtensionNodes (ext))
+                foreach (PluginExtensionNode node in AddinManager.GetExtensionNodes (ext))
                 {
                     log.Info("[PLUGINS]: Trying plugin " + node.Path);
                     
@@ -197,9 +198,13 @@ namespace OpenSim.Framework
 
         private void on_addinloaderror_(object sender, AddinErrorEventArgs args)
         {
-            log.Error ("[PLUGINS]: Plugin Error: " + args.Message 
-                    + ": " + args.Exception.Message 
-                    + "\n"+ args.Exception.StackTrace);
+            if (args.Exception == null)
+                log.Error ("[PLUGINS]: Plugin Error: " 
+                        + args.Message);
+            else
+                log.Error ("[PLUGINS]: Plugin Error: "
+                        + args.Exception.Message + "\n"
+                        + args.Exception.StackTrace);
         }
 
         private void clear_registry_ ()
@@ -230,6 +235,39 @@ namespace OpenSim.Framework
         }
     }
 
+    public class PluginExtensionNode : ExtensionNode
+    {
+        [NodeAttribute]
+        string provider;
+
+        [NodeAttribute]
+        string type;
+
+        Type typeobj;
+
+        public string Provider { get { return provider; } }
+        public string TypeName { get { return type; } }
+
+        public Type TypeObject
+        {
+            get 
+            {
+                if (typeobj != null)
+                    return typeobj;
+
+                if (type.Length == 0)
+                    throw new InvalidOperationException ("Type name not specified.");
+
+                return typeobj = Addin.GetType (type, true);
+            }
+        }
+
+        public object CreateInstance () 
+        { 
+            return Activator.CreateInstance (TypeObject); 
+        }
+    }
+
     /// <summary>
     /// Constraint that bounds the number of plugins to be loaded.
     /// </summary>
@@ -271,25 +309,20 @@ namespace OpenSim.Framework
     }
     
     /// <summary>
-    /// Filters out which plugin to load based on its "Id", which is name given by the namespace or by Mono.Addins.
+    /// Filters out which plugin to load based on its "Provider", which is name given by in the addin.xml
     /// </summary>
-    public class PluginIdFilter : IPluginFilter
+    public class PluginProviderFilter : IPluginFilter
     {
-        private string id;
+        private string provider;
 
-        public PluginIdFilter (string id) 
+        public PluginProviderFilter (string p) 
         {
-            this.id = id;
+            provider = p;
         }
 
-        public bool Apply (ExtensionNode plugin)
+        public bool Apply (PluginExtensionNode plugin)
         {
-            System.Console.WriteLine ("[WTF]: " + plugin.Path);
-
-            if (plugin.HasId == false)
-                return false;
-            
-            return (plugin.Id == id);
+            return (plugin.Provider == provider);
         }
     }
 }
diff --git a/OpenSim/Grid/GridServer/GridManager.cs b/OpenSim/Grid/GridServer/GridManager.cs
index ec3ca19..98e670f 100644
--- a/OpenSim/Grid/GridServer/GridManager.cs
+++ b/OpenSim/Grid/GridServer/GridManager.cs
@@ -71,9 +71,9 @@ namespace OpenSim.Grid.GridServer
             logloader.AddExtensionPoint ("/OpenSim/LogData");
             
             // loader will try to load all providers (MySQL, MSSQL, etc) 
-            // unless it is constrainted to the correct "id"
-            gridloader.AddFilter ("/OpenSim/GridData", new PluginIdFilter (provider));
-            logloader.AddFilter ("/OpenSim/LogData", new PluginIdFilter (provider));
+            // unless it is constrainted to the correct "Provider" entry in the addin.xml
+            gridloader.AddFilter ("/OpenSim/GridData", new PluginProviderFilter (provider));
+            logloader.AddFilter ("/OpenSim/LogData", new PluginProviderFilter (provider));
             
             gridloader.Load();
             logloader.Load();