Correctly handle group owned land in the Datasnapshot module. Will return

owner uuid = grou uuid ans owner name = group anme for group land now.
Group name is now filled in

1 files changed, 28 insertions, 16 deletions

diff --git a/OpenSim/Region/DataSnapshot/LandSnapshot.cs b/OpenSim/Region/DataSnapshot/LandSnapshot.cs
index bc88a45..7235f2b 100644
--- a/OpenSim/Region/DataSnapshot/LandSnapshot.cs
+++ b/OpenSim/Region/DataSnapshot/LandSnapshot.cs
@@ -220,6 +220,8 @@ namespace OpenSim.Region.DataSnapshot.Providers
                             xmlparcel.AppendChild(textureuuid);
                         }
 
+                        string groupName = String.Empty;
+
                         //attached user and group
                         if (parcel.GroupID != UUID.Zero)
                         {
@@ -228,22 +230,31 @@ namespace OpenSim.Region.DataSnapshot.Providers
                             groupuuid.InnerText = parcel.GroupID.ToString();
                             groupblock.AppendChild(groupuuid);
 
-                            //No name yet, there's no way to get a group name since they don't exist yet.
-                            //TODO: When groups are supported, add the group handling code.
+                            IGroupsModule gm = m_scene.RequestModuleInterface<IGroupsModule>();
+                            if (gm != null)
+                            {
+                                GroupRecord g = gm.GetGroupRecord(parcel.GroupID);
+                                if (g != null)
+                                    groupName = g.GroupName;
+                            }
+
+                            XmlNode groupname = nodeFactory.CreateNode(XmlNodeType.Element, "groupname", "");
+                            groupname.InnerText = groupName;
+                            groupblock.AppendChild(groupname);
 
                             xmlparcel.AppendChild(groupblock);
                         }
 
-                        if (!parcel.IsGroupOwned)
-                        {
-                            XmlNode userblock = nodeFactory.CreateNode(XmlNodeType.Element, "owner", "");
+                        XmlNode userblock = nodeFactory.CreateNode(XmlNodeType.Element, "owner", "");
 
-                            UUID userOwnerUUID = parcel.OwnerID;
+                        UUID userOwnerUUID = parcel.OwnerID;
 
-                            XmlNode useruuid = nodeFactory.CreateNode(XmlNodeType.Element, "uuid", "");
-                            useruuid.InnerText = userOwnerUUID.ToString();
-                            userblock.AppendChild(useruuid);
+                        XmlNode useruuid = nodeFactory.CreateNode(XmlNodeType.Element, "uuid", "");
+                        useruuid.InnerText = userOwnerUUID.ToString();
+                        userblock.AppendChild(useruuid);
 
+                        if (!parcel.IsGroupOwned)
+                        {
                             try
                             {
                                 XmlNode username = nodeFactory.CreateNode(XmlNodeType.Element, "name", "");
@@ -256,14 +267,15 @@ namespace OpenSim.Region.DataSnapshot.Providers
                                 //m_log.Info("[DATASNAPSHOT]: Cannot find owner name; ignoring this parcel");
                             }
 
-                            xmlparcel.AppendChild(userblock);
                         }
-                        //else
-                        //{
-                        //    XmlAttribute type = (XmlAttribute)nodeFactory.CreateNode(XmlNodeType.Attribute, "type", "");
-                        //    type.InnerText = "owner";
-                        //    groupblock.Attributes.Append(type);
-                        //}
+                        else
+                        {
+                            XmlNode username = nodeFactory.CreateNode(XmlNodeType.Element, "name", "");
+                            username.InnerText = groupName;
+                            userblock.AppendChild(username);
+                        }
+
+                        xmlparcel.AppendChild(userblock);
 
                         parent.AppendChild(xmlparcel);
                     }