Talk:Artifacts

From Stardew Valley Wiki
Revision as of 15:56, 11 April 2023 by Cooperka (talk | contribs) (fix code block)
Jump to navigation Jump to search

Fixing some off-by-1 floor inaccuracies

I recently made some edits to several specific artifact pages (such as Dwarvish Helm) to fix what I believe to be inaccuracies in the mine floors they can found on when tilling (for example, it should say floors 1-39 rather than floors 1-40, because the item cannot be found on floor 40 even if the soil were tillable).

The edits were reverted with no explanation, so I'd like to add more context here and get feedback from Margotbean on whether it's enough to justify making the changes. I now realize this Artifacts page also has the same inaccuracies, so if approved, I'll make the edits here along with the specific item pages.

The relevant source code is here for the item with item code 96 (Dwarf Scroll I) which does not depend on floor number; and here for items 97 (Dwarf Scroll II), 99 (Dwarf Scroll IV), 121 (Dwarvish Helm), 122 (Dwarf Gadget), etc. which do depend on floor number: case 40: is code for changing the type of item which is possible to receive upon entering floor 40.

Similarly, as described in The_Mines#Floors, the environment drastically changes on floors 40, 70, 80, 110, etc. so my proposed fix seems to align with the existing Mines wiki too: things are different between floor 39 and 40, rather than being different between floor 40 and 41.

Is this enough to justify my changes, or are you looking for something else? Thanks! Cooperka (talk) 15:56, 11 April 2023 (UTC)