Skip to content

Some fixes for delete all boons/traits.#19

Open
adi1998 wants to merge 2 commits intoPonyWarrior:mainfrom
adi1998:main
Open

Some fixes for delete all boons/traits.#19
adi1998 wants to merge 2 commits intoPonyWarrior:mainfrom
adi1998:main

Conversation

@adi1998
Copy link
Copy Markdown
Contributor

@adi1998 adi1998 commented Jan 2, 2026

2 issues this fixes

  • Delete all boons was also removing arcana traits.
  • The deletion only deleted about half the boons due to the trait table being modified mid loop.

Fixed them by

  • Using the boon manager population check instead of what was previously there.
  • Deep copying the Hero.Traits table before looping.

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