* ModuleLoader: Privatized some too-public fields

* Scene: Changed name from MakeAvatarPhysical to MakeRootAgent and added ForEachClient
* SceneManager: Added ForEachScene
* Worked some on appearances.

3 files changed, 55 insertions, 42 deletions

diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index e86562d..1f50e1b 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -1134,7 +1134,7 @@ namespace OpenSim.Region.Environment.Scenes
             {
                 if (m_scenePresences.ContainsKey(agentID))
                 {
-                    m_scenePresences[agentID].MakeAvatarPhysical(position, isFlying);
+                    m_scenePresences[agentID].MakeRootAgent(position, isFlying);
                 }
             }
         }
@@ -1570,5 +1570,13 @@ namespace OpenSim.Region.Environment.Scenes
             avatar = null;
             return false;
         }
+
+        internal void ForEachClient( Action<IClientAPI> action )
+        {
+            foreach (ScenePresence presence in m_scenePresences.Values )
+            {
+                action(presence.ControllingClient);
+            }
+        }
     }
 }
diff --git a/OpenSim/Region/Environment/Scenes/SceneManager.cs b/OpenSim/Region/Environment/Scenes/SceneManager.cs
index 76ff6cf..e844462 100644
--- a/OpenSim/Region/Environment/Scenes/SceneManager.cs
+++ b/OpenSim/Region/Environment/Scenes/SceneManager.cs
@@ -310,5 +310,10 @@ namespace OpenSim.Region.Environment.Scenes
             avatar = null;
             return false;
         }
+
+        public void ForEachScene(Action<Scene> action )
+        {
+            m_localScenes.ForEach( action );
+        }
     }
 }
diff --git a/OpenSim/Region/Environment/Scenes/ScenePresence.cs b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
index 716aaa6..9b6e58d 100644
--- a/OpenSim/Region/Environment/Scenes/ScenePresence.cs
+++ b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
@@ -248,8 +248,8 @@ namespace OpenSim.Region.Environment.Scenes
             Animations.LoadAnims();
 
             //register for events
-            m_controllingClient.OnRequestWearables += SendOurAppearance;
-            m_controllingClient.OnSetAppearance += new SetAppearance(SetAppearance);
+            m_controllingClient.OnRequestWearables += SendAppearance;
+            m_controllingClient.OnSetAppearance += SetAppearance;
             m_controllingClient.OnCompleteMovementToRegion += CompleteMovement;
             m_controllingClient.OnCompleteMovementToRegion += SendInitialData;
             m_controllingClient.OnAgentUpdate += HandleAgentUpdate;
@@ -343,7 +343,7 @@ namespace OpenSim.Region.Environment.Scenes
 
         #region Status Methods
 
-        public void MakeAvatarPhysical(LLVector3 pos, bool isFlying)
+        public void MakeRootAgent(LLVector3 pos, bool isFlying)
         {
             m_newAvatar = true;
             m_isChildAgent = false;
@@ -412,7 +412,7 @@ namespace OpenSim.Region.Environment.Scenes
                 m_visualParams[i] = visualParam[i].ParamValue;
             }
 
-            SendAppearanceToAllOtherAgents();
+            SendAppearanceToAllOtherClients();
         }
 
         /// <summary>
@@ -433,7 +433,7 @@ namespace OpenSim.Region.Environment.Scenes
                 m_isChildAgent = false;
 
                 //this.m_scene.SendAllSceneObjectsToClient(this.ControllingClient);
-                this.MakeAvatarPhysical(this.AbsolutePosition, false);
+                this.MakeRootAgent(this.AbsolutePosition, false);
             }
         }
 
@@ -707,16 +707,19 @@ namespace OpenSim.Region.Environment.Scenes
 
         public void SendFullUpdateToAllClients()
         {
-            List<ScenePresence> avatars = m_scene.GetScenePresences();
-            foreach (ScenePresence avatar in avatars)
+            List<ScenePresence> agents = m_scene.GetScenePresences();
+            foreach (ScenePresence agent in agents)
             {
-                SendFullUpdateToOtherClient(avatar);
-                if (avatar.LocalId != LocalId)
+                IClientAPI client = agent.ControllingClient;
+                client.SendAvatarData(m_regionInfo.RegionHandle, m_firstname, m_lastname, m_uuid,
+                                                          LocalId, AbsolutePosition, m_textureEntry.ToBytes(), m_parentID );
+
+                if (agent.LocalId != LocalId)
                 {
-                    if (!avatar.m_isChildAgent)
-                    {
-                        avatar.SendFullUpdateToOtherClient(this);
-                        avatar.SendAppearanceToOtherAgent(this);
+                    if (!agent.m_isChildAgent)
+                    {                     
+                        client.SendAppearance(m_controllingClient.AgentId, m_visualParams,
+                                                        m_textureEntry.ToBytes());
                     }
                 }
             }
@@ -736,22 +739,28 @@ namespace OpenSim.Region.Environment.Scenes
             }
 
             SendFullUpdateToAllClients();
-            SendAppearanceToAllOtherAgents();
+            SendAppearanceToAllOtherClients();
         }
 
-        /// <summary>
-        /// 
-        /// </summary>
-        /// <param name="OurClient"></param>
-        public void SendOurAppearance(IClientAPI OurClient)
+        public void SetWearable( int wearableId, AvatarWearable wearable )
+        {
+            m_wearables[wearableId] = wearable;
+
+            m_scene.ForEachClient( delegate( IClientAPI client )
+                                       {
+                                           SendAppearance( client );
+                                       });            
+        }
+
+        public void SendAppearance(IClientAPI client)
         {
-            m_controllingClient.SendWearables(m_wearables);
+            client.SendWearables(m_wearables);
 
             //this.SendFullUpdateToAllClients();
-            //this.SendAppearanceToAllOtherAgents();
+            //this.SendAppearanceToAllOtherClients();
 
             m_scene.SendAllSceneObjectsToClient(this);
-            m_controllingClient.SendViewerTime(m_scene.TimePhase);
+            client.SendViewerTime(m_scene.TimePhase);
 
             //Please don't remove the following code (at least not yet), just leave it commented out
             //gives the user god powers, should help with debuging things in the future
@@ -767,25 +776,16 @@ namespace OpenSim.Region.Environment.Scenes
         /// <summary>
         /// 
         /// </summary>
-        public void SendAppearanceToAllOtherAgents()
-        {
-            m_scene.ForEachScenePresence(delegate(ScenePresence scenePresence)
-                                             {
-                                                 if (scenePresence != this)
-                                                 {
-                                                     SendAppearanceToOtherAgent(scenePresence);
-                                                 }
-                                             });
-        }
-
-        /// <summary>
-        /// 
-        /// </summary>
-        /// <param name="avatarInfo"></param>
-        public void SendAppearanceToOtherAgent(ScenePresence avatarInfo)
-        {
-            avatarInfo.m_controllingClient.SendAppearance(m_controllingClient.AgentId, m_visualParams,
-                                                        m_textureEntry.ToBytes());
+        public void SendAppearanceToAllOtherClients()
+        {
+            m_scene.ForEachClient(delegate(IClientAPI client)
+                                      {
+                                          if( client != m_controllingClient )
+                                          {
+                                              client.SendAppearance(m_controllingClient.AgentId, m_visualParams,
+                                                        m_textureEntry.ToBytes() );
+                                          }
+                                      });
         }
 
         /// <summary>