[Move implementation] Fully implementing forests curse and trick or treat

[PrabbyDD]

What are the changes the user will see?

Trick or treat and forests curse should now consistently properly interact with one another and with burn up/dual shock

Why am I making these changes?

fully implement the moves

What are the changes from a developer perspective?

Added a check in AddTypeAttr for cases related to these two moves

Screenshots/Videos

2024-10-22.13-41-19.mp4

2024-10-22.14-51-12.mp4

How to test the changes?

Override file, and new forests curse file

Checklist

• I'm using beta as my base branch
• There is no overlap with another PR?
• The PR is self-contained and cannot be split into smaller PRs?
• Have I provided a clear explanation of the changes?
• Have I considered writing automated tests for the issue?
• If I have text, did I make it translatable and add a key in the English locale file(s)?
• Have I tested the changes (manually)?
  • Are all unit tests still passing? (npm run test)
• Are the changes visual?
  • Have I provided screenshots/videos of the changes?

[innerthunder]

Duplicate of #4682

(guess this wasn't marked on the implementations issue 😖 )