Browse Source

Fixed: Windlight functions caused an error if called when the script's owner isn't in the scene

Oren Hurvitz 11 years ago
parent
commit
50ea2e0d67
1 changed files with 9 additions and 3 deletions
  1. 9 3
      OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LS_Api.cs

+ 9 - 3
OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LS_Api.cs

@@ -446,7 +446,9 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                 LSShoutError("LightShare functions are not enabled.");
                 LSShoutError("LightShare functions are not enabled.");
                 return 0;
                 return 0;
             }
             }
-            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && World.GetScenePresence(m_host.OwnerID).GodLevel < 200)
+
+            ScenePresence sp = World.GetScenePresence(m_host.OwnerID);
+            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && (sp == null || sp.GodLevel < 200))
             {
             {
                 LSShoutError("lsSetWindlightScene can only be used by estate managers or owners.");
                 LSShoutError("lsSetWindlightScene can only be used by estate managers or owners.");
                 return 0;
                 return 0;
@@ -474,7 +476,9 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                 LSShoutError("LightShare functions are not enabled.");
                 LSShoutError("LightShare functions are not enabled.");
                 return;
                 return;
             }
             }
-            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && World.GetScenePresence(m_host.OwnerID).GodLevel < 200)
+            
+            ScenePresence sp = World.GetScenePresence(m_host.OwnerID);
+            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && (sp == null || sp.GodLevel < 200))
             {
             {
                 LSShoutError("lsSetWindlightScene can only be used by estate managers or owners.");
                 LSShoutError("lsSetWindlightScene can only be used by estate managers or owners.");
                 return;
                 return;
@@ -497,7 +501,9 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                 LSShoutError("LightShare functions are not enabled.");
                 LSShoutError("LightShare functions are not enabled.");
                 return 0;
                 return 0;
             }
             }
-            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && World.GetScenePresence(m_host.OwnerID).GodLevel < 200)
+            
+            ScenePresence sp = World.GetScenePresence(m_host.OwnerID);
+            if (!World.RegionInfo.EstateSettings.IsEstateManagerOrOwner(m_host.OwnerID) && (sp == null || sp.GodLevel < 200))
             {
             {
                 LSShoutError("lsSetWindlightSceneTargeted can only be used by estate managers or owners.");
                 LSShoutError("lsSetWindlightSceneTargeted can only be used by estate managers or owners.");
                 return 0;
                 return 0;