Skip to content

[TMT] Implement Brilliance Unleashed#14545

Open
muz wants to merge 2 commits into
magefree:masterfrom
muz:tmt_brilliance_unleashed
Open

[TMT] Implement Brilliance Unleashed#14545
muz wants to merge 2 commits into
magefree:masterfrom
muz:tmt_brilliance_unleashed

Conversation

@muz
Copy link
Copy Markdown
Contributor

@muz muz commented Feb 23, 2026

Linked to #14006

@github-actions github-actions Bot added the cards label Feb 23, 2026
Card card = game.getCard(getTargetPointer().getFirst(game, source));
if (card != null && game.getState().getZone(card.getId()) == Zone.GRAVEYARD) {
controller.moveCards(card, Zone.BATTLEFIELD, source, game);
if (!card.isCreature()) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this method should have the game parameter, also check if it's a creature before moving it (not sure if that actually makes a difference though tbh)

Copy link
Copy Markdown
Contributor Author

@muz muz Feb 28, 2026

Choose a reason for hiding this comment

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

this method should have the game parameter

👍

also check if it's a creature before moving it

I was thinking about this when writing it and figured that it doesn't actually make a difference. Functionally the card gets moved regardless, the only thing that's conditional is it becoming the 3/3 if it's not already a creature, and per rules that would be based on printed properties on the card itself as I understand it.

Comment thread Mage.Sets/src/mage/cards/b/BrillianceUnleashed.java Outdated
@muz muz force-pushed the tmt_brilliance_unleashed branch from 9dfc66f to 004e941 Compare February 28, 2026 01:43
@muz muz requested a review from theelk801 February 28, 2026 01:43
game.addEffect(new BecomesCreatureTargetEffect(
new CreatureToken(3, 3, "3/3 Robot artifact creature with flying", SubType.ROBOT)
.withType(CardType.ARTIFACT).withAbility(FlyingAbility.getInstance()),
false, true, Duration.Custom
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

third param true - still a land??
I guess it's only text that isn't used, but still

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants