Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Equipment statuses interfere/cancel regular statuses #7

Open
kiedtl opened this issue May 10, 2024 · 0 comments
Open

Equipment statuses interfere/cancel regular statuses #7

kiedtl opened this issue May 10, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@kiedtl
Copy link
Owner

kiedtl commented May 10, 2024

(This is a bug that I noticed a while back and recorded, where "a while back" is 1-2 years ago. I don't know if it still exists or not, though it likely does.)

Basically: if the player has a Tmp status, and then equips/dequips something that grants the Equ version of that status, the Tmp status is also canceled because the Equ status overwrites the Tmp status.

Two ways to fix it:

  • Painful refactoring, store statuses in a vector instead of an enum array
  • Ignore it and ensure it never happens in practice. I.e. don't add pieces of equipment that have statuses that could be inflicted by other means in a temporary way.
@kiedtl kiedtl added the bug Something isn't working label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant