diff options
-rw-r--r-- | OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs index 9969a15..e959821 100644 --- a/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs +++ b/OpenSim/Region/OptionalModules/Avatar/XmlRpcGroups/GroupsModule.cs | |||
@@ -332,15 +332,20 @@ namespace OpenSim.Region.OptionalModules.Avatar.XmlRpcGroups | |||
332 | private void HandleObjectGroupUpdate( | 332 | private void HandleObjectGroupUpdate( |
333 | IClientAPI remoteClient, UUID GroupID, uint objectLocalID, UUID Garbage) | 333 | IClientAPI remoteClient, UUID GroupID, uint objectLocalID, UUID Garbage) |
334 | { | 334 | { |
335 | GroupMembershipData gmd = GetMembershipData(GroupID, remoteClient.AgentId); | 335 | // XXX: Might be better to get rid of this special casing and have GetMembershipData return something |
336 | 336 | // reasonable for a UUID.Zero group. | |
337 | if (gmd == null) | 337 | if (GroupID != UUID.Zero) |
338 | { | 338 | { |
339 | // m_log.WarnFormat( | 339 | GroupMembershipData gmd = GetMembershipData(GroupID, remoteClient.AgentId); |
340 | // "[GROUPS]: User {0} is not a member of group {1} so they can't update {2} to this group", | 340 | |
341 | // remoteClient.Name, GroupID, objectLocalID); | 341 | if (gmd == null) |
342 | 342 | { | |
343 | return; | 343 | // m_log.WarnFormat( |
344 | // "[GROUPS]: User {0} is not a member of group {1} so they can't update {2} to this group", | ||
345 | // remoteClient.Name, GroupID, objectLocalID); | ||
346 | |||
347 | return; | ||
348 | } | ||
344 | } | 349 | } |
345 | 350 | ||
346 | SceneObjectGroup so = ((Scene)remoteClient.Scene).GetGroupByPrim(objectLocalID); | 351 | SceneObjectGroup so = ((Scene)remoteClient.Scene).GetGroupByPrim(objectLocalID); |