Skip to content

Add missing doXMath on some variables#11003

Open
Eradev wants to merge 1 commit into
Card-Forge:masterfrom
Eradev:fix-activated-count
Open

Add missing doXMath on some variables#11003
Eradev wants to merge 1 commit into
Card-Forge:masterfrom
Eradev:fix-activated-count

Conversation

@Eradev

@Eradev Eradev commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Add missing doXMath on some variables so we can do stuff like /LimitMax.1.

@tool4ever

Copy link
Copy Markdown
Contributor

AbilityUtils alone currently has an "amazing" 228 doXMath calls...
Refactoring that number down might be a better fix instead instead of making that situation worse for a custom card 🤷‍♂️

@Jetz72

Jetz72 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Would it work to just shift the logic of the xCount method into a private one and turn the original into a wrapper that always applies doXCount?

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.

4 participants