diff options
Diffstat (limited to 'OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs')
-rw-r--r-- | OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs | 66 |
1 files changed, 20 insertions, 46 deletions
diff --git a/OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs b/OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs index 8e32fcc..4cc0e19 100644 --- a/OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs +++ b/OpenSim/Region/CoreModules/Avatar/Friends/FriendsModule.cs | |||
@@ -57,7 +57,6 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
57 | { | 57 | { |
58 | public UUID PrincipalID; | 58 | public UUID PrincipalID; |
59 | public FriendInfo[] Friends; | 59 | public FriendInfo[] Friends; |
60 | public int Refcount; | ||
61 | 60 | ||
62 | public bool IsFriend(string friend) | 61 | public bool IsFriend(string friend) |
63 | { | 62 | { |
@@ -255,6 +254,9 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
255 | 254 | ||
256 | private void OnNewClient(IClientAPI client) | 255 | private void OnNewClient(IClientAPI client) |
257 | { | 256 | { |
257 | if (client.SceneAgent.IsChildAgent) | ||
258 | return; | ||
259 | |||
258 | client.OnInstantMessage += OnInstantMessage; | 260 | client.OnInstantMessage += OnInstantMessage; |
259 | client.OnApproveFriendRequest += OnApproveFriendRequest; | 261 | client.OnApproveFriendRequest += OnApproveFriendRequest; |
260 | client.OnDenyFriendRequest += OnDenyFriendRequest; | 262 | client.OnDenyFriendRequest += OnDenyFriendRequest; |
@@ -281,23 +283,14 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
281 | UUID agentID = client.AgentId; | 283 | UUID agentID = client.AgentId; |
282 | lock (m_Friends) | 284 | lock (m_Friends) |
283 | { | 285 | { |
284 | UserFriendData friendsData; | 286 | UserFriendData friendsData = new UserFriendData(); |
285 | if (m_Friends.TryGetValue(agentID, out friendsData)) | 287 | friendsData.PrincipalID = agentID; |
286 | { | 288 | friendsData.Friends = GetFriendsFromService(client); |
287 | friendsData.Refcount++; | ||
288 | return false; | ||
289 | } | ||
290 | else | ||
291 | { | ||
292 | friendsData = new UserFriendData(); | ||
293 | friendsData.PrincipalID = agentID; | ||
294 | friendsData.Friends = GetFriendsFromService(client); | ||
295 | friendsData.Refcount = 1; | ||
296 | 289 | ||
297 | m_Friends[agentID] = friendsData; | 290 | m_Friends[agentID] = friendsData; |
298 | return true; | ||
299 | } | ||
300 | } | 291 | } |
292 | |||
293 | return true; | ||
301 | } | 294 | } |
302 | 295 | ||
303 | private void OnClientClosed(UUID agentID, Scene scene) | 296 | private void OnClientClosed(UUID agentID, Scene scene) |
@@ -307,23 +300,17 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
307 | { | 300 | { |
308 | // do this for root agents closing out | 301 | // do this for root agents closing out |
309 | StatusChange(agentID, false); | 302 | StatusChange(agentID, false); |
310 | } | ||
311 | 303 | ||
312 | lock (m_Friends) | 304 | lock (m_Friends) |
313 | { | 305 | m_Friends.Remove(agentID); |
314 | UserFriendData friendsData; | ||
315 | if (m_Friends.TryGetValue(agentID, out friendsData)) | ||
316 | { | ||
317 | friendsData.Refcount--; | ||
318 | if (friendsData.Refcount <= 0) | ||
319 | m_Friends.Remove(agentID); | ||
320 | } | ||
321 | } | 306 | } |
322 | } | 307 | } |
323 | 308 | ||
324 | private void OnMakeRootAgent(ScenePresence sp) | 309 | private void OnMakeRootAgent(ScenePresence sp) |
325 | { | 310 | { |
326 | RecacheFriends(sp.ControllingClient); | 311 | // FIXME: Ideally, we want to avoid doing this here since it sits the EventManager.OnMakeRootAgent event |
312 | // is on the critical path for transferring an avatar from one region to another. | ||
313 | CacheFriends(sp.ControllingClient); | ||
327 | } | 314 | } |
328 | 315 | ||
329 | private void OnClientLogin(IClientAPI client) | 316 | private void OnClientLogin(IClientAPI client) |
@@ -628,8 +615,8 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
628 | ccm.CreateCallingCard(client.AgentId, friendID, UUID.Zero); | 615 | ccm.CreateCallingCard(client.AgentId, friendID, UUID.Zero); |
629 | } | 616 | } |
630 | 617 | ||
631 | // Update the local cache | 618 | // Update the local cache. |
632 | RecacheFriends(client); | 619 | CacheFriends(client); |
633 | 620 | ||
634 | // | 621 | // |
635 | // Notify the friend | 622 | // Notify the friend |
@@ -691,7 +678,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
691 | client.SendAlertMessage("Unable to terminate friendship on this sim."); | 678 | client.SendAlertMessage("Unable to terminate friendship on this sim."); |
692 | 679 | ||
693 | // Update local cache | 680 | // Update local cache |
694 | RecacheFriends(client); | 681 | CacheFriends(client); |
695 | 682 | ||
696 | client.SendTerminateFriend(exfriendID); | 683 | client.SendTerminateFriend(exfriendID); |
697 | 684 | ||
@@ -812,7 +799,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
812 | 799 | ||
813 | 800 | ||
814 | // Update the local cache | 801 | // Update the local cache |
815 | RecacheFriends(friendClient); | 802 | CacheFriends(friendClient); |
816 | 803 | ||
817 | // we're done | 804 | // we're done |
818 | return true; | 805 | return true; |
@@ -845,7 +832,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
845 | // the friend in this sim as root agent | 832 | // the friend in this sim as root agent |
846 | friendClient.SendTerminateFriend(exfriendID); | 833 | friendClient.SendTerminateFriend(exfriendID); |
847 | // update local cache | 834 | // update local cache |
848 | RecacheFriends(friendClient); | 835 | CacheFriends(friendClient); |
849 | // we're done | 836 | // we're done |
850 | return true; | 837 | return true; |
851 | } | 838 | } |
@@ -946,19 +933,6 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
946 | return FriendsService.GetFriends(client.AgentId); | 933 | return FriendsService.GetFriends(client.AgentId); |
947 | } | 934 | } |
948 | 935 | ||
949 | protected void RecacheFriends(IClientAPI client) | ||
950 | { | ||
951 | // FIXME: Ideally, we want to avoid doing this here since it sits the EventManager.OnMakeRootAgent event | ||
952 | // is on the critical path for transferring an avatar from one region to another. | ||
953 | UUID agentID = client.AgentId; | ||
954 | lock (m_Friends) | ||
955 | { | ||
956 | UserFriendData friendsData; | ||
957 | if (m_Friends.TryGetValue(agentID, out friendsData)) | ||
958 | friendsData.Friends = GetFriendsFromService(client); | ||
959 | } | ||
960 | } | ||
961 | |||
962 | /// <summary> | 936 | /// <summary> |
963 | /// Are friends cached on this simulator for a particular user? | 937 | /// Are friends cached on this simulator for a particular user? |
964 | /// </summary> | 938 | /// </summary> |
@@ -1052,4 +1026,4 @@ namespace OpenSim.Region.CoreModules.Avatar.Friends | |||
1052 | } | 1026 | } |
1053 | } | 1027 | } |
1054 | } | 1028 | } |
1055 | } \ No newline at end of file | 1029 | } |