fix(bank chests): fix ArgumentException when a bank chest is broken

CoderCow · CoderCow · commit 0454bb82afc6 · 2017-08-05T20:02:11.000+02:00
diff --git a/Implementation/UserInteractionHandler.cs b/Implementation/UserInteractionHandler.cs
@@ -2332,7 +2332,8 @@ public bool HandleTileEdit(TSPlayer player, TileEditType editType, int blockType
             }
           }
 
-          if (isChest) {
+          // note: if the chest was a bank chest, then it was already removed
+          if (isChest && TerrariaUtils.Tiles[location].active()) {
             ObjectMeasureData measureData = TerrariaUtils.Tiles.MeasureObject(location);
             DPoint chestLocation = measureData.OriginTileLocation;
             IChest chest = this.ChestManager.ChestFromLocation(chestLocation);

Original file line number	Diff line number	Diff line change
`@@ -2332,7 +2332,8 @@ public bool HandleTileEdit(TSPlayer player, TileEditType editType, int blockType`
`2332`	`2332`	`}`
`2333`	`2333`	`}`
`2334`	`2334`
`2335`		`- if (isChest) {`
	`2335`	`+ // note: if the chest was a bank chest, then it was already removed`
	`2336`	`+ if (isChest && TerrariaUtils.Tiles[location].active()) {`
`2336`	`2337`	`ObjectMeasureData measureData = TerrariaUtils.Tiles.MeasureObject(location);`
`2337`	`2338`	`DPoint chestLocation = measureData.OriginTileLocation;`
`2338`	`2339`	`IChest chest = this.ChestManager.ChestFromLocation(chestLocation);`