From 60c52ac0c44a395e3bd3b3ca761ad71ba10906c2 Mon Sep 17 00:00:00 2001
From: "Justin Clark-Casey (justincc)" <jjustincc@googlemail.com>
Date: Wed, 21 Jul 2010 14:25:21 +0100
Subject: [PATCH] start adding user ids to the media urls

---
 .../World/Media/Moap/MoapModule.cs            | 28 +++++++++++--------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/OpenSim/Region/CoreModules/World/Media/Moap/MoapModule.cs b/OpenSim/Region/CoreModules/World/Media/Moap/MoapModule.cs
index cbe9af2864..6755df70d6 100644
--- a/OpenSim/Region/CoreModules/World/Media/Moap/MoapModule.cs
+++ b/OpenSim/Region/CoreModules/World/Media/Moap/MoapModule.cs
@@ -189,7 +189,7 @@ namespace OpenSim.Region.CoreModules.Media.Moap
                 part.Shape.Media = new List<MediaEntry>(new MediaEntry[part.GetNumberOfSides()]);
                         
             part.Shape.Media[face] = me;                                   
-            UpdateMediaUrl(part);                      
+            UpdateMediaUrl(part, UUID.Zero);                      
             part.ScheduleFullUpdate();
             part.TriggerScriptChangedEvent(Changed.MEDIA);
         }
@@ -300,6 +300,11 @@ namespace OpenSim.Region.CoreModules.Media.Moap
                 return string.Empty;
             }
             
+            UUID agentId = default(UUID);
+            
+            lock (m_omCapUsers)
+                agentId = m_omCapUsers[path]; 
+            
             List<MediaEntry> media = part.Shape.Media;
             
             if (null == media)
@@ -310,12 +315,7 @@ namespace OpenSim.Region.CoreModules.Media.Moap
             else
             {
                 // We need to go through the media textures one at a time to make sure that we have permission 
-                // to change them                    
-                UUID agentId = default(UUID);
-                
-                lock (m_omCapUsers)
-                    agentId = m_omCapUsers[path];                
-                
+                // to change them                                                  
                 for (int i = 0; i < media.Count; i++)
                 {
                     if (m_scene.Permissions.CanControlPrimMedia(agentId, part.UUID, i))
@@ -326,7 +326,7 @@ namespace OpenSim.Region.CoreModules.Media.Moap
                 }
             }
             
-            UpdateMediaUrl(part);                        
+            UpdateMediaUrl(part, agentId);                        
             
             // Arguably, we could avoid sending a full update to the avatar that just changed the texture.
             part.ScheduleFullUpdate();
@@ -402,13 +402,13 @@ namespace OpenSim.Region.CoreModules.Media.Moap
             
             me.CurrentURL = omn.URL;
             
-            UpdateMediaUrl(part);
+            UpdateMediaUrl(part, agentId);
             
             part.ScheduleFullUpdate();      
             
             part.TriggerScriptChangedEvent(Changed.MEDIA);
             
-            return string.Empty;
+            return OSDParser.SerializeLLSDXmlString(new OSD());
         }      
         
         /// <summary>
@@ -431,12 +431,16 @@ namespace OpenSim.Region.CoreModules.Media.Moap
         /// Update the media url of the given part
         /// </summary>
         /// <param name="part"></param>
-        protected void UpdateMediaUrl(SceneObjectPart part)
+        /// <param name="updateId">
+        /// The id to attach to this update.  Normally, this is the user that changed the
+        /// texture
+        /// </param>
+        protected void UpdateMediaUrl(SceneObjectPart part, UUID updateId)
         {
             if (null == part.MediaUrl)
             {
                 // TODO: We can't set the last changer until we start tracking which cap we give to which agent id
-                part.MediaUrl = "x-mv:0000000000/" + UUID.Zero;
+                part.MediaUrl = "x-mv:0000000000/" + updateId;
             }
             else
             {