* Refactored SOP.FolderID weirdness by removing calls to empty setter. YEs, I do realize the setter has to be there for legacy reasons, but since the calls will never acually DO anyhting, I'm removing them.

* So, SOP.FolderID is actually a cruft field that should be removed.

1 files changed, 3 insertions, 1 deletions

diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
index 1470191..5b0480f 100644
--- a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
@@ -375,13 +375,15 @@ namespace OpenSim.Region.Framework.Scenes
         }
 
         /// <summary>
+        /// A relic from when we we thought that prims contained folder objects. In 
+        /// reality, prim == folder  
         /// Exposing this is not particularly good, but it's one of the least evils at the moment to see
         /// folder id from prim inventory item data, since it's not (yet) actually stored with the prim.
         /// </summary>
         public UUID FolderID
         {
             get { return UUID; }
-            set { } // Don't allow assignment, or legacy prims wil b0rk
+            set { } // Don't allow assignment, or legacy prims wil b0rk - but we need the setter for legacy serialization.
         }
 
         /// <value>