recover PermissionsUtil.ApplyFoldedPermissions (well my version). its use easys code readability

2 files changed, 13 insertions, 22 deletions

diff --git a/OpenSim/Region/CoreModules/Framework/InventoryAccess/InventoryAccessModule.cs b/OpenSim/Region/CoreModules/Framework/InventoryAccess/InventoryAccessModule.cs
index d916cc2..bce0610 100644
--- a/OpenSim/Region/CoreModules/Framework/InventoryAccess/InventoryAccessModule.cs
+++ b/OpenSim/Region/CoreModules/Framework/InventoryAccess/InventoryAccessModule.cs
@@ -573,7 +573,7 @@ namespace OpenSim.Region.CoreModules.Framework.InventoryAccess
             InventoryItemBase item, SceneObjectGroup so, List<SceneObjectGroup> objsForEffectivePermissions,
             IClientAPI remoteClient)
         {
-            uint effectivePerms = (uint)(PermissionMask.Copy | PermissionMask.Transfer | PermissionMask.Modify | PermissionMask.Move | PermissionMask.Export) | 7;
+            uint effectivePerms = (uint)(PermissionMask.Copy | PermissionMask.Transfer | PermissionMask.Modify | PermissionMask.Move | PermissionMask.Export | PermissionMask.FoldedMask);
  
             // For the porposes of inventory, an object is modify if the prims
             // are modify. This allows renaming an object that contains no
@@ -586,19 +586,16 @@ namespace OpenSim.Region.CoreModules.Framework.InventoryAccess
 
                 effectivePerms &= groupPerms;
             }
-            effectivePerms |= (uint)PermissionMask.Move;
-
+ 
             if (remoteClient != null && (remoteClient.AgentId != so.RootPart.OwnerID) && m_Scene.Permissions.PropagatePermissions())
             {
-                if ((effectivePerms & (uint)PermissionMask.FoldedCopy) == 0)
-                    effectivePerms &= ~(uint)PermissionMask.Copy;
-                if ((effectivePerms & (uint)PermissionMask.FoldedTransfer) == 0)
-                    effectivePerms &= ~(uint)PermissionMask.Transfer;
-                if ((effectivePerms & (uint)PermissionMask.FoldedExport) == 0)
-                    effectivePerms &= ~(uint)PermissionMask.Export;
-                
+                PermissionsUtil.ApplyNoModFoldedPermissions(effectivePerms, ref effectivePerms);
+
                 uint basePerms = effectivePerms & so.RootPart.NextOwnerMask;
                 
+                // rebuild folded perms since we don't have then on inworld objects
+                // possible existent ones where already unfolded
+
                 if((basePerms & (uint)PermissionMask.Copy) == 0)
                     basePerms |= (uint)PermissionMask.Transfer;
 
diff --git a/OpenSim/Region/CoreModules/World/Objects/BuySell/BuySellModule.cs b/OpenSim/Region/CoreModules/World/Objects/BuySell/BuySellModule.cs
index f90285d..84f33d1 100644
--- a/OpenSim/Region/CoreModules/World/Objects/BuySell/BuySellModule.cs
+++ b/OpenSim/Region/CoreModules/World/Objects/BuySell/BuySellModule.cs
@@ -208,19 +208,13 @@ namespace OpenSim.Region.CoreModules.World.Objects.BuySell
                 
                 perms = group.GetEffectivePermissions(false);
                
-//                if((perms & (uint)PermissionMask.FoldedMask) != 0)
-                {
-                    if ((perms & (uint)PermissionMask.FoldedCopy) == 0)
-                        perms &= ~(uint)PermissionMask.Copy;
-                    if ((perms & (uint)PermissionMask.FoldedTransfer) == 0)
-                        perms &= ~(uint)PermissionMask.Transfer;
-                    if ((perms & (uint)PermissionMask.FoldedModify) == 0)
-                        perms &= ~(uint)PermissionMask.Modify;
-                    if ((perms & (uint)PermissionMask.FoldedExport) == 0)
-                        perms &= ~(uint)PermissionMask.Export;
-                }
+                PermissionsUtil.ApplyFoldedPermissions(perms, ref perms);
+
                 item.BasePermissions = perms & part.NextOwnerMask;
-                item.CurrentPermissions = perms & part.NextOwnerMask;
+
+// we need to rebuild folded here
+
+                item.CurrentPermissions = item.BasePermissions;
                 item.NextPermissions = part.NextOwnerMask;
                 item.EveryOnePermissions = part.EveryoneMask &
                                            part.NextOwnerMask;