PlayerData changes

TShockAPI/DB/CharacterManager.cs

@@ -79,7 +79,7 @@ public CharacterManager(IDbConnection db)
 
 		public PlayerData GetPlayerData(TSPlayer player, int acctid)
 		{
-			PlayerData playerData = new PlayerData(player);
+			PlayerData playerData = new PlayerData(false);
 
 			try
 			{

TShockAPI/GetDataHandlers.cs

@@ -2619,9 +2619,9 @@ private static bool HandlePlayerSlot(GetDataHandlerArgs args)
 		private static bool HandleConnecting(GetDataHandlerArgs args)
 		{
 			var account = TShock.UserAccounts.GetUserAccountByName(args.Player.Name);//
-			args.Player.DataWhenJoined = new PlayerData(args.Player);
+			args.Player.DataWhenJoined = new PlayerData(false);
 			args.Player.DataWhenJoined.CopyCharacter(args.Player);
-			args.Player.PlayerData = new PlayerData(args.Player);
+			args.Player.PlayerData = new PlayerData(false);
 			args.Player.PlayerData.CopyCharacter(args.Player);
 
 			if (account != null && !TShock.Config.Settings.DisableUUIDLogin)

TShockAPI/PlayerData.cs

@@ -23,6 +23,7 @@ You should have received a copy of the GNU General Public License
 using Terraria.GameContent.NetModules;
 using Terraria.Net;
 using Terraria.ID;
+using System;
 
 namespace TShockAPI
 {
@@ -63,18 +64,27 @@ public class PlayerData
 		public int unlockedSuperCart;
 		public int enabledSuperCart;
 
-		public PlayerData(TSPlayer player)
+		/// <summary>
+		/// Sets the default values for the inventory.
+		/// </summary>
+		[Obsolete("The player argument is not used.")]
+		public PlayerData(TSPlayer player) : this(true) { }
+
+		/// <summary>
+		/// Sets the default values for the inventory.
+		/// </summary>
+		/// <param name="includingStarterInventory">Is it necessary to load items from TShock's config</param>
+		public PlayerData(bool includingStarterInventory = true)
 		{
 			for (int i = 0; i < NetItem.MaxInventory; i++)
-			{
 				this.inventory[i] = new NetItem();
-			}
 
-			for (int i = 0; i < TShock.ServerSideCharacterConfig.Settings.StartingInventory.Count; i++)
-			{
-				var item = TShock.ServerSideCharacterConfig.Settings.StartingInventory[i];
-				StoreSlot(i, item.NetId, item.PrefixId, item.Stack);
-			}
+			if (includingStarterInventory)
+				for (int i = 0; i < TShock.ServerSideCharacterConfig.Settings.StartingInventory.Count; i++)
+				{
+					var item = TShock.ServerSideCharacterConfig.Settings.StartingInventory[i];
+					StoreSlot(i, item.NetId, item.PrefixId, item.Stack);
+				}
 		}
 
 		/// <summary>
@@ -86,12 +96,22 @@ public PlayerData(TSPlayer player)
 		/// <param name="stack"></param>
 		public void StoreSlot(int slot, int netID, byte prefix, int stack)
 		{
-			if (slot > (this.inventory.Length - 1)) //if the slot is out of range then dont save
+			StoreSlot(slot, new NetItem(netID, stack, prefix));
+		}
+
+		/// <summary>
+		/// Stores an item at the specific storage slot
+		/// </summary>
+		/// <param name="slot"></param>
+		/// <param name="item"></param>
+		public void StoreSlot(int slot, NetItem item)
+		{
+			if (slot > (this.inventory.Length - 1) || slot < 0) //if the slot is out of range then dont save
 			{
 				return;
 			}
 
-			this.inventory[slot] = new NetItem(netID, stack, prefix);
+			this.inventory[slot] = item;
 		}
 
 		/// <summary>

TShockAPI/TSPlayer.cs

@@ -1267,7 +1267,7 @@ public void Logout()
 				}
 			}
 
-			PlayerData = new PlayerData(this);
+			PlayerData = new PlayerData();
 			Group = TShock.Groups.GetGroupByName(TShock.Config.Settings.DefaultGuestGroupName);
 			tempGroup = null;
 			if (tempGroupTimer != null)

docs/changelog.md

@@ -78,7 +78,9 @@ Use past tense when adding new entries; sign your name off when you add or chang
   * If there is no section called "Upcoming changes" below this line, please add one with `## Upcoming changes` as the first line, and then a bulleted item directly after with the first change. -->
 
 ## Upcoming changes
-Your changes could be here!
+* Added a constructor for `TShockAPI.PlayerData` that accepts the `includingStarterInventory` parameter, which is responsible for loading the TShock inventory.
+* Declared the constructor `TShockAPI.PlayerData` accepting the argument `TShockAPI.TSPlayer` obsolete.
+* Updated the `PlayerData.StoreSlot` method: Added an overload that takes `TShockAPI.NetItem`.
 
 ## TShock 5.2
 * An additional option `pvpwithnoteam` is added at `PvPMode` to enable PVP with no team. (@CelestialAnarchy, #2617, @ATFGK)