From 442a938a3f7fa09f9aa00cdebcb0f1fbe9088df3 Mon Sep 17 00:00:00 2001
From: Justin Clarke Casey
Date: Fri, 27 Jun 2008 18:18:44 +0000
Subject: * fix a bug where I was assuming all entities would be
 SceneObjectGroups - of course, some are ScenePresences * would be nice to
 have a separate list in InnerScene in the future since a lot of methods end
 up screen entities for SOGs

---
 .../World/Archiver/ArchiveWriteRequestPreparation.cs  | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

(limited to 'OpenSim/Region/Environment/Modules/World/Archiver/ArchiveWriteRequestPreparation.cs')

diff --git a/OpenSim/Region/Environment/Modules/World/Archiver/ArchiveWriteRequestPreparation.cs b/OpenSim/Region/Environment/Modules/World/Archiver/ArchiveWriteRequestPreparation.cs
index 5d63ce4..bed30b0 100644
--- a/OpenSim/Region/Environment/Modules/World/Archiver/ArchiveWriteRequestPreparation.cs
+++ b/OpenSim/Region/Environment/Modules/World/Archiver/ArchiveWriteRequestPreparation.cs
@@ -163,24 +163,31 @@ namespace OpenSim.Region.Environment.Modules.World.Archiver
             Dictionary<LLUUID, int> assetUuids = new Dictionary<LLUUID, int>();
 
             List<EntityBase> entities = m_scene.GetEntities();
+            List<SceneObjectGroup> sceneObjects = new List<SceneObjectGroup>();
 
+            // Filter entities so that we only have scene objects.
+            // FIXME: Would be nicer to have this as a proper list in InnerScene, since lots of methods
+            // end up having to do this
             foreach (EntityBase entity in entities)
             {
                 if (entity is SceneObjectGroup)
-                {
-                    GetSceneObjectAssetUuids((SceneObjectGroup)entity, assetUuids);
-                }
+                    sceneObjects.Add((SceneObjectGroup)entity);
+            }
+            
+            foreach (SceneObjectGroup sceneObject in sceneObjects)
+            {
+                GetSceneObjectAssetUuids(sceneObject, assetUuids);
             }
 
-            if (entities.Count > 0)
+            if (sceneObjects.Count > 0)
             {
-                m_log.DebugFormat("[ARCHIVER]: Successfully got serialization for {0} entities", entities.Count);
+                m_log.DebugFormat("[ARCHIVER]: Successfully got serialization for {0} scene objects", sceneObjects.Count);
                 m_log.DebugFormat("[ARCHIVER]: Requiring save of {0} assets", assetUuids.Count);
 
                 // Asynchronously request all the assets required to perform this archive operation
                 ArchiveWriteRequestExecution awre 
                     = new ArchiveWriteRequestExecution(
-                        entities, m_scene.RequestModuleInterface<IRegionSerialiser>(), m_savePath);
+                        sceneObjects, m_scene.RequestModuleInterface<IRegionSerialiser>(), m_savePath);
                 new AssetsRequest(assetUuids.Keys, m_scene.AssetCache, awre.ReceivedAllAssets).Execute();
             }
         }
-- 
cgit v1.1