Skip to content

Fix base types of Dissolution of the Flesh and Rigwald's Curse - #10369

Open
mcagnion wants to merge 2 commits into
PathOfBuildingCommunity:devfrom
mcagnion:bugfix/unique-base-lines
Open

mcagnion wants to merge 2 commits into
PathOfBuildingCommunity:devfrom
mcagnion:bugfix/unique-base-lines

Conversation

@mcagnion

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

  • Dissolution of the Flesh was listed as a Prismatic Jewel. It drops as a Crimson Jewel, and the trade site only lists it under Crimson Jewel.
  • The current Rigwald's Curse variant had a space between {variant:3} and Wolf Alpha Talisman. The base line wasn't recognised, so the item stayed a Wereclaw Talisman, and the unparsed line was cached in ModCache.

Steps taken to verify a working solution:

  • Checked both base lines against the trade site's item list.
  • Loaded the current Rigwald's Curse variant: it now resolves to Wolf Alpha Talisman.
  • Regenerated ModCache.

Dissolution of the Flesh drops as a Crimson Jewel, not a Prismatic Jewel.
The trade site lists it only under Crimson Jewel.
The current variant's base line had a space after the variant tag, so it
did not match any base and the item kept Wereclaw Talisman. Regenerate
ModCache, which had cached that line as an unparsed mod.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant