Merge pull request #1338 from ProfessorXZ/mintaka

A possible fix for #1337

CHANGELOG.md

@@ -26,6 +26,7 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin
 * Added `/uploadssc [player]` which allows someone to upload SSC data for [player] and store it on the server. Adds `tshock.ssc.upload` and `tshock.ssc.upload.others` permission nodes to match (@DogooFalchion).
 * Added hardened stone to the whitelist of tiles editable by players (@DogooFalchion).
 * Added conversion system to send convert old MOTD format into smart text, while preserving initial line starting values to keep byte optimization for background colors Thanks to (@WhiteXZ, @Simon311, and especially @DogooFalchion) for the hard work on this issue.
+* Fixed server-sided inventory issues caused by bank3 (@ProfessorXZ)
 
 ## TShock 4.3.20
 * Security improvement: The auth system is now automatically disabled if a superadmin exists in the database (@Enerdy).

TShockAPI/PlayerData.cs

@@ -180,7 +180,7 @@ namespace TShockAPI
 				{
 					//179-219
 					var index = i - (NetItem.InventorySlots + NetItem.ArmorSlots + NetItem.DyeSlots
-						+ NetItem.MiscEquipSlots + NetItem.MiscDyeSlots + NetItem.PiggySlots + NetItem.ForgeSlots);
+						+ NetItem.MiscEquipSlots + NetItem.MiscDyeSlots + NetItem.PiggySlots);
 					this.inventory[i] = (NetItem)forge[index];
 				}
 				else
@@ -464,4 +464,4 @@ namespace TShockAPI
 			NetMessage.SendData(39, player.Index, -1, "", 400);
 		}
 	}
-}
+}