aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim
diff options
context:
space:
mode:
authorMelanie2011-12-10 14:47:00 +0100
committerMelanie2011-12-10 14:47:00 +0100
commitdb98698bbea1777fd3ba50d7c441d581a733addd (patch)
tree24e30a6e13f5acd1010769d49730409f2d5b1028 /OpenSim
parentMerge branch 'master' into careminster (diff)
downloadopensim-SC_OLD-db98698bbea1777fd3ba50d7c441d581a733addd.zip
opensim-SC_OLD-db98698bbea1777fd3ba50d7c441d581a733addd.tar.gz
opensim-SC_OLD-db98698bbea1777fd3ba50d7c441d581a733addd.tar.bz2
opensim-SC_OLD-db98698bbea1777fd3ba50d7c441d581a733addd.tar.xz
Prevent spurious error message when client tries to move a null item
Diffstat (limited to 'OpenSim')
-rw-r--r--OpenSim/Region/Framework/Scenes/Scene.Inventory.cs4
-rw-r--r--OpenSim/Region/Framework/Scenes/ScenePresence.cs258
2 files changed, 143 insertions, 119 deletions
diff --git a/OpenSim/Region/Framework/Scenes/Scene.Inventory.cs b/OpenSim/Region/Framework/Scenes/Scene.Inventory.cs
index 83990b1..1746cfe 100644
--- a/OpenSim/Region/Framework/Scenes/Scene.Inventory.cs
+++ b/OpenSim/Region/Framework/Scenes/Scene.Inventory.cs
@@ -1105,6 +1105,10 @@ namespace OpenSim.Region.Framework.Scenes
1105 { 1105 {
1106 SceneObjectPart part = GetSceneObjectPart(primLocalId); 1106 SceneObjectPart part = GetSceneObjectPart(primLocalId);
1107 1107
1108 // Can't move a null item
1109 if (itemId == UUID.Zero)
1110 return;
1111
1108 if (null == part) 1112 if (null == part)
1109 { 1113 {
1110 m_log.WarnFormat( 1114 m_log.WarnFormat(
diff --git a/OpenSim/Region/Framework/Scenes/ScenePresence.cs b/OpenSim/Region/Framework/Scenes/ScenePresence.cs
index 5dda9e1..8ff804b 100644
--- a/OpenSim/Region/Framework/Scenes/ScenePresence.cs
+++ b/OpenSim/Region/Framework/Scenes/ScenePresence.cs
@@ -214,6 +214,9 @@ namespace OpenSim.Region.Framework.Scenes
214 //private int m_moveToPositionStateStatus; 214 //private int m_moveToPositionStateStatus;
215 //***************************************************** 215 //*****************************************************
216 216
217 private bool m_collisionEventFlag = false;
218 private object m_collisionEventLock = new Object();
219
217 protected AvatarAppearance m_appearance; 220 protected AvatarAppearance m_appearance;
218 221
219 public AvatarAppearance Appearance 222 public AvatarAppearance Appearance
@@ -3860,147 +3863,164 @@ namespace OpenSim.Region.Framework.Scenes
3860 3863
3861 private void RaiseCollisionScriptEvents(Dictionary<uint, ContactPoint> coldata) 3864 private void RaiseCollisionScriptEvents(Dictionary<uint, ContactPoint> coldata)
3862 { 3865 {
3863 List<uint> thisHitColliders = new List<uint>(); 3866 lock(m_collisionEventLock)
3864 List<uint> endedColliders = new List<uint>();
3865 List<uint> startedColliders = new List<uint>();
3866
3867 foreach (uint localid in coldata.Keys)
3868 {
3869 thisHitColliders.Add(localid);
3870 if (!m_lastColliders.Contains(localid))
3871 {
3872 startedColliders.Add(localid);
3873 }
3874 //m_log.Debug("[SCENE PRESENCE]: Collided with:" + localid.ToString() + " at depth of: " + collissionswith[localid].ToString());
3875 }
3876
3877 // calculate things that ended colliding
3878 foreach (uint localID in m_lastColliders)
3879 { 3867 {
3880 if (!thisHitColliders.Contains(localID)) 3868 if (m_collisionEventFlag)
3881 { 3869 return;
3882 endedColliders.Add(localID); 3870 m_collisionEventFlag = true;
3883 }
3884 }
3885 //add the items that started colliding this time to the last colliders list.
3886 foreach (uint localID in startedColliders)
3887 {
3888 m_lastColliders.Add(localID);
3889 }
3890 // remove things that ended colliding from the last colliders list
3891 foreach (uint localID in endedColliders)
3892 {
3893 m_lastColliders.Remove(localID);
3894 } 3871 }
3895 3872
3896 // do event notification 3873 Util.FireAndForget(delegate(object x)
3897 if (startedColliders.Count > 0)
3898 { 3874 {
3899 ColliderArgs StartCollidingMessage = new ColliderArgs(); 3875 try
3900 List<DetectedObject> colliding = new List<DetectedObject>();
3901 foreach (uint localId in startedColliders)
3902 { 3876 {
3903 if (localId == 0) 3877 List<uint> thisHitColliders = new List<uint>();
3904 continue; 3878 List<uint> endedColliders = new List<uint>();
3879 List<uint> startedColliders = new List<uint>();
3905 3880
3906 SceneObjectPart obj = Scene.GetSceneObjectPart(localId); 3881 foreach (uint localid in coldata.Keys)
3907 string data = "";
3908 if (obj != null)
3909 { 3882 {
3910 DetectedObject detobj = new DetectedObject(); 3883 thisHitColliders.Add(localid);
3911 detobj.keyUUID = obj.UUID; 3884 if (!m_lastColliders.Contains(localid))
3912 detobj.nameStr = obj.Name; 3885 {
3913 detobj.ownerUUID = obj.OwnerID; 3886 startedColliders.Add(localid);
3914 detobj.posVector = obj.AbsolutePosition; 3887 }
3915 detobj.rotQuat = obj.GetWorldRotation(); 3888 //m_log.Debug("[SCENE PRESENCE]: Collided with:" + localid.ToString() + " at depth of: " + collissionswith[localid].ToString());
3916 detobj.velVector = obj.Velocity; 3889 }
3917 detobj.colliderType = 0; 3890
3918 detobj.groupUUID = obj.GroupID; 3891 // calculate things that ended colliding
3919 colliding.Add(detobj); 3892 foreach (uint localID in m_lastColliders)
3893 {
3894 if (!thisHitColliders.Contains(localID))
3895 {
3896 endedColliders.Add(localID);
3897 }
3898 }
3899 //add the items that started colliding this time to the last colliders list.
3900 foreach (uint localID in startedColliders)
3901 {
3902 m_lastColliders.Add(localID);
3903 }
3904 // remove things that ended colliding from the last colliders list
3905 foreach (uint localID in endedColliders)
3906 {
3907 m_lastColliders.Remove(localID);
3920 } 3908 }
3921 }
3922 3909
3923 if (colliding.Count > 0) 3910 // do event notification
3924 { 3911 if (startedColliders.Count > 0)
3925 StartCollidingMessage.Colliders = colliding; 3912 {
3913 ColliderArgs StartCollidingMessage = new ColliderArgs();
3914 List<DetectedObject> colliding = new List<DetectedObject>();
3915 foreach (uint localId in startedColliders)
3916 {
3917 if (localId == 0)
3918 continue;
3926 3919
3927 foreach (SceneObjectGroup att in GetAttachments()) 3920 SceneObjectPart obj = Scene.GetSceneObjectPart(localId);
3928 Scene.EventManager.TriggerScriptCollidingStart(att.LocalId, StartCollidingMessage); 3921 string data = "";
3929 } 3922 if (obj != null)
3930 } 3923 {
3924 DetectedObject detobj = new DetectedObject();
3925 detobj.keyUUID = obj.UUID;
3926 detobj.nameStr = obj.Name;
3927 detobj.ownerUUID = obj.OwnerID;
3928 detobj.posVector = obj.AbsolutePosition;
3929 detobj.rotQuat = obj.GetWorldRotation();
3930 detobj.velVector = obj.Velocity;
3931 detobj.colliderType = 0;
3932 detobj.groupUUID = obj.GroupID;
3933 colliding.Add(detobj);
3934 }
3935 }
3931 3936
3932 if (endedColliders.Count > 0) 3937 if (colliding.Count > 0)
3933 { 3938 {
3934 ColliderArgs EndCollidingMessage = new ColliderArgs(); 3939 StartCollidingMessage.Colliders = colliding;
3935 List<DetectedObject> colliding = new List<DetectedObject>();
3936 foreach (uint localId in endedColliders)
3937 {
3938 if (localId == 0)
3939 continue;
3940 3940
3941 SceneObjectPart obj = Scene.GetSceneObjectPart(localId); 3941 foreach (SceneObjectGroup att in GetAttachments())
3942 string data = ""; 3942 Scene.EventManager.TriggerScriptCollidingStart(att.LocalId, StartCollidingMessage);
3943 if (obj != null) 3943 }
3944 {
3945 DetectedObject detobj = new DetectedObject();
3946 detobj.keyUUID = obj.UUID;
3947 detobj.nameStr = obj.Name;
3948 detobj.ownerUUID = obj.OwnerID;
3949 detobj.posVector = obj.AbsolutePosition;
3950 detobj.rotQuat = obj.GetWorldRotation();
3951 detobj.velVector = obj.Velocity;
3952 detobj.colliderType = 0;
3953 detobj.groupUUID = obj.GroupID;
3954 colliding.Add(detobj);
3955 } 3944 }
3956 }
3957 3945
3958 if (colliding.Count > 0) 3946 if (endedColliders.Count > 0)
3959 { 3947 {
3960 EndCollidingMessage.Colliders = colliding; 3948 ColliderArgs EndCollidingMessage = new ColliderArgs();
3949 List<DetectedObject> colliding = new List<DetectedObject>();
3950 foreach (uint localId in endedColliders)
3951 {
3952 if (localId == 0)
3953 continue;
3961 3954
3962 foreach (SceneObjectGroup att in GetAttachments()) 3955 SceneObjectPart obj = Scene.GetSceneObjectPart(localId);
3963 Scene.EventManager.TriggerScriptCollidingEnd(att.LocalId, EndCollidingMessage); 3956 string data = "";
3964 } 3957 if (obj != null)
3965 } 3958 {
3959 DetectedObject detobj = new DetectedObject();
3960 detobj.keyUUID = obj.UUID;
3961 detobj.nameStr = obj.Name;
3962 detobj.ownerUUID = obj.OwnerID;
3963 detobj.posVector = obj.AbsolutePosition;
3964 detobj.rotQuat = obj.GetWorldRotation();
3965 detobj.velVector = obj.Velocity;
3966 detobj.colliderType = 0;
3967 detobj.groupUUID = obj.GroupID;
3968 colliding.Add(detobj);
3969 }
3970 }
3966 3971
3967 if (thisHitColliders.Count > 0) 3972 if (colliding.Count > 0)
3968 { 3973 {
3969 ColliderArgs CollidingMessage = new ColliderArgs(); 3974 EndCollidingMessage.Colliders = colliding;
3970 List<DetectedObject> colliding = new List<DetectedObject>();
3971 foreach (uint localId in thisHitColliders)
3972 {
3973 if (localId == 0)
3974 continue;
3975 3975
3976 SceneObjectPart obj = Scene.GetSceneObjectPart(localId); 3976 foreach (SceneObjectGroup att in GetAttachments())
3977 string data = ""; 3977 Scene.EventManager.TriggerScriptCollidingEnd(att.LocalId, EndCollidingMessage);
3978 if (obj != null) 3978 }
3979 {
3980 DetectedObject detobj = new DetectedObject();
3981 detobj.keyUUID = obj.UUID;
3982 detobj.nameStr = obj.Name;
3983 detobj.ownerUUID = obj.OwnerID;
3984 detobj.posVector = obj.AbsolutePosition;
3985 detobj.rotQuat = obj.GetWorldRotation();
3986 detobj.velVector = obj.Velocity;
3987 detobj.colliderType = 0;
3988 detobj.groupUUID = obj.GroupID;
3989 colliding.Add(detobj);
3990 } 3979 }
3991 }
3992
3993 if (colliding.Count > 0)
3994 {
3995 CollidingMessage.Colliders = colliding;
3996 3980
3997 lock (m_attachments) 3981 if (thisHitColliders.Count > 0)
3998 { 3982 {
3999 foreach (SceneObjectGroup att in m_attachments) 3983 ColliderArgs CollidingMessage = new ColliderArgs();
4000 Scene.EventManager.TriggerScriptColliding(att.LocalId, CollidingMessage); 3984 List<DetectedObject> colliding = new List<DetectedObject>();
3985 foreach (uint localId in thisHitColliders)
3986 {
3987 if (localId == 0)
3988 continue;
3989
3990 SceneObjectPart obj = Scene.GetSceneObjectPart(localId);
3991 string data = "";
3992 if (obj != null)
3993 {
3994 DetectedObject detobj = new DetectedObject();
3995 detobj.keyUUID = obj.UUID;
3996 detobj.nameStr = obj.Name;
3997 detobj.ownerUUID = obj.OwnerID;
3998 detobj.posVector = obj.AbsolutePosition;
3999 detobj.rotQuat = obj.GetWorldRotation();
4000 detobj.velVector = obj.Velocity;
4001 detobj.colliderType = 0;
4002 detobj.groupUUID = obj.GroupID;
4003 colliding.Add(detobj);
4004 }
4005 }
4006
4007 if (colliding.Count > 0)
4008 {
4009 CollidingMessage.Colliders = colliding;
4010
4011 lock (m_attachments)
4012 {
4013 foreach (SceneObjectGroup att in m_attachments)
4014 Scene.EventManager.TriggerScriptColliding(att.LocalId, CollidingMessage);
4015 }
4016 }
4001 } 4017 }
4002 } 4018 }
4003 } 4019 finally
4020 {
4021 m_collisionEventFlag = false;
4022 }
4023 });
4004 } 4024 }
4005 } 4025 }
4006} 4026}