diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectGroup.cs b/OpenSim/Region/Framework/Scenes/SceneObjectGroup.cs index 3fa6bb086d..4355394952 100644 --- a/OpenSim/Region/Framework/Scenes/SceneObjectGroup.cs +++ b/OpenSim/Region/Framework/Scenes/SceneObjectGroup.cs @@ -1873,7 +1873,7 @@ namespace OpenSim.Region.Framework.Scenes } /// - /// Send metadata about the root prim (name, description, sale price, etc.) to a client. + /// Send metadata about the root prim (name, description, sale price, permissions, etc.) to a client. /// /// public void SendPropertiesToClient(IClientAPI client) @@ -2481,6 +2481,11 @@ namespace OpenSim.Region.Framework.Scenes parts[i].UpdatePermissions(AgentID, field, localID, mask, addRemTF); HasGroupChanged = true; + + // Send the group's properties to all clients once all parts are updated + IClientAPI client; + if (Scene.TryGetClient(AgentID, out client)) + SendPropertiesToClient(client); } #endregion diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs index 407115962f..ad60604993 100644 --- a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs +++ b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs @@ -1338,22 +1338,6 @@ namespace OpenSim.Region.Framework.Scenes client.SendObjectPropertiesReply(this); } - /// - /// For the scene object group to which this part belongs, send that scene object's root part properties to a client. - /// - /// - private void SendRootPartPropertiesToClient(UUID AgentID) - { - m_parentGroup.Scene.ForEachClient(delegate(IClientAPI client) - { - // Ugly reference :( - if (client.AgentId == AgentID) - { - m_parentGroup.SendPropertiesToClient(client); - } - }); - } - // TODO: unused: // private void handleTimerAccounting(uint localID, double interval) // { @@ -4214,6 +4198,15 @@ namespace OpenSim.Region.Framework.Scenes } } + /// + /// Update permissions on the SOP. Should only be called from SOG.UpdatePermissions because the SOG + /// will handle the client notifications once all of its parts are updated. + /// + /// + /// + /// + /// + /// public void UpdatePermissions(UUID AgentID, byte field, uint localID, uint mask, byte addRemTF) { bool set = addRemTF == 1; @@ -4262,8 +4255,6 @@ namespace OpenSim.Region.Framework.Scenes } SendFullUpdateToAllClients(); - - SendRootPartPropertiesToClient(AgentID); } }