add several options for NPC creation so abusive use can be reduced (restrictive by default) UNTESTED

OpenSim/Region/Framework/Interfaces/INPCModule.cs

@@ -31,6 +31,16 @@ using OpenSim.Region.Framework.Scenes;
 
 namespace OpenSim.Region.Framework.Interfaces
 {
+    // option flags for NPCs
+    public enum NPCOptionsFlags : int
+    {
+        None                    = 0x00, // no flags (max restriction)
+        AllowNotOwned           = 0x01, // allow NPCs to be created not Owned
+        AllowSenseAsAvatar      = 0x02, // allow NPCs to set to be sensed as Avatars
+        AllowCloneOtherAvatars  = 0x04, // allow NPCs to created cloning a avatar in region
+        NoNPCGroup              = 0x08  // NPCs will have no group title, otherwise will have "- NPC -"
+    }
+
     /// <summary>
     /// Temporary interface. More methods to come at some point to make NPCs
     /// more object oriented rather than controlling purely through module
@@ -284,5 +294,7 @@ namespace OpenSim.Region.Framework.Interfaces
         /// agent, the agent is unowned  or the agent was not an NPC.
         /// </returns>
         UUID GetOwner(UUID agentID);
+ 
+        NPCOptionsFlags NPCOptionFlags {get;}
     }
 }

OpenSim/Region/OptionalModules/World/NPC/NPCModule.cs

@@ -52,6 +52,11 @@ namespace OpenSim.Region.OptionalModules.World.NPC
         private Dictionary<UUID, NPCAvatar> m_avatars =
                 new Dictionary<UUID, NPCAvatar>();
 
+
+
+        private NPCOptionsFlags m_NPCOptionFlags;
+        public NPCOptionsFlags NPCOptionFlags {get {return m_NPCOptionFlags;}}
+
         public bool Enabled { get; private set; }
 
         public void Initialise(IConfigSource source)
@@ -59,6 +64,27 @@ namespace OpenSim.Region.OptionalModules.World.NPC
             IConfig config = source.Configs["NPC"];
 
             Enabled = (config != null && config.GetBoolean("Enabled", false));
+            m_NPCOptionFlags = NPCOptionsFlags.None;
+            if(Enabled)
+            {
+                bool opt = false;
+
+                opt = config.GetBoolean("AllowNotOwned", false);
+                if(opt)
+                    m_NPCOptionFlags |= NPCOptionsFlags.AllowNotOwned;
+
+                opt = config.GetBoolean("AllowSenseAsAvatar", false);
+                if(opt)
+                    m_NPCOptionFlags |= NPCOptionsFlags.AllowSenseAsAvatar;
+
+                opt = config.GetBoolean("AllowCloneOtherAvatars", false);
+                if(opt)
+                    m_NPCOptionFlags |= NPCOptionsFlags.AllowCloneOtherAvatars;
+
+                opt = config.GetBoolean("NoNPCGroup", false);
+                if(opt)
+                    m_NPCOptionFlags |= NPCOptionsFlags.NoNPCGroup;
+            }
         }
 
         public void AddRegion(Scene scene)

OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs

@@ -142,7 +142,6 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
         internal float m_ScriptDistanceFactor = 1.0f;
         internal bool m_debuggerSafe = false;
         internal Dictionary<string, FunctionPerms > m_FunctionPerms = new Dictionary<string, FunctionPerms >();      
-
         protected IUrlModule m_UrlModule = null;
 
         public void Initialize(
@@ -2578,7 +2577,14 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
             CheckThreatLevel(ThreatLevel.High, "osNpcCreate");
             m_host.AddScriptLPS(1);
             
-            return NpcCreate(firstname, lastname, position, notecard, false, false);
+            // have to get the npc module also here to set the default Not Owned
+            INPCModule module = World.RequestModuleInterface<INPCModule>();
+            if(module == null)
+                return new LSL_Key(UUID.Zero.ToString());
+            
+            bool owned = (module.NPCOptionFlags & NPCOptionsFlags.AllowNotOwned) == 0;
+
+            return NpcCreate(firstname, lastname, position, notecard, owned, false);
         }
 
         public LSL_Key osNpcCreate(string firstname, string lastname, LSL_Vector position, string notecard, int options)
@@ -2595,7 +2601,6 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
         private LSL_Key NpcCreate(
             string firstname, string lastname, LSL_Vector position, string notecard, bool owned, bool senseAsAgent)
         {
-            string groupTitle = String.Empty;
 
             if (!World.Permissions.CanRezObject(1, m_host.OwnerID, new Vector3((float)position.x, (float)position.y, (float)position.z)))
                 return new LSL_Key(UUID.Zero.ToString());
@@ -2603,8 +2608,35 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
             INPCModule module = World.RequestModuleInterface<INPCModule>();
             if (module != null)
             {
+                string groupTitle = String.Empty;
                 AvatarAppearance appearance = null;
 
+                // check creation options
+                NPCOptionsFlags createFlags = module.NPCOptionFlags;
+
+                if((createFlags & NPCOptionsFlags.AllowNotOwned) == 0 && !owned)
+                {
+                    OSSLError("Not owned NPCs disabled");
+                    owned = true; // we should get here...
+                }
+
+                if((createFlags & NPCOptionsFlags.AllowSenseAsAvatar) == 0 && senseAsAgent)
+                {
+                    OSSLError("NPC allow sense as Avatar disabled");
+                    senseAsAgent = false;
+                }
+
+                if((createFlags & NPCOptionsFlags.NoNPCGroup) == 0)
+                {
+                    if (firstname != String.Empty || lastname != String.Empty)
+                        {
+                            if (firstname != "Shown outfit:")
+                                groupTitle = "- NPC -";
+                        }
+                }
+               
+                if((createFlags & NPCOptionsFlags.AllowCloneOtherAvatars) != 0)
+                {
                     UUID id;
                     if (UUID.TryParse(notecard, out id))
                     {
@@ -2612,6 +2644,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                         if (clonePresence != null)
                             appearance = clonePresence.Appearance;
                     }
+                }
 
                 if (appearance == null)
                 {

bin/OpenSim.ini.example

@@ -1115,6 +1115,21 @@
     ;# {Enabled} {} {Enable Non Player Character (NPC) facilities} {true false} false
     ; Enabled = false
 	
+	;; several options to control NPCs creation
+	;; should only be set true on special uses.
+	
+	;; allow NPCs to be created not Owned {true false} false
+    ; AllowNotOwned = false
+	
+	;; allow NPCs to set to be sensed as Avatars {true false} false
+    ; AllowSenseAsAvatar = false
+	
+	;; allow NPCs to created cloning any avatar in region {true false} false
+    ; AllowCloneOtherAvatars = false
+	
+	;; if true NPCs will have no group title, if false display "- NPC -" for easy identification {true false} false
+    ; NoNPCGroup = false 
+
 
 [Terrain]
     ;# {InitialTerrain} {} {Initial terrain type} {pinhead-island flat} pinhead-island