1 files changed, 39 insertions, 28 deletions

diff --git a/OpenSim/Region/Environment/Scenes/SceneObjectGroup.cs b/OpenSim/Region/Environment/Scenes/SceneObjectGroup.cs
index 02098b7..32c9255 100644
--- a/OpenSim/Region/Environment/Scenes/SceneObjectGroup.cs
+++ b/OpenSim/Region/Environment/Scenes/SceneObjectGroup.cs
@@ -1830,29 +1830,42 @@ namespace OpenSim.Region.Environment.Scenes
             lock (m_parts)
             {
                 m_parts.Add(linkPart.UUID, linkPart);
-            }
-
-            linkPart.LinkNum = m_parts.Count;
-
-            linkPart.SetParent(this);
-            linkPart.AddFlag(PrimFlags.CreateSelected);
-
-            //if (linkPart.PhysActor != null)
-            //{
-            // m_scene.PhysicsScene.RemovePrim(linkPart.PhysActor);
 
-            //linkPart.PhysActor = null;
-            //}
-
-            //TODO: rest of parts
-            foreach (SceneObjectPart part in objectGroup.Children.Values)
-            {
-                if (part.UUID != objectGroup.m_rootPart.UUID)
-                {
-                    LinkNonRootPart(part, oldGroupPosition, oldRootRotation);
-                }
-                part.ClearUndoState();
-            }
+                // Insert in terms of link numbers, the new links
+                // before the current ones (with the exception of 
+                // the root prim. Shuffle the old ones up
+                foreach (KeyValuePair<UUID, SceneObjectPart> kvp in m_parts) 
+                {
+                    if (kvp.Value.LinkNum != 1) {
+                        // Don't update root prim link number
+                        kvp.Value.LinkNum += objectGroup.PrimCount;
+                    }
+                }
+
+
+                linkPart.LinkNum = 2;
+
+                linkPart.SetParent(this);
+                linkPart.AddFlag(PrimFlags.CreateSelected);
+
+                //if (linkPart.PhysActor != null)
+                //{
+                // m_scene.PhysicsScene.RemovePrim(linkPart.PhysActor);
+                
+                //linkPart.PhysActor = null;
+                //}
+
+                //TODO: rest of parts
+                int linkNum = 3;
+                foreach (SceneObjectPart part in objectGroup.Children.Values)
+                {
+                    if (part.UUID != objectGroup.m_rootPart.UUID)
+                    {
+                        LinkNonRootPart(part, oldGroupPosition, oldRootRotation, linkNum++);
+                    }
+                    part.ClearUndoState();
+                }
+            }
 
             m_scene.UnlinkSceneObject(objectGroup.UUID, true);
             objectGroup.Children.Clear();
@@ -1961,17 +1974,15 @@ namespace OpenSim.Region.Environment.Scenes
             m_isBackedUp = false;
         }
 
-        private void LinkNonRootPart(SceneObjectPart part, Vector3 oldGroupPosition, Quaternion oldGroupRotation)
+        private void LinkNonRootPart(SceneObjectPart part, Vector3 oldGroupPosition, Quaternion oldGroupRotation, int linkNum)
         {
             part.SetParent(this);
             part.ParentID = m_rootPart.LocalId;
 
-            lock (m_parts)
-            {
-                m_parts.Add(part.UUID, part);
-            }
+            // Caller locks m_parts for us
+            m_parts.Add(part.UUID, part);
 
-            part.LinkNum = m_parts.Count;
+            part.LinkNum = linkNum;
 
             Vector3 oldPos = part.OffsetPosition;
             oldPos *= oldGroupRotation;