another possible cause of some of the inventory wierdness is the 1/2 implemented

OSP resolver, and the caching of the uuid seperate from the string that is a UUID.
Change this behavior back to something that ensures the data for the 2 is the same.
Put the 2 unit tests that depend on the new behavior into ignore state.

1 files changed, 5 insertions, 9 deletions

diff --git a/OpenSim/Framework/InventoryItemBase.cs b/OpenSim/Framework/InventoryItemBase.cs
index d14ce23..2968712 100644
--- a/OpenSim/Framework/InventoryItemBase.cs
+++ b/OpenSim/Framework/InventoryItemBase.cs
@@ -84,8 +84,8 @@ namespace OpenSim.Framework
                 m_creatorId = value;
             }
         }
-        protected string m_creatorId;
 
+        protected string m_creatorId = UUID.Zero.ToString();
         /// <value>
         /// The creator of this item expressed as a UUID.  Database plugins don't need to set this, it will be set by
         /// upstream code (or set by the get accessor if left unset).
@@ -94,20 +94,16 @@ namespace OpenSim.Framework
         {
             get
             {
-                if (UUID.Zero == m_creatorIdAsUuid)
-                {
-                    UUID.TryParse(CreatorId, out m_creatorIdAsUuid);
-                }
-
-                return m_creatorIdAsUuid;
+                UUID temp = UUID.Zero;
+                UUID.TryParse(CreatorId, out temp);
+                return temp;
             }
             
             set
             {
-                m_creatorIdAsUuid = value;
+                CreatorId = value.ToString();
             }
         }     
-        protected UUID m_creatorIdAsUuid = UUID.Zero;
 
         /// <value>
         /// The description of the inventory item (must be less than 64 characters)