Fire detach events even in temp attachments. They don't really need it but keeping the tests happy

1 files changed, 9 insertions, 1 deletions

diff --git a/OpenSim/Region/CoreModules/Avatar/Attachments/AttachmentsModule.cs b/OpenSim/Region/CoreModules/Avatar/Attachments/AttachmentsModule.cs
index 3d90fa6..3b13e3f 100644
--- a/OpenSim/Region/CoreModules/Avatar/Attachments/AttachmentsModule.cs
+++ b/OpenSim/Region/CoreModules/Avatar/Attachments/AttachmentsModule.cs
@@ -779,7 +779,15 @@ namespace OpenSim.Region.CoreModules.Avatar.Attachments
             // on detach. It's likely a temp attachment.
             if (so.FromItemID != UUID.Zero)
             {
-                sp.RemoveAttachment(so);
+                lock (sp.AttachmentsSyncLock)
+                {
+                    bool changed = sp.Appearance.DetachAttachment(so.FromItemID);
+                    if (changed && m_scene.AvatarFactory != null)
+                        m_scene.AvatarFactory.QueueAppearanceSave(sp.UUID);
+
+                    sp.RemoveAttachment(so);
+                }
+
                 m_scene.DeleteSceneObject(so, false, false);
                 so.RemoveScriptInstances(true);
                 so.Clear();