* Implement help <command> from the region console

* So at the moment once can type 'help terrain fill' as well as 'terrain fill help'
* Current implementation is a transient hack that should be tidied up soon

2 files changed, 54 insertions, 4 deletions

diff --git a/OpenSim/Region/Framework/Interfaces/ICommander.cs b/OpenSim/Region/Framework/Interfaces/ICommander.cs
index 17a2e4a..f94e8e8 100644
--- a/OpenSim/Region/Framework/Interfaces/ICommander.cs
+++ b/OpenSim/Region/Framework/Interfaces/ICommander.cs
@@ -25,6 +25,8 @@
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+using System.Collections.Generic;
+
 namespace OpenSim.Region.Framework.Interfaces
 {
     public interface ICommander
@@ -39,6 +41,11 @@ namespace OpenSim.Region.Framework.Interfaces
         /// </value>
         string Help { get; }
         
+        /// <summary>
+        /// The commands available for this commander
+        /// </summary>
+        Dictionary<string, ICommand> Commands { get; }     
+        
         void ProcessConsoleCommand(string function, string[] args);
         void RegisterCommand(string commandName, ICommand command);
         void Run(string function, object[] args);
diff --git a/OpenSim/Region/Framework/Scenes/SceneBase.cs b/OpenSim/Region/Framework/Scenes/SceneBase.cs
index 6d61e9f..e58d3ce 100644
--- a/OpenSim/Region/Framework/Scenes/SceneBase.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneBase.cs
@@ -61,17 +61,24 @@ namespace OpenSim.Region.Framework.Scenes
         /// <value>
         /// The module interfaces available from this scene.
         /// </value>
-        protected Dictionary<Type, List<object> > ModuleInterfaces = new Dictionary<Type, List<object> >();
+        protected Dictionary<Type, List<object>> ModuleInterfaces = new Dictionary<Type, List<object>>();
 
         protected Dictionary<string, object> ModuleAPIMethods = new Dictionary<string, object>();
+        
+        /// <value>
+        /// The module commanders available from this scene
+        /// </value>
         protected Dictionary<string, ICommander> m_moduleCommanders = new Dictionary<string, ICommander>();
         
         /// <value>
+        /// The module commands available for this scene
+        /// </value>
+        protected Dictionary<string, ICommand> m_moduleCommands = new Dictionary<string, ICommand>();        
+        
+        /// <value>
         /// Registered classes that are capable of creating entities.
         /// </value>
-        protected Dictionary<PCode, IEntityCreator> m_entityCreators = new Dictionary<PCode, IEntityCreator>();        
-
-        //API module interfaces        
+        protected Dictionary<PCode, IEntityCreator> m_entityCreators = new Dictionary<PCode, IEntityCreator>();                
 
         /// <summary>
         /// The last allocated local prim id.  When a new local id is requested, the next number in the sequence is
@@ -278,11 +285,47 @@ namespace OpenSim.Region.Framework.Scenes
             }
         }
 
+        /// <summary>
+        /// Register a module commander.
+        /// </summary>
+        /// <param name="commander"></param>
         public void RegisterModuleCommander(ICommander commander)
         {
             lock (m_moduleCommanders)
             {
                 m_moduleCommanders.Add(commander.Name, commander);
+                
+                lock (m_moduleCommands)
+                {
+                    foreach (ICommand command in commander.Commands.Values)
+                    {
+                        if (m_moduleCommands.ContainsKey(command.Name))
+                        {
+                            m_log.ErrorFormat(
+                                "[MODULES]: Module commander {0} tried to register the command {1} which has already been registered",
+                                commander.Name, command.Name);
+                            continue;
+                        }
+                            
+                        m_moduleCommands[command.Name] = command;
+                    }                    
+                }
+            }
+        }
+        
+        /// <summary>
+        /// Get an available module command
+        /// </summary>
+        /// <param name="commandName"></param>
+        /// <returns>The command if it was found, null if no command is available with that name</returns>
+        public ICommand GetCommand(string commandName)
+        {
+            lock (m_moduleCommands)
+            {
+                if (m_moduleCommands.ContainsKey(commandName))
+                    return m_moduleCommands[commandName];
+                else
+                    return null;
             }
         }