2 files changed, 50 insertions, 46 deletions

diff --git a/OpenSim/Framework/Communications/OspResolver.cs b/OpenSim/Framework/Communications/OspResolver.cs
index 924a4a9..073abf8 100644
--- a/OpenSim/Framework/Communications/OspResolver.cs
+++ b/OpenSim/Framework/Communications/OspResolver.cs
@@ -50,7 +50,22 @@ namespace OpenSim.Framework.Communications
         public const string OSPA_PAIR_SEPARATOR = "=";
 
         /// <summary>
-        /// Make an OSPA given an avatar name
+        /// Make an OSPA given a user UUID
+        /// </summary>
+        /// <param name="userId"></param>
+        /// <param name="commsManager"></param>
+        /// <returns>The OSPA.  Null if a user with the given UUID could not be found.</returns>
+        public static string MakeOspa(UUID userId, CommunicationsManager commsManager)
+        {
+            CachedUserInfo userInfo = commsManager.UserProfileCacheService.GetUserDetails(userId);
+            if (userInfo != null)
+                return MakeOspa(userInfo.UserProfile.FirstName, userInfo.UserProfile.SurName);
+
+            return null;
+        }
+        
+        /// <summary>
+        /// Make an OSPA given a user name
         /// </summary>
         /// <param name="name"></param>
         /// <returns></returns>
@@ -58,7 +73,7 @@ namespace OpenSim.Framework.Communications
         {
             return 
                 OSPA_PREFIX + OSPA_NAME_KEY + OSPA_PAIR_SEPARATOR + firstName + OSPA_NAME_VALUE_SEPARATOR + lastName;
-        }
+        }        
         
         /// <summary>
         /// Resolve an osp string into the most suitable internal OpenSim identifier.
@@ -74,7 +89,7 @@ namespace OpenSim.Framework.Communications
         /// return that same string.  If the input string was ospi data but no valid profile information has been found,
         /// then returns null.
         /// </returns>
-        public static string Resolve(string ospa, CommunicationsManager commsManager)
+        public static string ResolveOspa(string ospa, CommunicationsManager commsManager)
         {
             m_log.DebugFormat("[OSP RESOLVER]: Resolving {0}", ospa);
             
diff --git a/OpenSim/Framework/InventoryItemBase.cs b/OpenSim/Framework/InventoryItemBase.cs
index b14ce7f..29142af 100644
--- a/OpenSim/Framework/InventoryItemBase.cs
+++ b/OpenSim/Framework/InventoryItemBase.cs
@@ -35,14 +35,14 @@ namespace OpenSim.Framework
     /// </summary>
     public class InventoryItemBase : InventoryNodeBase, ICloneable
     {       
-        /// <summary>
+        /// <value>
         /// The inventory type of the item.  This is slightly different from the asset type in some situations.
-        /// </summary>        
+        /// </value>        
         public int InvType;
 
-        /// <summary>
+        /// <value>
         /// The folder this item is contained in
-        /// </summary>        
+        /// </value>        
         public UUID Folder;
 
         /// <value>
@@ -69,9 +69,9 @@ namespace OpenSim.Framework
         /// </value>
         public UUID CreatorIdAsUuid { get; private set; }       
 
-        /// <summary>
+        /// <value>
         /// The description of the inventory item (must be less than 64 characters)
-        /// </summary>
+        /// </value>
         public string Description = String.Empty;
 
         /// <value>
@@ -79,80 +79,69 @@ namespace OpenSim.Framework
         /// </value>          
         public uint NextPermissions;
 
-        /// <summary>
+        /// <value>
         /// A mask containing permissions for the current owner (cannot be enforced)
-        /// </summary>        
+        /// </value>        
         public uint CurrentPermissions;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public uint BasePermissions;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public uint EveryOnePermissions;
         
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public uint GroupPermissions;
 
-        /// <summary>
+        /// <value>
         /// This is an enumerated value determining the type of asset (eg Notecard, Sound, Object, etc)
-        /// </summary>        
+        /// </value>        
         public int AssetType;
 
-        /// <summary>
+        /// <value>
         /// The UUID of the associated asset on the asset server
-        /// </summary>        
+        /// </value>        
         public UUID AssetID;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public UUID GroupID;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public bool GroupOwned;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public int SalePrice;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public byte SaleType;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
+        /// </value>        
         public uint Flags;
 
-        /// <summary>
+        /// <value>
         ///
-        /// </summary>        
-        public int CreationDate;
-        
-        public InventoryItemBase()
-        {
-            CreationDate = (int)(DateTime.UtcNow - new DateTime(1970, 1, 1)).TotalSeconds;
-        }
+        /// </value>        
+        public int CreationDate = (int)(DateTime.UtcNow - new DateTime(1970, 1, 1)).TotalSeconds;
                 
                 public object Clone()
                 {
-                        InventoryItemBase clone = new InventoryItemBase();
-                        clone.AssetID = AssetID;
-                        clone.AssetType = AssetType;
-                        clone.BasePermissions = BasePermissions;
-                        clone.CreationDate = CreationDate;
-                        clone.CreatorId = CreatorId;
-                        
+                        return MemberwiseClone();
                 }
     }
 }