Changes

==Done in Stardew Valley {{version|1.3.27}}==

==Done in Stardew Valley {{version|1.3.27}}==

===Bug fixes===

===Bug fixes===

* ☑ Fix strange null handling with net fields (mostly fixed).

* ☑ Fix strange null handling with net fields (mostly fixed).

==Done in Stardew Valley {{version|1.3.28}}==

==Done in Stardew Valley {{version|1.3.28}}==

===Small changes===

===Small changes===

* ☑ When creating honey, set <samp>preservedParentSheetIndex</samp> to the flower's item ID.

* ☑ When creating honey, set <samp>preservedParentSheetIndex</samp> to the flower's item ID.

==Done in Stardew Valley {{version|1.3.32}}==

==Done in Stardew Valley {{version|1.3.32}}==

===Small changes===

===Small changes===

<ul>

<ul>

==Done in Stardew Valley {{version|1.4}}==

==Done in Stardew Valley {{version|1.4}}==

===Bug fixes===

===Bug fixes===

* When a fence is placed, its <samp>maxHealth</samp> is assigned to the same <samp>NetFloat</samp> instance as <samp>health</samp>. This causes the max health to deteriorate over time until the game is reloaded. (To fix it, <code>this.maxHealth = this.health</code> should become <code>this.maxHealth.Value = this.health.Value</code>.)

* When a fence is placed, its <samp>maxHealth</samp> is assigned to the same <samp>NetFloat</samp> instance as <samp>health</samp>. This causes the max health to deteriorate over time until the game is reloaded. (To fix it, <code>this.maxHealth = this.health</code> should become <code>this.maxHealth.Value = this.health.Value</code>.)

==Done in Stardew Valley {{version|1.4.4}}==

==Done in Stardew Valley {{version|1.4.4}}==

===Bug fixes===

===Bug fixes===

* ☑ <samp>TrashBear.updateItemWanted</samp> is hardcoded to assume <samp>Data\CookingRecipes</samp> field 2 is an item ID, but it can actually be an item ID and count like <samp>272 5</samp> (e.g. see <samp>CraftingRecipe</samp> constructor). That means it crashes if it randomly chooses a recipe with multiple outputs.

* ☑ <samp>TrashBear.updateItemWanted</samp> is hardcoded to assume <samp>Data\CookingRecipes</samp> field 2 is an item ID, but it can actually be an item ID and count like <samp>272 5</samp> (e.g. see <samp>CraftingRecipe</samp> constructor). That means it crashes if it randomly chooses a recipe with multiple outputs.

==Done in Stardew Valley {{version|1.5}}==

==Done in Stardew Valley {{version|1.5}}==

===Bug fixes===

===Bug fixes===

* ☑ The game handles <samp>this.Window.ClientSizeChanged</samp> unsafely in <samp>Game1</samp>, so the resize logic can run in the middle of a draw/update cycle and [https://smapi.io/log/f29188142be44430a00c802eea75812f cause ObjectDisposedException crashes]. For most players the crash is rare (if they avoid resizing the window while it's loading something), but a minority of players report constant crashes. Proposed fix: when the window resize event is called, set a flag and perform the resize logic on the next update tick instead.

* ☑ The game handles <samp>this.Window.ClientSizeChanged</samp> unsafely in <samp>Game1</samp>, so the resize logic can run in the middle of a draw/update cycle and [https://smapi.io/log/f29188142be44430a00c802eea75812f cause ObjectDisposedException crashes]. For most players the crash is rare (if they avoid resizing the window while it's loading something), but a minority of players report constant crashes. Proposed fix: when the window resize event is called, set a flag and perform the resize logic on the next update tick instead.

==Done in Stardew Valley {{version|1.6}} (upcoming)==

==Done in Stardew Valley {{version|1.6}} (upcoming)==

===Small changes===

===Small changes===

* ☑ Change <code>Utility.isMale(name)</code> to check the NPC info instead of a hardcoded switch, so it works with custom NPCs.

* ☑ Change <code>Utility.isMale(name)</code> to check the NPC info instead of a hardcoded switch, so it works with custom NPCs.

* ☑ Remove <samp>Woods.stumps</samp> and use the new <samp>location.resourceClumps</samp> field instead.

* ☑ Remove <samp>Woods.stumps</samp> and use the new <samp>location.resourceClumps</samp> field instead.

* ☑ Tweak accessibility modifiers. <small>(Stardew Valley 1.6 marks a large number of methods <samp>virtual</samp> and <samp>public</samp>/<samp>protected</samp>, too many to list here.)</small>

<li>☑ Tweak accessibility modifiers:

* make field public: <samp>recolorOffsets</samp>.

* make fields public: <samp>Gil</samp>, <samp>talkedToGil</samp>.

* add <samp>ClearCachedConcessionTastes()</samp> method to reset <samp>_concessionTastes</samp>.

| <samp>Locations\ForgeMenu</samp>

* make methods virtual: <samp>CraftItem</samp>, <samp>GenerateHighlightDictionary</samp>, <samp>IsValidCraft</samp>, and <samp>IsValidCraftIngredient</samp>.

* make fields public: <samp>heldItem</samp>, <samp>hoveredItem</samp>.

| <samp>Menus\InventoryPage</samp>

* make fields public: <samp>hoveredItem</samp>, <samp>hoverAmount</samp>, <samp>hoverText</samp>, <samp>hoverTitle</samp>.

</ul>

===Medium changes===

===Medium changes===

* ☑ Some XNB files have a separate display name field, but only in non-English. Using display names consistently regardless of language would let mods rename things without breaking keys:

* ☑ Some XNB files have a separate display name field, but only in non-English. Using display names consistently regardless of language would let mods rename things without breaking keys:

** ☑ <samp>Data\Boots</samp>

** ☑ <samp>Data\Boots</samp>

** ☑ <samp>Data\CraftingRecipes</samp>

** ☑ <samp>Data\CraftingRecipes</samp>

** ☑ <samp>Data\CookingRecipes</samp>

** ☑ <samp>Data\CookingRecipes</samp>

** ☑ <samp>Data\Furniture</samp>

** ☑ <samp>Data\Furniture</samp>

** ☑ <samp>Data\Weapons</samp>

** ☑ <samp>Data\Weapons</samp>

===Refactoring===

===Refactoring===