Mantis#1207. Thank you, TGlion for a patch that addresses:

Implementation of llModifyLand() and There is a bug on 
permission-check of land-terraforming: x an y-coordinates 
are interchanged on function-call ExternalChecksCanTerraformLand.
Correct: x is west, and y is north. 2) Missing check of 
"Other allow to terraform-flag" (Parcel.ParcelFlags.AllowTerraform)

1 files changed, 11 insertions, 7 deletions

diff --git a/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs b/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
index 16743ea..abbf40e 100644
--- a/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
+++ b/OpenSim/Region/Environment/Modules/World/Permissions/PermissionsModule.cs
@@ -984,11 +984,9 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
                 DebugPermissionInformation(MethodInfo.GetCurrentMethod().Name);
                 if (m_bypassPermissions) return m_bypassPermissionsValue;
 
-                bool permission = false;
-
                 // Estate override
                 if (GenericEstatePermission(user))
-                    permission = true;
+                    return true;
 
                 float X = position.X;
                 float Y = position.Y;
@@ -1002,13 +1000,19 @@ namespace OpenSim.Region.Environment.Modules.World.Permissions
                 if (Y < 0)
                     Y = 0;
 
-                // Land owner can terraform too
                 ILandObject parcel = m_scene.LandChannel.GetLandObject(X, Y);
-                if (parcel != null && GenericParcelPermission(user, parcel))
-                    permission = true;
+                if (parcel == null)
+                    return false;
 
+                // Others allowed to terraform?
+                if ((parcel.landData.Flags & ((int)Parcel.ParcelFlags.AllowTerraform)) != 0)
+                    return true;
 
-                return permission;
+                // Land owner can terraform too
+                if (parcel != null && GenericParcelPermission(user, parcel))
+                    return true;
+
+                return false;
             }
 
             private bool CanViewScript(UUID script, UUID objectID, UUID user, Scene scene)