* Assign incoming items with a random UUID so that archives can be loaded more than once

* Also remove a duplicate write archive call in the unit test which might be causing test failures for people using mono 2.2 (though not 1.9.1, it would seem)

1 files changed, 4 insertions, 11 deletions

diff --git a/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/InventoryArchiveReadRequest.cs b/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/InventoryArchiveReadRequest.cs
index 6b57ac5..dfa4100 100644
--- a/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/InventoryArchiveReadRequest.cs
+++ b/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/InventoryArchiveReadRequest.cs
@@ -104,17 +104,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Inventory.Archiver
             reader.ReadStartElement("Owner");
             item.Owner = UUID.Parse(reader.ReadString());
             reader.ReadEndElement();
-            //No description would kill it
-            if (reader.IsEmptyElement)
-            {
-                reader.ReadStartElement("Description");
-            }
-            else
-            {
-                reader.ReadStartElement("Description");
-                item.Description = reader.ReadString();
-                reader.ReadEndElement();
-            }
+            reader.ReadElementString("Description");
             reader.ReadStartElement("AssetType");
             item.AssetType = Convert.ToInt32(reader.ReadString());
             reader.ReadEndElement();
@@ -225,6 +215,9 @@ namespace OpenSim.Region.CoreModules.Avatar.Inventory.Archiver
 
                     if (item != null)
                     {
+                        // Don't use the item ID that's in the file
+                        item.ID = UUID.Random();
+                        
                         item.Creator = m_userInfo.UserProfile.ID;
                         item.Owner = m_userInfo.UserProfile.ID;