From 58a0d0bb32a942b88a97e9d2be415e24db7fa197 Mon Sep 17 00:00:00 2001
From: Sean Dague
Date: Wed, 16 Jul 2008 18:29:03 +0000
Subject: be a little more discriminating on firing ShapeChanged script events
 so that we don't get them on every object construction.

---
 OpenSim/Region/Environment/Scenes/SceneObjectPart.cs | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

(limited to 'OpenSim/Region/Environment')

diff --git a/OpenSim/Region/Environment/Scenes/SceneObjectPart.cs b/OpenSim/Region/Environment/Scenes/SceneObjectPart.cs
index 36d1f50..61300b5 100644
--- a/OpenSim/Region/Environment/Scenes/SceneObjectPart.cs
+++ b/OpenSim/Region/Environment/Scenes/SceneObjectPart.cs
@@ -191,7 +191,7 @@ namespace OpenSim.Region.Environment.Scenes
         protected byte[] m_particleSystem = new byte[0];
         protected ulong m_regionHandle;
         protected LLQuaternion m_rotationOffset;
-        protected PrimitiveBaseShape m_shape;
+        protected PrimitiveBaseShape m_shape = null;
         protected LLUUID m_uuid;
         protected LLVector3 m_velocity;
 
@@ -696,8 +696,18 @@ namespace OpenSim.Region.Environment.Scenes
             get { return m_shape; }
             set
             {
+                bool shape_changed = false;
+                // TODO: this should really be restricted to the right
+                // set of attributes on shape change.  For instance,
+                // changing the lighting on a shape shouldn't cause
+                // this.
+                if (m_shape != null) 
+                    shape_changed = true;
+
                 m_shape = value;
-                TriggerScriptChangedEvent(Changed.SHAPE);
+
+                if (shape_changed)
+                    TriggerScriptChangedEvent(Changed.SHAPE);
             }
         }
         public LLVector3 Scale
-- 
cgit v1.1