diff options
author | Dan Lake | 2011-11-03 17:06:08 -0700 |
---|---|---|
committer | Dan Lake | 2011-11-03 17:06:08 -0700 |
commit | 94dc7d07ebc22ce0e0d9b77e91538ddc90799bee (patch) | |
tree | 0d2ffc74fa937af0ca5d9e6fb2fafeac2c37dd61 /OpenSim/Region/CoreModules/World/Estate | |
parent | remove the pointless check of the face texture struct against null in Bot.Obj... (diff) | |
download | opensim-SC-94dc7d07ebc22ce0e0d9b77e91538ddc90799bee.zip opensim-SC-94dc7d07ebc22ce0e0d9b77e91538ddc90799bee.tar.gz opensim-SC-94dc7d07ebc22ce0e0d9b77e91538ddc90799bee.tar.bz2 opensim-SC-94dc7d07ebc22ce0e0d9b77e91538ddc90799bee.tar.xz |
Renamed ForEachRootScenePresence to ForEachAvatar. Cleaned up calls to
the 3 iteration functions so more of them are using the correct
iteration for the action they are performing. The 3 iterators that seem
to fit all actions within OpenSim at this time are:
ForEachAvatar: Perform an action on all avatars (root presences)
ForEachClient: Perform an action on all clients (root or child clients)
ForEachRootClient: Perform an action on all clients that have an avatar
There are still a dozen places or so calling the old
ForEachScenePresence that will take a little more refactoring to
eliminate.
Diffstat (limited to 'OpenSim/Region/CoreModules/World/Estate')
-rw-r--r-- | OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs b/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs index 5427b68..58d9455 100644 --- a/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs +++ b/OpenSim/Region/CoreModules/World/Estate/EstateManagementModule.cs | |||
@@ -658,14 +658,14 @@ namespace OpenSim.Region.CoreModules.World.Estate | |||
658 | if (!Scene.Permissions.CanIssueEstateCommand(remover_client.AgentId, false)) | 658 | if (!Scene.Permissions.CanIssueEstateCommand(remover_client.AgentId, false)) |
659 | return; | 659 | return; |
660 | 660 | ||
661 | Scene.ForEachRootScenePresence(delegate(ScenePresence sp) | 661 | Scene.ForEachRootClient(delegate(IClientAPI client) |
662 | { | 662 | { |
663 | if (sp.UUID != senderID) | 663 | if (client.AgentId != senderID) |
664 | { | 664 | { |
665 | // make sure they are still there, we could be working down a long list | 665 | // make sure they are still there, we could be working down a long list |
666 | // Also make sure they are actually in the region | 666 | // Also make sure they are actually in the region |
667 | ScenePresence p; | 667 | ScenePresence p; |
668 | if(Scene.TryGetScenePresence(sp.UUID, out p)) | 668 | if(Scene.TryGetScenePresence(client.AgentId, out p)) |
669 | Scene.TeleportClientHome(p.UUID, p.ControllingClient); | 669 | Scene.TeleportClientHome(p.UUID, p.ControllingClient); |
670 | } | 670 | } |
671 | }); | 671 | }); |
@@ -927,9 +927,9 @@ namespace OpenSim.Region.CoreModules.World.Estate | |||
927 | 927 | ||
928 | public void sendRegionInfoPacketToAll() | 928 | public void sendRegionInfoPacketToAll() |
929 | { | 929 | { |
930 | Scene.ForEachRootScenePresence(delegate(ScenePresence sp) | 930 | Scene.ForEachRootClient(delegate(IClientAPI client) |
931 | { | 931 | { |
932 | HandleRegionInfoRequest(sp.ControllingClient); | 932 | HandleRegionInfoRequest(client); |
933 | }); | 933 | }); |
934 | } | 934 | } |
935 | 935 | ||