Skip to content

[MSH] Implement Wonder Man, Hollywood Hero - #15700

Open
Grath wants to merge 3 commits into
magefree:masterfrom
Grath:grath/implement-wonder-man
Open

[MSH] Implement Wonder Man, Hollywood Hero#15700
Grath wants to merge 3 commits into
magefree:masterfrom
Grath:grath/implement-wonder-man

Conversation

@Grath

@Grath Grath commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Also change "max activations per game" to always be accessed via a Getter, and allow Power-Up abilities to be activated more times by overriding that Get method using the same design as Boast.

Grath added 2 commits July 19, 2026 16:44
Also change "max activations per game" to always be accessed via a Get helper method, and allow Power-Up abilities to be activated more times
).stream()
.map(p -> p.getAbilities(game))
.flatMap(Collection::stream)
.filter(PowerUpAbility.PowerUpMoreAbility.class::isInstance)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It must be implemented by replace effect, not fake ability. See example with any game events like DAMAGE_CAUSES_LIFE_LOSS.

So both getMaxActivationsPerGame and hasMoreActivationsThisTurn can use it. It's allow to make a shared game engine feature and support diff logic-calcs-combos with multiple effects.

            GameEvent maxActivationsEvent = new GameEvent(
                    GameEvent.EventType.MAX_ACTIVATIONS,
                    playerId, ability, playerId, maxActivationsPerGame, false);
            game.replaceEvent(maxActivationsEvent);
            return maxActivationsEvent.getAmount();

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried doing this, and while I could get it to show the ability as usable multiple times it didn't actually allow it to be activated more than once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants