From 64f177e68fada231df23a4c0ffa03b25d12cdd8d Mon Sep 17 00:00:00 2001
From: ProfessorXZ <ssbiljan@gmail.com>
Date: Sat, 6 Aug 2016 22:54:58 +0200
Subject: [PATCH] Fix rope placement range checks. Fixes #1236

---
 CHANGELOG.md                 |  1 +
 TShockAPI/GetDataHandlers.cs | 10 +++++-----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 80318d35..4cc32b53 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,7 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin
 * Fixed an issue allowing clients to teleport NPCs to arbitrary locations (@ProfessorXZ)
 * Fixed a bug where players would get teleported to their previous location after dismounting the Unicorn Mount (@ProfessorXZ)
 * Players can no longer quick stack items into region protected chests (@ProfessorXZ)
+* Rope placement is no longer blocked by range checks (@ProfessorXZ)
 
 ## TShock 4.3.17
 
diff --git a/TShockAPI/GetDataHandlers.cs b/TShockAPI/GetDataHandlers.cs
index 7fd823cf..a1904eac 100755
--- a/TShockAPI/GetDataHandlers.cs
+++ b/TShockAPI/GetDataHandlers.cs
@@ -2021,7 +2021,7 @@ namespace TShockAPI
 				}
 				else if (action == EditAction.KillWall)
 				{
-					// If they aren't selecting an hammer, they could be hacking.
+					// If they aren't selecting a hammer, they could be hacking.
 					if (selectedItem.hammer == 0 && !ItemID.Sets.Explosives[selectedItem.netID] && args.Player.RecentFuse == 0 && selectedItem.createWall == 0)
 					{
 
@@ -2144,10 +2144,10 @@ namespace TShockAPI
 
 				// Ignore rope placement range
 				if ((editData != TileID.Rope
-					|| editData != TileID.SilkRope
-					|| editData != TileID.VineRope
-					|| editData != TileID.WebRope
-					|| action != EditAction.PlaceTile)
+					&& editData != TileID.SilkRope
+					&& editData != TileID.VineRope
+					&& editData != TileID.WebRope
+					&& action == EditAction.PlaceTile)
 					&& TShock.CheckRangePermission(args.Player, tileX, tileY))
 				{
 					args.Player.SendTileSquare(tileX, tileY, 4);